-
Notifications
You must be signed in to change notification settings - Fork 753
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
Change sso callback verification #1092
Conversation
// .name, | ||
// }))); | ||
|
||
const computedPlatformOptions = computed(() => CrowdIntegrations.enabledConfigs.map((i) => ({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this seems like it doesn't belong here - maybe add a separate PR for this and explain what this is and add Joana as reviewer :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah it was for debugging one issue, already removed
This reverts commit 5aee96b.
This reverts commit 5aee96b.
(cherry picked from commit 5aee96b)
(cherry picked from commit 5aee96b)
Changes proposed ✍️
What
🤖 Generated by Copilot at 742932f
This pull request improves the SSO authentication flow by using the
jwks-rsa
package to verify JWT tokens from Auth0. It also updates the frontend and backend configurations to use the JWKS endpoint instead of the certificate file. Additionally, it modifies theNewActivityFilterOptions
component to show all the enabled platforms for creating new activities.
🤖 Generated by Copilot at 742932f
Why
How
🤖 Generated by Copilot at 742932f
cert
property withjwks
property inauth0
configuration to dynamically fetch public keys for JWT verification (link, link)jwks-rsa
package as a dependency and use its client and helper function inssoCallback
API handler to verify JWT tokens using JWKS endpoint (link, link, link)computedPlatformOptions
inNewActivityFilterOptions
component to useCrowdIntegrations.enabledConfigs
instead ofactive
value to display available platforms for creating new activities (link)Checklist ✅
Feature
,Improvement
, orBug
.