Skip to content

Comments

[CORL-3221] Add an SSO token testing endpoint to Coral#4755

Merged
tessalt merged 8 commits intodevelopfrom
feat/CORL-3221-tokenTest
Feb 28, 2025
Merged

[CORL-3221] Add an SSO token testing endpoint to Coral#4755
tessalt merged 8 commits intodevelopfrom
feat/CORL-3221-tokenTest

Conversation

@nick-funk
Copy link
Contributor

@nick-funk nick-funk commented Feb 27, 2025

What does this PR do?

Adds an SSO token testing endpoint at: /api/tokenTest

i.e. http://localhost:3000/api/tokenTest

image

These changes will impact:

  • commenters
  • moderators
  • admins
  • developers

What changes to the GraphQL/Database Schema does this PR introduce?

None

Does this PR introduce any new environment variables or feature flags?

No

If any indexes were added, were they added to INDEXES.md?

N/A

How do I test this PR?

You can use https://jwt.io/ to modify and generate new tokens for testing.

Were any tests migrated to React Testing Library?

No

How do we deploy this PR?

  • Merge into develop

@nick-funk nick-funk requested review from kabeaty and tessalt February 27, 2025 10:42
@netlify
Copy link

netlify bot commented Feb 27, 2025

Deploy Preview for gallant-galileo-14878c canceled.

Name Link
🔨 Latest commit 5ed375a
🔍 Latest deploy log https://app.netlify.com/sites/gallant-galileo-14878c/deploys/67c1dfe2ab89350008fc046a

Copy link
Contributor

@kabeaty kabeaty left a comment

Choose a reason for hiding this comment

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

this is fantastic! just left a comment about a small typo that I noticed.

{
type: AnalysisMessageType.Error,
message:
"Payload does not match expect structure of a Coral SSO token.",
Copy link
Contributor

Choose a reason for hiding this comment

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

small typo, expect structure --> expected structure

Copy link
Contributor Author

Choose a reason for hiding this comment

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

is fixed

@tessalt tessalt merged commit 53f16dc into develop Feb 28, 2025
6 checks passed
@tessalt tessalt deleted the feat/CORL-3221-tokenTest branch February 28, 2025 16:37
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