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

Support --secret flag setting x-kde-passwordManagerHint #177

Open
yorickvP opened this issue May 22, 2023 · 3 comments · May be fixed by #214
Open

Support --secret flag setting x-kde-passwordManagerHint #177

yorickvP opened this issue May 22, 2023 · 3 comments · May be fixed by #214
Labels
enhancement New feature or request

Comments

@yorickvP
Copy link

Some password managers, like kde wallet and keepassxc, mark copied passwords as "secret", by
offering the mimetype x-kde-passwordManagerHint with the value secret.
This stops clipboard managers from saving passwords in history, and optionally stops KDE Connect from forwarding this clipboard data to other devices.

It would be nice to have support for this in wl-clipboard, so I can safely use it to copy passwords from my password store. Adding arbitrary multi-mime copy seems complicated, but it should be easy to add a flag to mark copy data as secret.

@bugaevc bugaevc added the enhancement New feature or request label May 22, 2023
@bugaevc
Copy link
Owner

bugaevc commented May 22, 2023

Hm, sounds like a good idea indeed, thanks for bringing it to my attention!

It's kind of a shame that this is KDE-specific, I'd prefer a more generic solution at the Wayland protocol level. But on the other hand wl-clipboard has never shied away from making use of GTK- and wlroots-specific protocols, so why not support a KDE-specific MIME type as well.

It would probably make sense to also expose this is in wl-paste --watch as CLIPBOARD_STATE=sensitive in the new CLIPBOARD_STATE protocol, which is about to be released in 2.2.0. So the flag should probably be named --sensitive as well.

@yorickvP
Copy link
Author

I'm currently using this patch

@Cybso
Copy link

Cybso commented May 23, 2024

I wrote a similar request at https://bugs.kde.org/show_bug.cgi?id=487359 yesterday and was referred to the x-kde-passwordManagerHint. Is there any chance that this pull request gets merged into main?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants