Skip to content
This repository has been archived by the owner on May 10, 2018. It is now read-only.

Keepassx Not Responding with 1.8.0 #1484

Closed
ghost opened this issue Oct 5, 2014 · 32 comments
Closed

Keepassx Not Responding with 1.8.0 #1484

ghost opened this issue Oct 5, 2014 · 32 comments

Comments

@ghost
Copy link

ghost commented Oct 5, 2014

Hi,

I'm using Slack 14.1 x86_64
QT5 5.2.1
WebKit version 538.1
keepassx-0.4.3

When using Keepassx and pressing my key shortcut I assigned to it, nothing happens, my username and password will not get filled in and auto completed on any site I've tried.

When I compiled QupZilla against QT4 which gave me 534.34 it was working.

@JHooverman
Copy link
Member

Please checkout this tutorial, it works flawlessly.

@ghost
Copy link
Author

ghost commented Oct 7, 2014

Let's not make assumptions when people make bug reports, I've been using Slackware and KeepassX for 14 years, I know how to use it and it doesn't work at all with 1.8.0 compiled against QT5 5.2.1 with WebKit version 538.1

I'll try it later with QT 5.3.2 and see if it changes it and report back...

@JHooverman
Copy link
Member

I was trying to reproduce your issue and found a working solution in this tutorial, which I wanted to share with you. However, I am using Qt-5.4.0.
I am sorry for making you feel uncomfortable.

@ghost ghost mentioned this issue Oct 7, 2014
@ghost
Copy link
Author

ghost commented Oct 7, 2014

Sorry too if I came across harsh/rude, didn't mean to be...

I'm not uncomfortable, I'm just reporting a bug is all...

The Global Auto-Type shortcut keys don't work with QupZilla using this QT version I'm on.

The BUILDING file also says this; QupZilla requires Qt (>=4.7.0) and QtWebKit (at least version included in Qt 4.7)

So now it makes me wonder should I really be compiling against QT5 in the first place?

Hmm

@ghost
Copy link
Author

ghost commented Oct 17, 2014

Still not working with QT 5.3.2

@The-Compiler
Copy link

This is probably this issue in Qt and KeePassX:

https://bugreports.qt-project.org/browse/QTBUG-42209
https://www.keepassx.org/dev/issues/236

@ghost
Copy link
Author

ghost commented Dec 12, 2014

Well I'm waiting on 5.4 and see how that goes...

@pejakm
Copy link
Member

pejakm commented Dec 12, 2014

I just installed qt-5.4: webkit's version number hasn't changed (still at 538.1) but it feels that browsing is a bit faster then with qt-5.3's edition. :)

@ghost
Copy link
Author

ghost commented Dec 29, 2014

I just compiled 1.8.5 with QT 5.4, WebKit version 538.1 and the KeePassX Auto Type is not working.

So I'm clear here, I'm saying that when I use the setting; Global Auto-Type Shortcut which is listed in the Settings - Advanced, and I either go to a site with only one account and KeePassX auto fills in the user/password, or I go to a site with more than one account, and I get the Auto Type popup window to click the account I want, neither of these actions work with the assigned keys I use for this to happen.

I use Ctrl z for KeePassX for the Global Auto-Type Shortcut

