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.

Sign up for GitHub

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

Jump to bottom

I want to sign in with OIDC #42

Closed
Tracked by #103
manuroe opened this issue May 5, 2022 · 1 comment · Fixed by #145
Closed
Tracked by #103

I want to sign in with OIDC #42

manuroe opened this issue May 5, 2022 · 1 comment · Fixed by #145
Assignees
@pixlwave
Labels
A-Signin X-Needs-Design X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue Z-BBQ-Alpha Required for ElementX - BBQ - Alpha

Comments

@manuroe
Copy link
Member

manuroe commented May 5, 2022 •
edited by VolkerJunginger
Loading

OIDC starts to be ready. We should showcase it using ElementX-iOS Alpha. It will help the OIDC team to get more feedbacks and to improve the UX.

After #40, we need to detect from the selected homeserver .well-known the .m.authentication field (example) and to display a "Log in via OIDC" that will open the authentication web service.

A sample swift app is available here.
Test homeserver with OIDC: https://synapse-oidc.lab.element.dev/
Hydrogen with OIDC in action: https://hydrogen-oidc.lab.element.dev/
@manuroe manuroe added Z-BBQ-Alpha Required for ElementX - BBQ - Alpha A-Signin X-Needs-Design labels May 5, 2022
@manuroe manuroe mentioned this issue May 5, 2022
Closed
@stefanceriu stefanceriu mentioned this issue Jun 29, 2022
Closed
@pixlwave pixlwave self-assigned this Jul 7, 2022
@pixlwave
Copy link
Member

For a complete implementation this is blocked by matrix-org/matrix-rust-sdk#859.

However in the short term we can merge the prototype as discussed in #137.

@pixlwave pixlwave added the X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue label Jul 19, 2022
@pixlwave pixlwave mentioned this issue Jul 26, 2022
Merged
pixlwave added a commit that referenced this issue Jul 28, 2022
@pixlwave
#42: Proof of concept login via OIDC.
c20bc6b 
Uses AppAuth library in the app, ignoring token refresh
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pixlwave pixlwave

Labels
A-Signin X-Needs-Design X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue Z-BBQ-Alpha Required for ElementX - BBQ - Alpha
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Proof of concept login via OIDC. element-hq/element-x-ios
2 participants
@pixlwave @manuroe