Skip to content

Commit

Permalink
[Unticketed] Add cryptopgraphy dependency so RS256 is available (#3485)
Browse files Browse the repository at this point in the history
## Summary

### Time to review: __2 mins__

## Changes proposed
Add dependency on crypto library so other digital signature approaches
are available

## Context for reviewers
We use RS256 encryption which requires crypto to be installed:
https://pyjwt.readthedocs.io/en/latest/installation.html

Locally this wasn't an issue, but getting an error non-locally. We setup
dependencies differently locally, so some test dependency probably
brings the library in.
  • Loading branch information
chouinar authored Jan 10, 2025
1 parent 830931f commit 91dff13
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
5 changes: 4 additions & 1 deletion api/poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion api/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ psycopg = { extras = ["binary"], version = "^3.1.19" }
pydantic-settings = "^2.0.3"
flask-cors = "^5.0.0"
opensearch-py = "^2.5.0"
pyjwt = "^2.9.0"
pyjwt = { extras = ["crypto"], version = "^2.9.0" }
newrelic = "10.4.0"
jinja2 = ">=3.1.5"
tenacity = "^8.0"
Expand Down

0 comments on commit 91dff13

Please sign in to comment.