Oh SAD, I was so hoping this was going to work with QT 5.4... :(

@JHooverman
Copy link
Member

If you have a non-latin keyboard (do you?), you might be affected by this upstream issue as well.

@ghost
Copy link
Author

ghost commented Dec 29, 2014

I have the typical US 104 Qwerty keys...

@ghost
Copy link
Author

ghost commented Dec 29, 2014

Oh, that was reported over a year ago, and QT is still having keyboard shortcut issues, that's just nonsense...

@ghost
Copy link
Author

ghost commented Jan 3, 2015

I was told on this report below that the shortcut key issues is on non-latin keyboard layouts and that has nothing to do with US keyboard layouts;

https://bugreports.qt-project.org/browse/QTBUG-32274#comment-268986

I'm using a US keyboard layout...

@JHooverman
Copy link
Member

@MouLei: I did the following to reproduce your issue (please let me know, when I am on the wrong way) with keepassx-2.0 alpha 6:

  • Set Global Auto-Type Shortcut to CRTL+z via Extras -> Settings -> General -> Global Auto-Type Shortcut

kp1

  • Check Enable Auto-Type for this entry in a test account, here tumblr
  • Verify, that QupZilla is up and running
  • Press + to add a window title for applying CRTL+z (should be QupZilla), see image

kp2

Now, keepassx does not offer an entry for QupZilla to be selected. Qt does currently not set the WM_NAME property. This might be the reason, that auto-completion will not work.

@ghost
Copy link
Author

ghost commented Jan 5, 2015

I'm using 0.4.3

The first screenshot is like 0.4.3

The second seems to be like the Tools entry in 0.4.3 where you can pick either of these settings;

Auto-Type: Customize Sequence
Auto-Type: Select Target Window

So if the second screenshot is for the Select Target Window I rarely have to use this in Firefox with KeePassX, as example, maybe out of 50 sites, I only use it on 1-2...

@JHooverman
Copy link
Member

In the meantime I did some tests and I found out, that the KeePassX global auto-type only works as you already stated with QupZilla compiled against qt4. Furthermore, this seems to work only with KeePassX-0.4.3 and not with version 2.0 alpha6 (tested with openSUSE Tumbleweed only).

The global auto-type option works as follows: If you enter in the browser window the combination of the global hot keys from the keyboard, KeePassX searches the entire password database for a matching entry. KeePassX takes the title of the currently opened window (eg Firefox) and then searches all records in the database for a matching title line entry.

This is exactly the reason why this does not work in QupZilla compiled against Qt5: Qt5 does currently not set the WM_NAME property. The global auto-type option supposes to work, once this Qt5 bug is resolved.

@ghost
Copy link
Author

ghost commented Jan 8, 2015

I've waited a year to use QupZilla, oh well ANOTHER FLIPPIN QT BUG! :(

Errrr

THANKS

@The-Compiler
Copy link

Keep in mind this also was fixed in KeePassX, so you can as well update that (or use a git version): https://www.keepassx.org/dev/issues/236

@JHooverman
Copy link
Member

@The-Compiler: Do I understand correctly, that using a current keepassx version from git will make the global auto-type option work with Qt5?

@The-Compiler
Copy link

I believe so, yes. Though I haven't tested it.

Basically there a WM_NAME atom (window title in an essentially unknown encoding), and _NET_WM_NAME (window title in UTF-8). The older standard (ICCM) does not explicitely require WM_NAME, and the newer standard (EWMH) doesn't say anything about WM_NAME when _NET_WM_NAME is set. Due to a wrong comparison this case was handled incorrectly in KeePassX.

So basically this is not a Qt bug, but a KeePassX bug - arguably Qt should set WM_NAME to improve interoperability, but it's not required to do that.

@JHooverman
Copy link
Member

Thank you for your explanation. I will compile a keepassx git-revision and report back back the result.

@JHooverman
Copy link
Member

@MouLei: The global auto-type option works!

Tested with:
Application version: 1.9.0 (c1901cd)
Qt version: 5.4.1
WebKit version: 538.1
Build time: Dec 28 2014 15:49:07
Platform: openSUSE Tumbleweed (Linux)

KeePassX: Latest revision from git (2.0 alpha 6 git revision 835c5411)

@The-Compiler: Thanks again!

@ghost
Copy link
Author

ghost commented Jan 9, 2015

@The-Compiler: Thanks

I am a geek but even I stay away from Alpha in Linux, beta apps are as far as I go...

But yes as they keep telling us Geeks, Alpha/Beta isnt the same in Windows as Unix... :)

@JHooverman did you also test this with QupZilla 1.8.5?

KeePassX 2 hasn't had an update in 9 months to either another Alpha or switching to Beta version...

Ahh roadmap shows 88% of the way to the first beta;

https://www.keepassx.org/dev/projects/keepassx/roadmap

@The-Compiler
Copy link

@MouLei Another possibility would probably be to build whatever version you're running currently yourself, and apply the patch on top of that.

@JHooverman
Copy link
Member

@MouLei: The global auto-type option works as well with the QupZilla package from the official repos from openSuSE:

Application version: 1.8.5
Qt version: 5.4.0
WebKit version: 538.1
Platform: openSUSE Tumbleweed (Linux)

KeePassX: Latest revision from git (2.0 alpha 6 git revision 835c5411)

KeePassX is located on github as well. If you check the last commits, you will see, that the last one has been committed on Dec 22, 2014, which is quite recent. The current git-repo can be downloaded here.

@ghost
Copy link
Author

ghost commented Jan 10, 2015

Ok thanks

@ghost
Copy link
Author

ghost commented Jan 29, 2015

Well I was hoping 1.8.6 might work with KeePassX 0.4.3, but no such luck...

Is it possible to get it working with 0.4.3? I don't want to use a Alpha version of KeePassX and it would be great to get this working with the Stable version, until 2 is released final...

@The-Compiler
Copy link

I don't think there's anything QupZilla can do about this - you will have to wait for a Qt fix, or update KeePassX, or apply that patch on your stable KeePassX.

@ghost
Copy link
Author

ghost commented Jan 30, 2015

What that .diff patch works for 0.4.3?

@The-Compiler
Copy link

I don't know, but it's worth a try ;)

@ghost
Copy link
Author

ghost commented Jan 30, 2015

Ahh I looked at the src and /src/autotype/x11/AutoTypeX11.cpp doesn't exist in 0.4.3, so that takes care of that...

@JHooverman
Copy link
Member

Solved in Qt-5.5.0 beta

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

No branches or pull requests

3 participants