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

build(deps): update dgrijalva/jwt-go to patched version #324

Merged
merged 1 commit into from
Jan 6, 2022

Conversation

ebaron
Copy link
Member

@ebaron ebaron commented Jan 4, 2022

This PR uses module replacement to patch the vulnerable transitive dependency, now maintained in a fork. Once we upgrade the Operator SDK (#287), we can upgrade client-go and remove this replacement.

Fixes: #323

Copy link
Member

@andrewazores andrewazores left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Patch is pretty straightforward, builds and appears to run as expected.

@ebaron ebaron merged commit 9117977 into cryostatio:main Jan 6, 2022
mergify bot pushed a commit that referenced this pull request Jan 6, 2022
ebaron added a commit that referenced this pull request Jan 6, 2022
(cherry picked from commit 9117977)

Co-authored-by: Elliott Baron <ebaron@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Operator has vulnerable dgrijalva/jwt-go dependency
2 participants