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

Storing git (and other) credentials still broken ? #282

Closed
ghost opened this issue Jan 10, 2024 · 8 comments
Closed

Storing git (and other) credentials still broken ? #282

ghost opened this issue Jan 10, 2024 · 8 comments
Assignees

Comments

@ghost
Copy link

ghost commented Jan 10, 2024

#239
flathub/com.visualstudio.code#426
flathub/com.visualstudio.code#417

It has been almost 3-4 months and syncing is still broken on GNOME, atleast enabling access to the kwallet session bus has made it work on KDE again, are there any workaround/fixes currently available ?
How are other users dealing with the issue on GNOME ?

@noonsleeper noonsleeper self-assigned this Jan 18, 2024
@noonsleeper
Copy link
Collaborator

@larina3315 As user of gnome, I can say that I never rely on a DE (or a browser) to store anything sensible like passwords/credentials, are they already audit by a third party?

@ghost
Copy link
Author

ghost commented Jan 21, 2024

@larina3315 As user of gnome, I can say that I never rely on a DE (or a browser) to store anything sensible like passwords/credentials, are they already audit by a third party?

If I understand it correctly, it isn't an issue with GNOME itself, but with libsecret being broken with the VSCodium (and other electron ?) app(s) packaged as flatpak.
On KDE people can use Kwallet to have syncing work, but it has been broken on GNOME (and other DEs etc) utilizing libsecret for a very long time.

This has rendered VSCode/VSCodium practically unusable for people who rely on the inbuilt Github integration that VSCode/VSCodium provides, and sync being broken for so long has made the whole ordeal incredibly frustrating, thus the original intention of this issue was to discuss workarounds for the sync issue amongst affected users.

On top of that, there are currently 3 issues, linked in the original post, that are tracking the same issue under different naming that might sound confusing, So I linked all of them at the start to track current discussions.

@ghost
Copy link
Author

ghost commented Jan 29, 2024

I have tried to edit the manifest to workaround the sync issue (and have wayland enabled) by appending the following line alongside the other sed lines in the .yaml flatpak manifest

- sed -i '/Exec=codium/s/$/ --password-store=basic --enable-features=UseOzonePlatform --ozone-platform=wayland --enable-features=WaylandWindowDecorations/' usr/share/applications/codium.desktop

So that

Exec=codium --unity-launch %F

became

Exec=codium --unity-launch %F --password-store=basic --enable-features=UseOzonePlatform --ozone-platform=wayland --enable-features=WaylandWindowDecorations

however, when i manually build/install the flatpak and try to run it via flatpak run com.vscodium.codium-l3115 (I changed the appid temporarily) it doesn't work, but when i spawn a shell inside the app's container (?) using But, if I did flatpak run --command=sh --devel com.vscodium.codium-l3115

and then execute

codium --unity-launch %F --password-store=basic --enable-features=UseOzonePlatform --ozone-platform=wayland --enable-features=WaylandWindowDecorations

It works correctly.
This has been extremely confusing for me, as the same args seem to work and NOT work, depending on whether they're appended to the Exec= line or manually launched from inside the flatpak container (?) from a shell

@ghost
Copy link
Author

ghost commented Jan 29, 2024

It appears that flatpak basically ignores the Exec= line of the .desktop file, and seems to have it's own .desktop file ?

@noonsleeper
Copy link
Collaborator

Hi @larina3315, to add arguments to the CLI directly in the yaml manifest will be better to add them to https://github.com/flathub/com.vscodium.codium/blob/master/com.vscodium.codium.yaml#L133

But maybe edit the .desktop file after install will be more useful also you will keep the updates #162 (comment)

@noonsleeper
Copy link
Collaborator

It appears that flatpak basically ignores the Exec= line of the .desktop file, and seems to have it's own .desktop file ?

The DE launches the app using the exec line from desktop file maybe the problem is that the modifications on your .desktop is not calling flatpak but instead calls to codium?

Exec=/usr/bin/flatpak run --branch=stable --arch=x86_64 --command=codium --file-forwarding com.vscodium.codium --unity-launch --password-store=basic --enable-features=UseOzonePlatform,WaylandWindowDecorations --ozone-platform=wayland @@ %F @@

You need to add only one --enable-features argument and separate the values with , and remember @ on @@ %F @@

@ghost
Copy link
Author

ghost commented Feb 5, 2024

Hi @larina3315, to add arguments to the CLI directly in the yaml manifest will be better to add them to https://github.com/flathub/com.vscodium.codium/blob/master/com.vscodium.codium.yaml#L133

But maybe edit the .desktop file after install will be more useful also you will keep the updates #162 (comment)

thank you for pointing that out, I have submitted a PR with a workaround for the sync issue : #294

@ghost ghost mentioned this issue Mar 7, 2024
@ghost
Copy link
Author

ghost commented Mar 14, 2024

Closing as fixed by #319

@ghost ghost closed this as completed Mar 14, 2024
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant