search

tiuub / KeeOtp2

KeeOtp2 is a plugin for KeePass. It provides a form to display one time passwords and is fully compatible with the built-in OTP function.
MIT License
126 stars 11 forks source link

QR code scanning only works on last one of multiple monitors #37

Closed meandthemachine closed 2 years ago

meandthemachine commented 2 years ago

Thank you for this plugin. I found this issue while using it.

https://github.com/tiuub/KeeOtp2/blob/bf8739a3c3ed6356b7b294784f6e04d60803ecbb/KeeOtp2/Forms/OtpInformation.cs#L562

The uri gets overwritten in every loop turn. Instead, the loop should be left as soon as uri is not null, I assume.

tiuub commented 2 years ago

Thank you for your submission. I'm working on it.

Kind regards

tiuub commented 2 years ago

Sorry, accidentally closed this issue beforehand. I fixed this issue. Check out version 1.5.8.1.

Kind regards

rasa commented 2 years ago

Amazing turnaround!

meandthemachine commented 2 years ago

I'm happy to confirm that version 1.5.8.1 works as expected regarding this issue. Thank you very much!