-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[release/7.0] Fix isolated storage path on mobile #76254
Conversation
Tagging subscribers to this area: @dotnet/area-system-io Issue DetailsBackport of #75541 to release/7.0 /cc @steveisok @mkhamoyan Customer ImpactTestingRiskIMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.
|
Tagging subscribers to 'os-maccatalyst': @steveisok, @akoeplinger Issue DetailsBackport of #75541 to release/7.0 /cc @steveisok @mkhamoyan Customer ImpactTestingRiskIMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.
|
@mkhamoyan can you please fill out the template, add the @simonrozsival @steveisok can you please provide a code review sign-off? |
Approved by tactics via email. |
Approved by Tactics. Signed off. CI green. Ready to merge. |
Backport of #75541 to release/7.0
/cc @steveisok @mkhamoyan
Customer Impact
The path of a file created by IsolatedStorageFile.GetUserStoreForApplication() changes between different builds on iOS, making accessing any previous files created not possible.
Environment.ProcessPath
was being used as part of the identity hash that goes into the path and this appears to change version to version under iOS.The fix was to bring in legacy mono behavior that makes a fixed
.isolated-storage
folder for each app, without the identity hash.Testing
Automated tests and validated version bumps with a MAUI app.
Risk
Low