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

Fix error alert when importing files from external apps #1078

Merged
merged 4 commits into from
Jan 19, 2025

Conversation

keeshux
Copy link
Member

@keeshux keeshux commented Jan 19, 2025

URL.startAccessingSecurityScopedResource() fails in that case, but permission is not required at all.

Could reproduce by importing .ovpn file from a Telegram chat.

Required when browsing external apps, but not when it's the
external app that is calling us.
@keeshux keeshux self-assigned this Jan 19, 2025
@keeshux keeshux added the regression Used to work but doesn't anymore label Jan 19, 2025
@keeshux keeshux added this to the Hotfixes for v3 milestone Jan 19, 2025
@keeshux keeshux merged commit 524243e into master Jan 19, 2025
1 check passed
@keeshux keeshux deleted the bugfix/external-import branch January 19, 2025 10:59
@keeshux keeshux added iOS Related to the iOS app macOS Related to the macOS app and removed macOS Related to the macOS app labels Jan 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iOS Related to the iOS app macOS Related to the macOS app regression Used to work but doesn't anymore
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant