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

Add support for KeePassXC flatpak app #7

Closed
wants to merge 1 commit into from

Conversation

gasinvein
Copy link

This adds support for KeePassXC packaged as Flatpak.
The flatpak app places the socket at a path different from non-flatpak KeePassXC installation, $XDG_RUNTIME_DIR/app/org.keepassxc.KeePassXC/ (instead of directly under $XDG_RUNTIME_DIR/), so check the flatpak-specific path first, and if it exists - try to connect to the socket within it.

@varjolintu varjolintu self-requested a review April 20, 2022 16:40
@WhyNotHugo
Copy link
Contributor

Please see #8

This can now be further simplified, since Flatpak is no longer special-cased.

WhyNotHugo pushed a commit to WhyNotHugo/keepassxc-proxy-rust that referenced this pull request Jun 2, 2022
Rather than try a single path, try all supported paths, in the preferred
order. This avoids breaking compatibility with previous KeePassXC
versions.

See: keepassxreboot/keepassxc#8018
Fixes: varjolintu#8
Supercedes: varjolintu#7
@varjolintu
Copy link
Owner

Let's close this if #10 already solves it.

@varjolintu varjolintu closed this Jun 8, 2022
This pull request 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

Successfully merging this pull request may close these issues.

3 participants