Skip to content

Conversation

mrts
Copy link
Member

@mrts mrts commented Oct 26, 2021

WE2-586

@mrts mrts marked this pull request as draft October 26, 2021 20:10
@mrts mrts force-pushed the change-token-format-to-nonstandard-json-with-signature branch 3 times, most recently from 8c0bc4b to 45f6056 Compare November 1, 2021 16:04
@mrts mrts force-pushed the change-token-format-to-nonstandard-json-with-signature branch 13 times, most recently from 5602d9c to 87407b4 Compare November 12, 2021 19:23
@web-eid web-eid deleted a comment from lgtm-com bot Nov 12, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 12, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 12, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 12, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 12, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 12, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 12, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 12, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 12, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 12, 2021
@web-eid web-eid deleted a comment from sonarqubecloud bot Nov 15, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 15, 2021
@mrts mrts force-pushed the change-token-format-to-nonstandard-json-with-signature branch 2 times, most recently from 4b6ec3b to 2d0ffc9 Compare November 16, 2021 12:48
@web-eid web-eid deleted a comment from lgtm-com bot Nov 16, 2021
@mrts mrts force-pushed the change-token-format-to-nonstandard-json-with-signature branch from 2d0ffc9 to daf6f6a Compare November 30, 2021 17:28
@mrts mrts marked this pull request as ready for review November 30, 2021 17:28
@mrts mrts requested a review from metsma November 30, 2021 17:28
@mrts mrts force-pushed the change-token-format-to-nonstandard-json-with-signature branch 2 times, most recently from 5ad1780 to 6def04c Compare November 30, 2021 18:25
@lgtm-com
Copy link

lgtm-com bot commented Nov 30, 2021

This pull request fixes 1 alert when merging 6def04c into d7f291c - view on LGTM.com

fixed alerts:

  • 1 for Missing JWT signature check

@mrts mrts force-pushed the change-token-format-to-nonstandard-json-with-signature branch from 6def04c to b39c5c1 Compare November 30, 2021 18:42
@web-eid web-eid deleted a comment from lgtm-com bot Nov 30, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 30, 2021
@web-eid web-eid deleted a comment from lgtm-com bot Nov 30, 2021
@lgtm-com
Copy link

lgtm-com bot commented Nov 30, 2021

This pull request fixes 1 alert when merging b39c5c1 into d7f291c - view on LGTM.com

fixed alerts:

  • 1 for Missing JWT signature check

…g to final spec, rename exception base class to AuthTokenException

WE2-586

Signed-off-by: Mart Somermaa <[email protected]>
@mrts mrts force-pushed the change-token-format-to-nonstandard-json-with-signature branch from b39c5c1 to 60b7b85 Compare December 2, 2021 15:57
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 2, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

87.0% 87.0% Coverage
0.0% 0.0% Duplication

@lgtm-com
Copy link

lgtm-com bot commented Dec 2, 2021

This pull request fixes 1 alert when merging 60b7b85 into d7f291c - view on LGTM.com

fixed alerts:

  • 1 for Missing JWT signature check

@mrts mrts merged commit 2cbf9fd into main Dec 6, 2021
@mrts mrts deleted the change-token-format-to-nonstandard-json-with-signature branch December 6, 2021 14:03
@mrts mrts added this to the vNext milestone Jan 21, 2022
svenzik pushed a commit that referenced this pull request May 6, 2025
svenzik pushed a commit that referenced this pull request May 21, 2025
svenzik pushed a commit that referenced this pull request May 26, 2025
mrts pushed a commit that referenced this pull request May 26, 2025
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.

2 participants