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
fix(providers): update providers config from verification walk-throughs #10582
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
Can you also modify the |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10582 +/- ##
==========================================
+ Coverage 38.53% 39.76% +1.22%
==========================================
Files 171 172 +1
Lines 27080 27638 +558
Branches 1117 1165 +48
==========================================
+ Hits 10435 10989 +554
- Misses 16645 16649 +4 ☔ View full report in Codecov by Sentry. |
): OAuthConfig<P> { | ||
const { issuer = "https://api.workos.com/" } = options | ||
const { issuer = "https://api.workos.com/", connection = "" } = options |
Check failure
Code scanning / CodeQL
Hard-coded credentials Critical
☕️ Reasoning
?scope
to theauthorizationUrl
checks: "nonce"
connection
argument / env var which gets appended toauthorziationUrl
as a search param@balazsorban44 two question:
checks: ("none" | "state" | "pkce")
, are the types wrong or isnonce
really an invalid value?AUTH_WORKOS_CONNECTION
. Usingprocess.env
directly in the provider seems wrong since that won't work in all environments.. Do you have a better idea for this one? 🤔🧢 Checklist
🎫 Affected issues
📌 Resources