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

solo-python is broken #78032

Closed
d4g opened this issue Jan 19, 2020 · 6 comments
Closed

solo-python is broken #78032

d4g opened this issue Jan 19, 2020 · 6 comments

Comments

@d4g
Copy link
Contributor

d4g commented Jan 19, 2020

Describe the bug
The solo-python module/script is broken.
The function "make-credential" fails.

To Reproduce
Steps to reproduce the behavior:

solo key make-credential                                                                     ~
Touch your authenticator to generate a credential...
Traceback (most recent call last):
  File "/nix/store/wdhccfihy44jcy7cac9ix8ll5spfq3vh-python3.7-solo-python-0.0.21/bin/.solo-wrapped", line 9, in <module>
    sys.exit(solo_cli())
  File "/nix/store/zzmkaz2y9d9lzhkgimbfp6y20h8i3iv2-python3.7-click-7.0/lib/python3.7/site-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/nix/store/zzmkaz2y9d9lzhkgimbfp6y20h8i3iv2-python3.7-click-7.0/lib/python3.7/site-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/nix/store/zzmkaz2y9d9lzhkgimbfp6y20h8i3iv2-python3.7-click-7.0/lib/python3.7/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/nix/store/zzmkaz2y9d9lzhkgimbfp6y20h8i3iv2-python3.7-click-7.0/lib/python3.7/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/nix/store/zzmkaz2y9d9lzhkgimbfp6y20h8i3iv2-python3.7-click-7.0/lib/python3.7/site-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/nix/store/zzmkaz2y9d9lzhkgimbfp6y20h8i3iv2-python3.7-click-7.0/lib/python3.7/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/nix/store/wdhccfihy44jcy7cac9ix8ll5spfq3vh-python3.7-solo-python-0.0.21/lib/python3.7/site-packages/solo/cli/key.py", line 135, in make_credential
    host=host, user_id=user, serial=serial, output=True, prompt=prompt, udp=udp
  File "/nix/store/wdhccfihy44jcy7cac9ix8ll5spfq3vh-python3.7-solo-python-0.0.21/lib/python3.7/site-packages/solo/hmac_secret.py", line 44, in make_credential
    rp, user, challenge, extensions=hmac_ext.create_dict(), pin=pin
TypeError: make_credential() takes 2 positional arguments but 4 were given

Same behavior for python38

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

  • system: "x86_64-linux"
  • host os: Linux 5.4.12, NixOS, 20.03pre209690.c438ce12a85 (Markhor)
  • multi-user?: yes
  • sandbox: yes
  • version: nix-env (Nix) 2.3.2
  • channels(root): "nixos-20.03pre209690.c438ce12a85"
  • nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:

# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:
  - python37Packages.solo-python
  - python38Packages.solo-python
@d4g
Copy link
Contributor Author

d4g commented Jan 19, 2020

@wucke13

@veprbl
Copy link
Member

veprbl commented Jan 19, 2020

The upstream issue solokeys/solo1-cli#52

@veprbl veprbl added 2.status: wait-for-upstream Waiting for upstream fix (or their other action). and removed 2.status: wait-for-upstream Waiting for upstream fix (or their other action). labels Jan 19, 2020
@veprbl
Copy link
Member

veprbl commented Jan 19, 2020

Needs a bump to 0.0.23

@d4g
Copy link
Contributor Author

d4g commented Jan 19, 2020

The upstream issue solokeys/solo-python#52

Thanks for that. I was expecting it to be a nixpkgs issue, because of the recent update, but I did not check for an upstream issue.

@wucke13
Copy link
Contributor

wucke13 commented Jan 20, 2020

I'll prepare a PR

> result/bin/solo key verify
Please press the button on your Solo key
Register valid
Valid Solo Tap with firmware from SoloKeys.
> result/bin/solo key make-credential
Touch your authenticator to generate a credential...
[...]

Done.

@marsam
Copy link
Contributor

marsam commented Jan 21, 2020

Addressed by #78097

@marsam marsam closed this as completed Jan 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants