PR#160: Option for the mode_yubikey_otp must be bytes and not string. -

fedora-packager

#160 Option for the mode_yubikey_otp must be bytes and not string.

Merged Mar 10, 2020 by mohanboddu. Opened Mar 10, 2020 by cverna.

cverna/fedora-packager yubikey into master

Download 160.patch

cverna commented Mar 10, 2020

This yubico library expect these parameters to be bytes
instead of string.

Also replacing the string formating by f-strings.

fixes #158

Signed-off-by: Clement Verna cverna@tutanota.com

cverna commented Mar 10, 2020

There is still a weird exception at the end, but at least we reach the part of the script that writes to the yubi and prints Success! Your Yubikey ID is .........................

Exception ignored in: <function YubiKeyHIDDevice.__del__ at 0x7faf4776d200>
Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/yubico/yubikey_usb_hid.py", line 138, in __del__
  File "/usr/lib/python3.7/site-packages/yubico/yubikey_usb_hid.py", line 345, in _close
  File "/usr/lib/python3.7/site-packages/usb/legacy.py", line 235, in releaseInterface
  File "/usr/lib/python3.7/site-packages/usb/util.py", line 217, in release_interface
AttributeError: 'NoneType' object has no attribute '_ctx'

zlopez commented Mar 10, 2020

LGTM

Pull-Request has been merged by mohanboddu

Mar 10, 2020

Metadata

Assignee

None

Tags

No Tags

fedora-packager

#160 Option for the mode_yubikey_otp must be bytes and not string. Merged Mar 10, 2020 by mohanboddu. Opened Mar 10, 2020 by cverna. cverna/fedora-packager yubikey into master

Metadata

#160 Option for the mode_yubikey_otp must be bytes and not string.

Merged Mar 10, 2020 by mohanboddu. Opened Mar 10, 2020 by cverna.

cverna/fedora-packager yubikey into master