Skip to content

Detect home server from matrix id #810

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
Open
Kaki-In wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Detect home server from matrix id #810

Kaki-In wants to merge 22 commits into from
+110 −37

Conversation

@Kaki-In
Copy link
Contributor

@Kaki-In Kaki-In commented Jul 21, 2022 •
edited
Loading

Detect home server from matrix id

Added functionnality :

  • you can now enter the user matrix id (e.g. @username:server.local) in the login input.
  • if the home server that you've entered into youre id is not the same that those into the home server input, it will be use instead to connect to the default matrix server.

Should fix this issue (#479).

@Kaki-In Kaki-In changed the title Detect hs from mxid Detect home server from matrix id Jul 21, 2022
bwindels
bwindels reviewed Jul 25, 2022
View reviewed changes
Copy link
Contributor

@bwindels bwindels left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few comments, haven't done a full review yet.

Kaki-In added 4 commits July 25, 2022 15:58
@Kaki-In
New corrections (from base of element-hq#810 pull request commentarie…
d75c937 
…s) :

 - fix methods miss;
 - corrected some identation mistakes.
@Kaki-In
Update about the serverName constant : missing the result id (from th…
c7c32a6 
…e regex)
@Kaki-In
Detected inattention mistake.
17730e8 
PasswordLoginViewModel.parseUsernameLogin() should return the username, not the servername.
@Kaki-In
Merge branch 'vector-im:master' into detect-hs-from-mxid
5542335
MidhunSureshR
MidhunSureshR reviewed Jul 26, 2022
View reviewed changes
Copy link
Member

@MidhunSureshR MidhunSureshR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some comments and questions:

bwindels
bwindels reviewed Jul 27, 2022
View reviewed changes
MidhunSureshR
MidhunSureshR reviewed Jul 29, 2022
View reviewed changes
Copy link
Member

@MidhunSureshR MidhunSureshR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some more ts changes:

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

Reviewers

@MidhunSureshR MidhunSureshR MidhunSureshR left review comments

+1 more reviewer

@bwindels bwindels bwindels left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Support .well-known in the way the spec says

3 participants

@Kaki-In @bwindels @MidhunSureshR