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

Registration disabled, hide register and login form except for OIDC providers #146

Closed
samip5 opened this issue Oct 24, 2021 · 5 comments
Closed
Labels
type: feature New feature or request

Comments

@samip5
Copy link

samip5 commented Oct 24, 2021

Describe the bug

When a Synapse Matrix server has disabled registration, the links should be hidden for registration aswell as hide login input fields but show OIDC authentication provider links.

To Reproduce

Steps to reproduce the behavior:

  1. Deploy Matrix Synapse with OIDC auth only enabled, registration disabled.
  2. Deploy Cinny, change default HS url.
  3. Go to Cinny and see that OIDC provider link has appeared but the login from and registration link is still visiable.

Expected behavior

The login input fields would be hidden and registration link too so only shown would be OIDC/SSO provider login method.

Screenshots

image

Additional context

Element does it like this:
image

@ajbura ajbura added the type: feature New feature or request label Oct 24, 2021
ajbura added a commit that referenced this issue Nov 6, 2021
Signed-off-by: Ajay Bura <ajbura@gmail.com>
@ajbura ajbura closed this as completed Nov 6, 2021
@simonwuelker
Copy link

@ajbura why is this closed? It is not fixed.
image

@ajbura
Copy link
Member

ajbura commented Jan 24, 2022

@Wuelle It has been fixed. Why you think it’s not fixed?

@simonwuelker
Copy link

simonwuelker commented Jan 24, 2022

Because this Registration link
grafik
still shows up, despite registration being disabled. Once you click it, you get
grafik
Two things:

  • In the first screenshot, the "Dont have an account? Register" link should not appear.
  • In the second screenshot, the "Already have an account? Login" link should not appear.
    (This is part of the issue the OP was referring to, so i did not open a new issue and commented on this one instead)
    I am using v1.6.1

@ajbura
Copy link
Member

ajbura commented Jan 24, 2022

@Wuelle OP was referring to hide login and registration form not these links.

The reason we’ve kept these links because,
a). User can know that registrations are disabled on xyz homeserver when they are on login page.
b). And if someone goes to registration page then can come back on login page.

@simonwuelker
Copy link

OP was referring to hide login and registration form not these links.

from the original issue:

The login input fields would be hidden and registration link too so only shown would be OIDC/SSO provider login method.

If you already thought about removing the link and decided against it, fair enough.
IMO the better way of letting the user know registration is disabled would be by not showing a registration link.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants