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

riot-desktop open SSO in browser so user doesn't have to auth twice #37

Merged
merged 8 commits into from
Mar 3, 2020

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Mar 2, 2020

Requires element-hq/element-web#12590 & matrix-org/matrix-react-sdk#4158
Fixes element-hq/element-web#8247

Known issues:

  • Only works for packaged builds on Mac/Linux as we can't specify arguments to send to the executable.
  • Only works for the default --profile on Mac/Linux

The above are both limitations of app.setAsDefaultProtocolClient path and args being Windows-only.

All changes from this PR have been backported into riot-web and thus are duplicated in that PR anyone building using the old electron_app setup before it is discarded.

scripts/in-docker.sh Show resolved Hide resolved
@t3chguy t3chguy merged commit 6637787 into master Mar 3, 2020
@t3chguy t3chguy deleted the t3chguy/sso branch May 3, 2022 09:57
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.

SSO in desktop app depends on Electron bug
2 participants