Skip to content

Update just the JWT core package to get the latest Token Factory.#180

Merged
Duslerke merged 1 commit intomasterfrom
test-adding-cognito-token-support
Feb 25, 2026
Merged

Update just the JWT core package to get the latest Token Factory.#180
Duslerke merged 1 commit intomasterfrom
test-adding-cognito-token-support

Conversation

@Duslerke
Copy link
Copy Markdown
Contributor

@Duslerke Duslerke commented Feb 25, 2026

What:

  • Update the Hackney.Core.JWT package 2 versions up (1.72 -> 1.84 -> 1.87).

Why:

  • To get the latest version of the ITokenFactory implementation that adds support for the Cognito token schema.
  • The current v1.72 implementation crashes upon encountering Cognito Token due to empty groups key failing to map and leading Token.Groups to getting set as unhandled null.

Notes:

  • This is just a preview version update.
  • Updating only the JWT core package without the Authorizer as after additional digging through source code I've found that Authorizer core package merely references the JWT core package's ITokenFactory interface definition, and does not do anything more with the package. The way the Google groups authentication setup works is that for any given API, it uses JWT core package to register TokenFactory instance DI, and then that gets used by the Authorizer middleware to populate its Auth filter dependencies. The rest of the JWT usage seems to be limited down to Token class to extract data like email or user name for Activity History purposes.
  • The version 1.84 was a fake release, accidentally triggered by some house cleaning done by the cyber team.
  • What changes does latest JWT intoduce? See PR: Add cognito authentication flow token schema support. lbh-core#68.

@Duslerke Duslerke requested a review from a team as a code owner February 25, 2026 11:10
Copy link
Copy Markdown

@manimaran-ramalingam manimaran-ramalingam left a comment

Choose a reason for hiding this comment

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

Approved

@Duslerke Duslerke merged commit 8417d7e into master Feb 25, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants