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

Ability to disable all identity server functionality via the config file #5921

Closed
turt2live opened this issue Jan 4, 2018 · 13 comments · Fixed by matrix-org/matrix-react-sdk#3285

Comments

@turt2live
Copy link
Member

turt2live commented Jan 4, 2018

By setting the default_is_url to an empty string/null or something.

@lampholder
Copy link
Member

Interesting. What consequences would this have?

Threepid invites wouldn't work; what else?

@lampholder
Copy link
Member

Also, just so I'm not guessing, what's the reason behind this?

@turt2live
Copy link
Member Author

Someone wanted it for privacy concerns (I really should have linked the message). They were concerned that their users would accidentally use the identity server in a privacy-centric group.

The other thing (that I know of) to disable would be adding/editing 3pids.

@jasmir
Copy link

jasmir commented Jan 29, 2019

Is there any movement here? I'd like to add that using the .well-known/matrix/client stuff for it would be very convenient.
For example by "m.identity_server": { "base_url": "" }

@meskio
Copy link

meskio commented Mar 15, 2019

As a work around I've notice that having an 'identity server' that just replies an application/json {} in https://example.com/_matrix/identity/api/v1 works to keep riot happy and don't use an identity server.

But will be really nice to get an official way to disable the identity server.

@aaronraimist
Copy link
Collaborator

Would also be nice to have this when the identity server you wanted to use is down :)

@lampholder
Copy link
Member

#10094

@aaronraimist
Copy link
Collaborator

Doesn't matrix-org/matrix-react-sdk#3088 fix 90% of this? All you need to do now is make it also work with a completely blank field instead of only working if you enter a URL that isn't an identity server.

Related: #10274

@dbkr dbkr self-assigned this Aug 2, 2019
@dbkr dbkr removed their assignment Aug 2, 2019
@lampholder lampholder changed the title Ability to disable all identity server functionality Ability to disable all identity server functionality via the config file Aug 2, 2019
@lampholder
Copy link
Member

Updated the issue title to distinguish it from #10425

@dbkr dbkr self-assigned this Aug 6, 2019
@dbkr
Copy link
Member

dbkr commented Aug 6, 2019

Annoyingly we overlooked this in MSC2078 and the id_server param is still require in all situations, so spec-wise we'd be unable to register with an email at all if no IS were specified, even if the HS sends its own emails.

@turt2live points out that (again, spec-wise) a solution to this could be to abuse params from UIA to have a flag for requires_identity_server: true/false. Default true.

@rubo77
Copy link
Contributor

rubo77 commented Aug 17, 2019

Why is this closed?

@t3chguy
Copy link
Member

t3chguy commented Aug 17, 2019

Because it got implemented by #3285?

@rltas
Copy link

rltas commented Oct 29, 2024

Because it got implemented by #3285?

I sifted through related issues and commits but still can't figure it out - what's the right way to do it?

/e: nevermind, got it. The right way to do is to do nothing at all and just leave it out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants