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

Accounts with user ids containing forward slashes can't be loaded correctly #842

Open
TobiasFella opened this issue Dec 2, 2024 · 1 comment
Labels
bug/fix The library doesn't work as expected

Comments

@TobiasFella
Copy link
Member

Reported as https://bugs.kde.org/show_bug.cgi?id=491252

The user id is used as a key for QSettings, which uses forward slashes as separators. When listing the account, wrong groups are thus returned.

We need to somehow come up with a way of escaping forward slashes while not breaking existing logged-in accounts (at least those that do not contain forward slashes; for the other ones, logging in again is probably inevitable but also not problematic. I haven't been able to come up with a nice way to do that yet.

@TobiasFella TobiasFella added the bug/fix The library doesn't work as expected label Dec 2, 2024
@KitsuneRal
Copy link
Member

I was about to say that forward slashes in user ids are a deprecated legacy but no ;(
Why can't we "simply" percent-encode the forward slashes though?

@KitsuneRal KitsuneRal moved this to In work in libQuotient 1 Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/fix The library doesn't work as expected
Projects
Status: In work
Development

No branches or pull requests

2 participants