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

Can't use app tokens since #326 #349

Closed
TrishGillett opened this issue Jan 27, 2025 · 3 comments · Fixed by #350
Closed

Can't use app tokens since #326 #349

TrishGillett opened this issue Jan 27, 2025 · 3 comments · Fixed by #350
Assignees
Labels
bug Something isn't working dependencies Pull requests that update a dependency file

Comments

@TrishGillett
Copy link
Contributor

👋 We're experiencing a regression in the ability to use app keys beginning with this PR. This is the error:

  File \"/project/.meltano/extractors/tap-github/venv/lib/python3.10/site-packages/tap_github/authenticator.py\", line 130, in generate_app_access_token
    jwt_token = generate_jwt_token(github_app_id, github_private_key)
  File \"/project/.meltano/extractors/tap-github/venv/lib/python3.10/site-packages/tap_github/authenticator.py\", line 116, in generate_jwt_token
    token = jwt.encode(payload, github_private_key, algorithm=algorithm)
  File \"/project/.meltano/extractors/tap-github/venv/lib/python3.10/site-packages/jwt/api_jwt.py\", line 74, in encode
    return api_jws.encode(
  File \"/project/.meltano/extractors/tap-github/venv/lib/python3.10/site-packages/jwt/api_jws.py\", line 160, in encode
    alg_obj = self.get_algorithm_by_name(algorithm_)
  File \"/project/.meltano/extractors/tap-github/venv/lib/python3.10/site-packages/jwt/api_jws.py\", line 99, in get_algorithm_by_name
    raise NotImplementedError(
NotImplementedError: Algorithm 'RS256' could not be found. Do you have cryptography installed?

The issue does not happen if I peg the tap-github version to (v1.8.0). Pegging to any newer commits causes the issue to happen, the first such commit being the merge commit for #326. @edgarrmondragon do you know of anything in the singer-sdk 0.35.2 -> 0.42.1 updates that could be related to this problem? Maybe something changing dependency handling?

@TrishGillett TrishGillett changed the title Can't use app tokens since v1.9.0 Can't use app tokens since #326 Jan 27, 2025
@edgarrmondragon
Copy link
Member

Hey @TrishGillett, thanks for reporting!

I think we might just need an explicit dependency on cryptography. I'll have a PR shortly.

@edgarrmondragon
Copy link
Member

#350

@edgarrmondragon edgarrmondragon self-assigned this Jan 27, 2025
@edgarrmondragon edgarrmondragon added bug Something isn't working dependencies Pull requests that update a dependency file labels Jan 27, 2025
@edgarrmondragon
Copy link
Member

v1.11.1 is tagged. Thanks @TrishGillett!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dependencies Pull requests that update a dependency file
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants