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

Update single sign on guide #3131

Merged
merged 6 commits into from
Jul 11, 2024
Merged

Update single sign on guide #3131

merged 6 commits into from
Jul 11, 2024

Conversation

mooreds
Copy link
Contributor

@mooreds mooreds commented Jun 24, 2024

As part of https://linear.app/fusionauth/issue/DAN-54/finish-sessions-documentation I'm reviewing this single sign-on guide. I updated it to:

  • have the code use .env instead of hardcoding
  • use kickstart
  • use the RemoteCode component to pull in code so it stays up to date
  • fixed old dependencies
  • updated to use proper components for formatting (Breadcrumb, etc)

Would love to have some other eyes on these changes.

@alex-fusionauth alex-fusionauth self-requested a review July 2, 2024 20:00
Copy link
Contributor

@alex-fusionauth alex-fusionauth left a comment

Choose a reason for hiding this comment

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

Nothing stopping this from moving forward, just a few suggestions.


Here's a demo video of the single sign-on process from the end user perspective:

<YouTube id="UV9FW57dfbQ" />

### Caveat

If you are testing with Safari or Chrome on macOS, the multi application logout won't work due to browser quirks. However, if you set up TLS and change the redirects to happen over `https`, then it will work. FireFox v83.0 works with either scenario.
If you are testing these applications with a modern browser, logout won't work due to browser quirks when you are running over `http`. However, if you set up TLS and change the redirects to happen over `https`, then logout works.

## Other Scenarios

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should add a cover to the SSO with FusionAuth and Zendesk as well

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can you please add linear tasks for that? Or do you need me to do so?

mooreds added a commit to FusionAuth/fusionauth-example-node-sso that referenced this pull request Jul 11, 2024
which unfortunately doesn't work.
see FusionAuth/fusionauth-site#3131 (comment) for more details
This reverts commit 693dc39.

ngrok doesn't work, see #3131 (comment) for more details.
@mooreds mooreds merged commit 6126345 into main Jul 11, 2024
2 checks passed
@mooreds mooreds deleted the mooreds/update-single-sign-on branch July 11, 2024 23:15
This pull request was closed.
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.

2 participants