-
Notifications
You must be signed in to change notification settings - Fork 21
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
GitHub Extension Requests Authentication Every Time on Gnome #379
Comments
Was just about to create an issue for this as I am on the same boat, under fedora sericea 41. I have found a workaround for the time being, which is to mimic what the native package does with the keyring:
Both the There is an error though related to a bug since 2016, but it seems we can disregard it for now: [20:1106/025334.237722:VERBOSE1:libsecret_util_linux.cc(169)] Dummy store to unlock the default keyring failed: invalid password returned from portal I already tried making a dummy store as well but the error above persists so don't bother with that one. Hope this helps. |
Hi @czhang03 and @zentasumu, maybe you can look at #239 also https://code.visualstudio.com/docs/editor/settings-sync#_recommended-configure-the-keyring-to-use-with-vs-code |
Ah yes I have already followed those instructions to no avail as well. And apologies, with the workaround I've mentioned here is the log:
|
I am fairly sure this is pretty recent, did codium recently changed the default secret storage from plain text to gnome keyring? |
Signal Seems to think that their secrete-storage problem is related to this issue: https://gitlab.gnome.org/GNOME/libsecret/-/issues/49 , maybe we can try similar workarounds as suggested in the issue. Source: flathub/org.signal.Signal#756 |
The flatpak is now based on the runtime On the core app, no changes on our end (VSCodium) have been made to change the default secret storage. But maybe VSCode did. |
@daiyam This problem was present for so long on this repo, that workaround looks fine, but I will look if libsecret can be compiled from a git repo instead of that artifact then flatpak-external-data-checker can take care or notify us when libsecret is updated. I'll add this on codium-insiders repo first to test. @czhang03 thanks. |
@zentasumu @czhang03 you can test this flathub/com.vscodium.codium-insiders#46 (comment) to confirm if this already fix the problem? |
It works. The necessary login entries are being created in the keyring now, and the credentials (for Gitlab Workflow extension in my case) doesn't need to be re-added upon re-launch. |
I think this issue probably can be safely closed, and if I find any bug during testing, I will try to reopen it. |
I preferred this issue open until we merge the changes in codium, as a remainder |
Step to reproduce
Comment
I think it is probably just some key is not written to the keyring. It seems like signal flatpak also have similar issue flathub/org.signal.Signal#723 .
I wonder if this is related to electron decided to only support a not-yet-alpha version of xdg-desktop-protal electron/electron#43819 , since "gnome-keyring implements the Secret portal backend" (https://wiki.archlinux.org/title/XDG_Desktop_Portal).
Please let me know if this is just me experiencing this issue...
The text was updated successfully, but these errors were encountered: