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 KeePassXC-Browser in Snaps #1598

Closed
freelock opened this issue Mar 2, 2018 · 10 comments
Closed

Support KeePassXC-Browser in Snaps #1598

freelock opened this issue Mar 2, 2018 · 10 comments

Comments

@freelock
Copy link

freelock commented Mar 2, 2018

With the new KeePassXC 2.3.0, my snap install automatically updates, and warns me about KeePassHTTP, provides a nice link to the migration guide to change to KeePassXC-Browser.

However, when I follow the first step on these instructions, I get the warning:

We're sorry, but KeePassXC-Browser is not supported for Snap releases at the moment.

Expected Behavior

Support KeePassXC-Browser.

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

  1. Open Tools -> Settings
  2. Click "Browser Integration"
  3. See "not supported" message.

Context

I wasn't aware of this change -- which seems like a great addition -- but searching the issue queue, I'm not finding any issue for it. Is this the right place?

I basically want to subscribe to an issue so I know when this feature is complete!

Debug Info

KeePassXC - Version 2.3.0
Revision: 4c0ed74
Distribution: Snap

Libraries:

  • Qt 5.7.1
  • libgcrypt 1.7.6-beta

Operating system: Ubuntu Core 16
CPU architecture: x86_64
Kernel: linux 4.4.0-116-generic

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • Legacy Browser Integration (KeePassHTTP)
  • SSH Agent
  • YubiKey
@phoerious
Copy link
Member

We're having difficulties connecting Snaps through native messaging with the browser extension and haven't been able to find a solution yet. If you require KeePassXC-Browser, please use the AppImage or the version from the PPA.

@hackel
Copy link

hackel commented Mar 7, 2018

Please do not remove KeePassHTTP support until this critical issue is resolved. It looks like it may require writing a snap interface just for keepassxc-proxy.

Hopefully browsers will add dbus support to their Native Messaging API to avoid this whole mess. Could something like systemd-bus-proxyd be used to do this?

@droidmonkey
Copy link
Member

To be clear, the browser feature works just fine with snaps. However, you cannot install the native messaging json file from the KeePassXC snap due to sandboxing. My idea to solve this is to have snap users download a separate script executed outside the snap env that will install the necessary native messaging json files for your browser.

@jaguil
Copy link

jaguil commented May 22, 2018

Hello,
droidmonkey, if not too complex, would you please detail the way to fix it manually?
Many thanks in advance.

@droidmonkey
Copy link
Member

I haven't worked on it yet, but its in the pipeline.

@alexanderadam
Copy link

alexanderadam commented Jan 6, 2021

Is the Snap helper script properly documented? It would be great if Snap could be detected and KeePassXC would show a hint that's very present.

Also is this related to issues like 405, 2930, 4353, 2890 or 3741?

And it's basically the same issue with Flatpak (29, 92, 2656, 1621763), isn't it?

@erik78se
Copy link

Does the snap version still require this hack? I'm trying to get the browser integration to work, but I'm not sure what the path is or if I should just stop using the snap version?

@droidmonkey
Copy link
Member

Yes this is still required and the Browser Plugin settings page tells you what to do. You can also just go to our downloads page and grab the script: https://keepassxc.org/download/

@siddolo
Copy link

siddolo commented Apr 27, 2021

Why was this issue marked as fixed? browser snaps are not yet supported

askBrowserSnap() {
    if (whiptail --title "Snap Choice" --defaultno \
            --yesno "Is this browser installed as a snap (usually NO)?" 8 60); then
        # BASE_DIR="$1"
        whiptail --title "Snap Choice" --msgbox "Sorry, browsers installed as snaps are not supported at this time" 8 50
        exit 0
    fi
}

@droidmonkey
Copy link
Member

It is impossible to support native messaging when a browser is running as a sandboxed snap. This is a limitation in snapd not keepassxc. That is also not what this issue is about.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants