Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

When completing a compatibility SSO login, if the login was already consumed the error page is confusing #1865

Open
sandhose opened this issue Sep 28, 2023 · 1 comment
Labels
A-Compatibility-Layer Related to the legacy Matrix authentication compatibility layer A-Login-Flow Related to the user login flow O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Major functionality / product severely impaired, no satisfactory workaround. T-Enhancement New feature of request

Comments

@sandhose
Copy link
Member

When a compatibility SSO login is being completed, MAS redirects to the client redirect_uri. If that URI is a native callback, it can happen that the browser window stays open, and sometimes even reload.

When that happens, the SSO login gets "consumed" twice, which currently gives this feedback:

Two things to fix:

  • make the feedback more explicit (like "the login already completed" or something)
  • make the redirect happen ~client side? Like display an HTML page with an <meta http-equiv="refresh" content="0;{redirect_uri}"> or something, and a link on the page to open the callback
@sandhose sandhose added T-Enhancement New feature of request A-Compatibility-Layer Related to the legacy Matrix authentication compatibility layer A-Login-Flow Related to the user login flow O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Major functionality / product severely impaired, no satisfactory workaround. labels Oct 11, 2023
@matrixbot
Copy link
Member

For your information, this issue has been copied over to the Element fork of matrix-authentication-service: element-hq/matrix-authentication-service#1865

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Compatibility-Layer Related to the legacy Matrix authentication compatibility layer A-Login-Flow Related to the user login flow O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Major functionality / product severely impaired, no satisfactory workaround. T-Enhancement New feature of request
Projects
None yet
Development

No branches or pull requests

2 participants