Skip to content
This repository has been archived by the owner on Oct 24, 2024. It is now read-only.

chore: update configuration-validation dependency of jwt-verifier #952

Conversation

oleksandrpravosudko-okta
Copy link
Contributor

@oleksandrpravosudko-okta oleksandrpravosudko-okta commented Dec 2, 2020

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Adding Tests
  • Build related changes
  • CI related changes
  • Documentation changes
  • Other... Please describe: dependency update

What is the current behavior?

Issue Number: 340069

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Reviewers

@oleksandrpravosudko-okta oleksandrpravosudko-okta force-pushed the op-okta-340069-configuration-validation-dependency branch from fe036f9 to 5bb796f Compare December 2, 2020 14:24
@@ -86,11 +90,14 @@ describe('Jwt Verifier', () => {
});
}, LONG_TIMEOUT);

it('should fail if no kid is present in the JWT header', () => {
it('should fail if no kid is present in the JWT header and keys endpoint provides > 1 key', () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

to align with jwks-rsa update

@oleksandrpravosudko-okta oleksandrpravosudko-okta force-pushed the op-okta-340069-configuration-validation-dependency branch 2 times, most recently from 5bb796f to d9ceed0 Compare January 15, 2021 18:00

### Other

- [#952](https://github.com/okta/okta-oidc-js/pull/952) - Updates configuration-validation dependency of okta-jwt-verifier to 1.0.0
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: probably no need to mention okta-jwt-verifier since it's the changelog of jwt-verifier

@oleksandrpravosudko-okta oleksandrpravosudko-okta force-pushed the op-okta-340069-configuration-validation-dependency branch from d9ceed0 to 3e63397 Compare January 25, 2021 11:00
@oleksandrpravosudko-okta oleksandrpravosudko-okta deleted the op-okta-340069-configuration-validation-dependency branch January 25, 2021 11:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants