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

Using more than one account on ipad removes all accounts #1416

Open
nocturne213 opened this issue Oct 29, 2024 · 6 comments
Open

Using more than one account on ipad removes all accounts #1416

nocturne213 opened this issue Oct 29, 2024 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@nocturne213
Copy link

Describe the bug
What did you do to trigger the bug?
Added a second account on ipad

Expected Behavior
What did you expect to happen?
I would be able to swap between accounts

Actual Behavior
What actually happened?
The next time i opened mlem both of my accounts were removed and i had to log in again.

To Reproduce
What steps do I need to take to trigger the bug?
Log into an account (i was using sipuli.xyz) then add a second account (i used mtgzone.com) then when opening a link from within a post mlem fully closed, when i reopened mlem neither account was available and mlem was showing a guest Lemmy.world account.

Mlem Information:
Version + Build number
If this started happening recently, write which version introduced the bug
I remember this happening when version 2.0 first came out, i almost exclusively use my sopuli account so I never tried again after the first time until tonight. I am using ver 2.0 (114) on ipad pro iOS 18.1.

Additional Context
Any other relevant information

@nocturne213 nocturne213 added the bug Something isn't working label Oct 29, 2024
@nocturne213
Copy link
Author

It appear to not be limited to iPad, but possibly certain instances. I was logged into mtgzone on both my iPhone and iPad and was logged out of all accounts on both devices. I previously had 2 or 3 different accounts logged in on the iPhone. The only difference was adding the mtgzone account.

@Sjmarf
Copy link
Contributor

Sjmarf commented Oct 30, 2024

Hmm, that's interesting. I've not seen this issue so it being related to mtg.zone is plausible. Accounts being wiped is a result of a decoding error when opening the app, normally... we should really make Mlem log those errors somewhere permanent to make debugging easier.

When the accounts gets wiped, an "error" popup might be shown - if it happens again and you tap on the "error" popup before it goes away, it'll show the exact error. On the off chance it happens again, having that error information would be useful for us.

@nocturne213
Copy link
Author

nocturne213 commented Oct 30, 2024

The operation couldn’t be completed. (Mlem.UserAccount.DecodingError error 1.)

after more testing it happens on iPhone as well as iPad. It happens when I am logged in only on mtgzone.com as well as logged into other instances as well. It was more noticeable on iPad because pretty much every time you minimize Mlem on iPad it crashes. But if you log in on iPhone, then swipe away Mlem and reopen it it gives the above error and logs out all accounts.

@Sjmarf
Copy link
Contributor

Sjmarf commented Oct 30, 2024

Thanks, that's narrowed it down a lot.

The issue is caused by Mlem not being able to retrieve the account authorization token from the keychain. I can't reproduce this issue myself, which makes the exact cause tricky to pinpoint and fix. For now, I've submitted a PR that makes Mlem ask for the account password if it can't retrieve the token, rather than wiping the accounts list. I'll keep looking for a proper fix.

@Sjmarf Sjmarf self-assigned this Oct 31, 2024
@nocturne213
Copy link
Author

I do not know if you're change made it into the weekly, but the issue persisted today after updating.

@Sjmarf
Copy link
Contributor

Sjmarf commented Nov 1, 2024

That change hasn't merged yet, no. It'll probably be in the next weekly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: To Triage
Development

No branches or pull requests

2 participants