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
dbus: non-UTF8 key names #955
Comments
Happened during |
@waht Seems like dbus does not accept arbitrary strings as file names. redis et al. likely have similar issues. Can you take a look? |
Yes this happens if the key name passed to As far as I can see Elektra does not care about encoding of key names too. So to fix this issue in the D-Bus plugin keyNames could be converted to UTF-8 if they contain non-UTF-8 characters but what should be used as source encoding? ASCII? |
Thank you! Documenting this fact should be enough for now. In the longer term we should extend the iconv plugin to also fix key names. |
We should find a proper solution to this. E.g. KDE kopete can have keys with UTF-8 as it allows user-specified status or kmix uses user-specified audio names as section names. @mpranj did you already try to use our dbus notifications with KDE or GNOME? |
Yes, I used it with GSettings and it was working fine, but I never specifically tested with non-UTF-8 characters. |
Did you check if the APIs even allow non-UTF-8 to end up in the KeySet? Nevertheless, we also need some protection to not trigger asserts in the libs that we use. |
I mark this stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping by writing a message here or create a new issue with the remainder of this issue. |
I closed this now because it has been inactive for more than one year. If I closed it by mistake, please do not hesitate to reopen it or create a new issue with the remainder of this issue. |
The text was updated successfully, but these errors were encountered: