Skip to content
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

SSO troubleshooting #4330

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

SSO troubleshooting #4330

wants to merge 6 commits into from

Conversation

drernie
Copy link
Member

@drernie drernie commented Feb 18, 2025

Description

Draft troubleshooting guide for customers having SSO issues.

TODO

  • [X ] Documentation
  • Changelog entry (skip if change is not significant to end users, e.g. docs only)

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Added comprehensive SSO troubleshooting documentation focusing on Azure, Google Workspace, and Okta authentication redirect loop issues.

  • Added detailed step-by-step guide in /docs/Troubleshooting_SSO.md covering redirect URI verification, log analysis, and token claim verification
  • Included specific error codes (e.g., AADSTS50011, AADSTS50008) and their meanings for easier diagnosis
  • Added structured troubleshooting table with common issues and fixes
  • Integrated new SSO troubleshooting section into /docs/SUMMARY.md documentation hierarchy

2 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

Copy link

codecov bot commented Feb 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 39.09%. Comparing base (242b18f) to head (82bc8cd).

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #4330    +/-   ##
========================================
  Coverage   39.09%   39.09%            
========================================
  Files         783      783            
  Lines       34755    34755            
  Branches     5521     5521            
========================================
  Hits        13588    13588            
+ Misses      20624    19985   -639     
- Partials      543     1182   +639     
Flag Coverage Δ
api-python 91.39% <ø> (ø)
catalog 18.12% <ø> (ø)
lambda 91.48% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@drernie drernie requested review from akarve and QuiltSimon February 18, 2025 17:29
@drernie drernie requested a review from Copilot February 18, 2025 22:46

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

docs/Troubleshooting_SSO.md:67

  • [nitpick] The phrase 'Redirect loop after login' should be more specific. Consider changing it to 'Redirect loop after SSO login'.
| Redirect loop after login     | Verify redirect URIs and logout URL in Azure/Google/Okta |

docs/Troubleshooting_SSO.md:68

  • [nitpick] The phrase 'Check SSO provider logs for error codes' could be more specific. Consider changing it to 'Check SSO provider logs for specific error codes'.
| Authentication fails silently | Check SSO provider logs for error codes                  |
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.

1 participant