Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure compatibility with python-fido2 0.8 (fixes #49) #51

Merged
merged 1 commit into from
Jan 12, 2020

Conversation

windmueller
Copy link
Contributor

The Timeout has been removed from python-fido2 in version 0.8.

@nickray
Copy link
Member

nickray commented Jan 12, 2020

This is the new blessed way, yes?
Would replace #47

@windmueller
Copy link
Contributor Author

As I wrote in #49, Python is not my primary programming language. However, I tried to make the same adjustments as the authors of the library in the respective commit (Yubico/python-fido2@4c48977).

Please accept one of the pull requests and create a new release. Right now this package is broken for distributions shipping the new fido2 library, like Arch.

@nickray nickray merged commit bd1366c into solokeys:master Jan 12, 2020
@nickray
Copy link
Member

nickray commented Jan 12, 2020

Had some minor nits:

  • import order incorrect (make check and make fix to the rescue)
  • missing CHANGELOG.md entry
  • missing solo/VERSION bump

I did add these and published an update on PyPI.

Thanks for your contribution!

@nickray nickray mentioned this pull request Jan 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants