Skip to content
This repository was archived by the owner on Feb 20, 2025. It is now read-only.

NameError: name 'yubikey' is not defined #364

Open
Torxed opened this issue Sep 3, 2024 · 3 comments · May be fixed by #371
Open

NameError: name 'yubikey' is not defined #364

Torxed opened this issue Sep 3, 2024 · 3 comments · May be fixed by #371

Comments

@Torxed
Copy link

Torxed commented Sep 3, 2024 •
edited
Loading

  • YubiKey Manager version: 5.5.1
  • How was it installed?: pacman -S yubikey-manager-qt
  • Operating system and version: Arch Linux
  • YubiKey model and version: issue happens without inserting a yubikey as well
  • Bug description summary:

asciicast

Steps to reproduce

pacman -Sy yubikey-manager-qt
ykman-gui

I'm not sure if this is purely a packaging issue or a ykman-gui issue, but ykman works. Although there was no yubikey inserted.
@wolf-yuan-6115
Copy link

#320 seems already reported this

@Torxed
Copy link
Author

Torxed commented Sep 5, 2024 •
edited
Loading

Similar, but in my case ykman works. Just not ykman-gui. I could be wrong tho.

It's also old so not sure.

@taivlam taivlam mentioned this issue Jan 1, 2025
Open
@Disasm
Copy link

Disasm commented Jan 8, 2025

The issue is most likely caused by this removal: Yubico/yubikey-manager@2bcf81b
yubikey.py in this repo tries to import types which no longer exist:

yubikey-manager-qt/ykman-gui/py/yubikey.py

Lines 53 to 55 in 28dc02d

from ykman.otp import (
_PrepareUploadFailed as PrepareUploadFailed
, _prepare_upload_key as prepare_upload_key, generate_static_pw)

When I remove the relevant imports locally and build the application, GUI starts to work and recognizes the token I have.

@Disasm Disasm linked a pull request Jan 8, 2025 that will close this issue
Open
@avdv avdv mentioned this issue Jan 8, 2025
Merged
13 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add initial support for ykman 5.5 and fix error reporting Disasm/yubikey-manager-qt
3 participants
@Torxed @Disasm @wolf-yuan-6115