xdotool type wrong passwords most of the time. about rofi-rbw HOT 16 CLOSED

--delay affects the time between keystrokes, so I guess when the typer starts you are releasing shift and it's only completely done after the second key was typed?

Removing the delay was added in #52, so I wouldn't want to remove it completely. We could make it configurable, but I'd prefer some solution that works around the problem. A xdotool keyup shift_l beforehand maybe?

from rofi-rbw.

I should have mention that no, I clearly do not touch the Shift key at all: I run rofi-rbw -t password -a type --typer xdotool in a shell loop, only pressing Enter to select the first entry each time, and looking at the result.

xdotool keyup shift_l doesn't help, nor run(["xdotool", "keyup", "Shift_L", "Shift_R", "Alt_L", "Alt_R"]) before and after the main xdotool command in type_characters().

from rofi-rbw.

Also, I did try with different delay settings, and in my case it looks like the issue is still triggered very often at 5, but I can't reproduce it at 6.

from rofi-rbw.

That helps, thank you very much! 🙂

Can you maybe share some "password" that triggers this for you? Which of the first letters should be capitalized and which should be lower case?

from rofi-rbw.

Sure, I ran pwgen, and got the first result with lower cases first: nei8daiK. Then, selecting this test one on rofi with for i in (seq 10); rofi-rbw -t password -a type --typer xdotool; end, I get:

nei8daiKNEI8daiKnei8daiKNEI8daiKnei8daiKnei8daiKNEI8daiKNEI8daiKNEI8daiKNEI8daiK

The first is OK, but the second is not.

(this is fish syntax)

from rofi-rbw.

Oh, this might be because there is a digit after the 3 first lowercases (which gets wrongly uppercased sometimes here), and on my keyboard layout, shift is required for digits. After a setxkbmap us, I have a correct 10/10 nei8daiKnei8daiKnei8daiKnei8daiKnei8daiKnei8daiKnei8daiKnei8daiKnei8daiKnei8daiK.

So my usual layout is setxkbmap fr bepo, but setxkbmap fr also trigger the issue.

from rofi-rbw.

Okay, that is interesting. I'm wondering if it's because of the number (but then shift would have to be applied too early) or because of the K at the end. Can you reproduce it without a number, just an upper case letter at the end?

What would your preferred solution be? I think the arguments from #52 are still valid, but rofi-rbw not working is also not acceptable. I'm leaning towards a small delay of 5 or 6, but what do you think?

from rofi-rbw.

Changing it to 6 seems to have made a difference for me. There are still occasions where I have to retry pasting.

from rofi-rbw.

You have the same problem of xdotool upper casing some characters? Do you also use a non-US keymap?

Does a larger delay than 6 work better for you?

from rofi-rbw.

You have the same problem of xdotool upper casing some characters? Do you also use a non-US keymap?

Was that the issue? I use a US keymap.

Does a larger delay than 6 work better for you?

Have not tried yet.

from rofi-rbw.

I thought the problem was related to the keymap, like @nim65s wrote:

After a setxkbmap us, I have a correct 10/10

At least, I have no other explanation why xdotool would reproducibly upper case the first letters characters by itself.

However, if you have the same problem, but with a US keymap, it has to be something different 😕

from rofi-rbw.

I thought the problem was related to the keymap, like @nim65s wrote:

After a setxkbmap us, I have a correct 10/10

At least, I have no other explanation why xdotool would reproducibly upper case the first letters characters by itself.

However, if you have the same problem, but with a US keymap, it has to be something different 😕

Could just be human error on my part ┐('～`；)┌

from rofi-rbw.

Okay, so is the original problem (some letters are upper cased by rofi-rbw) still happening for someone? If so, is a delay of 5 or 6 enough?

Or is there some other solution?

from rofi-rbw.

IIRC I had tried 5 and found 6 to work better.

from rofi-rbw.

I've pushed something to main where you can define the delay with --typing-key-delay. It's still 0 by default, but you can change it if you experience problems.

Could you please test it and tell me if it works?

from rofi-rbw.

I just removed the comments of lines "--delay" and "0" on the setup of mine that was having the issue, and I can't reproduce it anymore, so I can't properly test main, sorry.
But I think 714e8fc is a nice idea, and I will use it if I get this issue again, thanks a lot !

I think we can close this issue, then, unless somebody else currently can reproduce the issue, and report that 714e8fc is not working.

from rofi-rbw.