-
-
Notifications
You must be signed in to change notification settings - Fork 656
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: disable SCIM for OIDC, nothing to test it on #6937
Conversation
Also checks scim enabled before saving
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
❌ Code Health Quality Gates: FAILED
- Declining Code Health: 1 findings(s) 🚩
- Improving Code Health: 1 findings(s) ✅
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.
I think you were supposed to do this in OidcAuth, not SamlAuth ?
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.
❌ Code Health Quality Gates: FAILED
- Declining Code Health: 1 findings(s) 🚩
- Improving Code Health: 1 findings(s) ✅
const { | ||
settings, | ||
enabled, | ||
setEnabled, | ||
assumeControlOfExisting, | ||
setAssumeControlOfExisting, | ||
newToken, | ||
tokenGenerationDialog, | ||
setTokenGenerationDialog, | ||
tokenDialog, | ||
setTokenDialog, | ||
loading: scimLoading, | ||
saveScimSettings, | ||
onGenerateNewTokenConfirm, | ||
} = useScim(); | ||
|
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.
✅ Getting better: Large Method
OidcAuth decreases from 307 to 266 lines of code, threshold = 120
const scimEnabled = useUiFlag('scimApi'); | ||
|
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.
❌ Getting worse: Large Method
SamlAuth increases from 283 to 285 lines of code, threshold = 120
Indeed, thanks @chriswk! Fixed now |
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.
❌ Code Health Quality Gates: FAILED
- Declining Code Health: 1 findings(s) 🚩
- Improving Code Health: 1 findings(s) ✅
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.
The change looks okay, I think there's some fixes to the flow that we'd need to do in another PR though
disable SCIM for OIDC, nothing to test it on
Also checks scim enabled before saving