Skip to content
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

[Bug] Update Webview2 userData folder location with the appropriate default value #2591

Closed
trwalke opened this issue Apr 29, 2021 · 3 comments

Comments

@trwalke
Copy link
Member

trwalke commented Apr 29, 2021

Continuation of issue here #2502

We are currently using %UserProfile%/.msal/webview2/data as the default location for the userData folder for webview2 but customers have complained that this can cause issues.

"it's important for packaged apps to store their app data within the %LocalAppData%\Packages\xxxxxxxxxxxx folder. Makes it much easier for clean install/uninstall, and enforces app isolation. Unless there is some magic to direct %UserProfile% in packaged apps (which there could be, but it didn't seem so in my preliminary tests), it's not going to get stored in the appropriate place if an app uses APPX/MSIX packaging."

We are waiting on the Webview2 team to provide a better implementation that will allow us to rely upon a default location provided by the webview2 component itself. (MicrosoftEdge/WebView2Feedback#297)

There are also suggestions provided:

  • (most preferred) Make the WebView2WebUi extendable without having to fork the entire repo, or have some extendable version of it that can be easily passed to .WithCustomWebUi()
  • Make the user data folder customizable (i.e. a property of EmbeddedWebViewOptions)
  • (least preferred, but acceptable) Handle the special case of detecting if the app is running with packaged identity and use a subfolder of %LocalAppData%\Packages\xxxxxxxxxxxx instead of %UserProfile%
@bgavrilMS
Copy link
Member

Blocked by partner

@pmaytak
Copy link
Contributor

pmaytak commented Jun 30, 2021

For tracking: MicrosoftEdge/WebView2Feedback#1410

@bgavrilMS
Copy link
Member

For AAD, please use WAM - https://aka.ms/msal-net-wam

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants