-
Notifications
You must be signed in to change notification settings - Fork 321
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Restart API not working with unpackaged apps in 1.1.3 #2792
Comments
@Balkoth thanks for providing the feedback. We will take a look at the issue and keep you posted on the updates |
With Template Studio 5.2 using 1.1.3, |
I can not confirm your observations. The attached sample was created with Template Studio 5.2 and does not ever restart. I can not find any relevant information on how to diagnose why. Eventlog does not show anything related in the Application log nor the AppLocker log. |
My next guess would be that it's a downlevel problem since you're on 19044. @kanismohammed are these APIs supported downlevel? |
@dhoehna - Can you please take a look? |
It was already answered here: #2792 (comment) The restart agent isn't being included when WASDK is self-contained. |
I can work on getting the restart agent to deploy with everything else. |
Thanks. There are two issues though. The self-contained issue is one. It's also possible the Restart API doesn't work for unpackaged apps downlevel. We need to investigate. |
Is there some more investigation ongoing or will this just be buried like all other bug reports? |
I'm looking into it. Progress has been slow, that's all. |
Sorry for the gap in communication. I'm reporting in that I did find the root cause of the issue. The root cause of the issue is in the OS code. More research is needed to figure out where a fix should go. Most likely Windows App SDK. Again, sorry for the delay in communication. It took a bit longer than expected to hunt down this issue. @mikebattista We might need to add internal hours to find a fix for this issue. |
@dhoehna I've already filed https://task.ms/40728649. Do we need another bug? |
Alrighty. Updating this. Sorry it took so long. The bug was deeper than I though. This is a platform issue in VB. The ACLs on the folder in Windows Apps is Low IL. What this means is, even if the parent app is medium-IL, any process made in the frameworks folder, for example, RestartAgent.exe, "...\windowsapps[Fwp]\RestartAgent.exe" is lowIL. Since restartAgent.exe creates the same process that was restarted, and, new processes inherit the parent processes IL, that means, the new process is also LowIL. Now I'll be talking with peeps to figure out "What do we do about this and how do we fix it." Again, sorry this is taking so long. I'm still working on it though. |
The ACLs are being changed and should be serviced in February. |
Will this also be fixed for unpackaged applications with |
Has this been serviced? |
Is this fixed? If so what needs to be updated on our ends? |
Sorry for the late reply. This change was serviced on February 2nd. To get the correct ACLs for Windows App SDK please either
Essentially the framework package needs to be removed, then installed. |
I'm currently looking into a way to verify the fix is on a system so I can let ya'll know if the patch is present. |
@dhoehna Thanks for the reply! This will work for self-contained as well? Will we need to upgrade to 1.3 preview/wait for stable? or Will the WASDK's Feb release 1.2.4 suffice? |
It is not fixed for self-contained apps as the original problem that |
Still not fixed in |
Any update on this? |
Is there any update on this issue? I have WindowsAppSDK 1.3.230724000 installed, and the problem is still there. My app is also an unpackaged application with |
It takes over one year and this issue is still there. Is that reasonable for WinUI team? |
The issue with RestartAgent.exe not being included in selfcontained apps is fixed in 1.4 preview 2. |
Describe the bug
Official documents https://docs.microsoft.com/en-us/windows/apps/windows-app-sdk/stable-channel#app-lifecycle--restart mention this should work for packaged and unpackaged apps, but using
Microsoft.Windows.AppLifecycle.AppInstance.Restart("");
as described here https://github.com/microsoft/WindowsAppSDK/blob/main/specs/AppLifecycle/Restart/restartApi.markdown#new-api
just crashes the app.
Steps to reproduce the bug
Just use this sample to verify:
TestRestartManager.zip
Expected behavior
No response
Screenshots
No response
NuGet package version
1.1.3
Packaging type
Unpackaged
Windows version
Windows 10 version 21H2 (19044, November 2021 Update)
IDE
Visual Studio 2022
Additional context
No response
The text was updated successfully, but these errors were encountered: