Skip to content

fix: prevent crash when reading user IDs from secure storage#460

Open
anthonioez wants to merge 1 commit into
dropbox:masterfrom
anthonioez:master
Open

fix: prevent crash when reading user IDs from secure storage#460
anthonioez wants to merge 1 commit into
dropbox:masterfrom
anthonioez:master

Conversation

@anthonioez

Copy link
Copy Markdown

Safely unwrap the "acct" keychain attribute using compactMap and as? String to prevent fatal errors when the value is missing or not a String.

Safely unwrap the "acct" keychain attribute using `compactMap` and `as? String` to prevent fatal errors when the value is missing or not a String.
@CLAassistant

CLAassistant commented Jun 5, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@shuyck

shuyck commented Jun 17, 2026

Copy link
Copy Markdown

Hi @anthonioez,

Thanks for opening this PR and for the proposed fix here.

Would you be able to share reproduction steps or a bit more context on when the fatal error happens when reading the user IDs from secure storage? In particular, it would be helpful to know what conditions lead to the acct keychain attribute being missing or not a String.

Those details will help us better understand the underlying issue and validate the fix. Thanks again for the contribution.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants