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

crypto: Updated jwt to use crypto lib #16905

Merged

Conversation

michael-redpanda
Copy link
Contributor

Fixes: https://github.com/redpanda-data/core-internal/issues/1123

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.3.x
  • v23.2.x

Release Notes

  • none

src/v/security/jwt.h Outdated Show resolved Hide resolved
@michael-redpanda
Copy link
Contributor Author

Force push 03569e1:

  • Fixed compilation bug
  • Removed some cryptopp references

BenPope
BenPope previously approved these changes Mar 5, 2024
Copy link
Member

@BenPope BenPope left a comment

Choose a reason for hiding this comment

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

🔥

src/v/security/jwt.h Outdated Show resolved Hide resolved
src/v/security/jwt.h Outdated Show resolved Hide resolved
Fixes: redpanda-data/core-internal#1123

Signed-off-by: Michael Boquard <michael@redpanda.com>
@michael-redpanda
Copy link
Contributor Author

Force push dc465f9:

@vbotbuildovich
Copy link
Collaborator

@michael-redpanda michael-redpanda merged commit d4eec40 into redpanda-data:dev Mar 6, 2024
16 checks passed
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.

None yet

3 participants