rohit/probo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Drop redundant SAML auth error helper comment

Browse Source 
Signed-off-by: Cursor Agent <cursoragent@cursor.com>

Co-authored-by: Bryan FRIMIN <bryan@frimin.fr>
This commit is contained in:
Bryan Frimin
2026-07-24 21:45:38 +00:00
committed by Cursor Agent
parent 00ed11e30b
commit 77c57cc4a5
1 changed files with 0 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
pkg/server/api/connect/v1/auth_error.go
View File

@@ -50,9 +50,6 @@ func redirectAuthError(w http.ResponseWriter, r *http.Request, code string) {
http.Redirect(w, r, redirectURL.String(), http.StatusFound) http.Redirect(w, r, redirectURL.String(), http.StatusFound)
} }
// authErrorCodeFromSAML maps known SAML assertion failures to a browser error
// code. Specific SAML reasons stay server-side only; users see a generic
// authentication failure to avoid leaking org/account state.
func authErrorCodeFromSAML(err error) (string, bool) { func authErrorCodeFromSAML(err error) (string, bool) {
if _, ok := errors.AsType[*saml.ErrSAMLDisabled](err); ok { if _, ok := errors.AsType[*saml.ErrSAMLDisabled](err); ok {
return authErrorAuthenticationFailed, true return authErrorAuthenticationFailed, true