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

Validating MSISDN with vector.im does not work right now #19317

Closed
3 tasks done
kittykat opened this issue Oct 7, 2021 · 9 comments
Closed
3 tasks done

Validating MSISDN with vector.im does not work right now #19317

kittykat opened this issue Oct 7, 2021 · 9 comments
Labels
A-3PIDs A-Identity-Server A-User-Settings O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround T-Defect X-Needs-Community-Testing X-Regression Z-Community-Testing

Comments

@kittykat
Copy link
Contributor

kittykat commented Oct 7, 2021

Steps to reproduce

  1. Go to settings
  2. Under Phone numbers, enter a valid phone number and press "Add"
    -> SMS message is not received but UI implies that it succeeded

Expected result

Verification SMS is received.

Screenshot from 2021-10-07 10-16-25

Critical because it fails silently and there is no workaround, and occasional because it fails for everyone but it doesn't feel like we have many users trying to use it.

Proposed solution

@kittykat kittykat added T-Defect S-Critical Prevents work, causes data loss and/or has no workaround A-User-Settings A-3PIDs O-Occasional Affects or can be seen by some users regularly or most users rarely labels Oct 7, 2021
@t3chguy
Copy link
Member

t3chguy commented Oct 7, 2021

It doesn't work for users using that IS, it could work for users of different ISes, hardcoding is vector.im wouldn't be great

@kittykat
Copy link
Contributor Author

kittykat commented Oct 7, 2021

@t3chguy is there a way to check what IS is being used?

@t3chguy
Copy link
Member

t3chguy commented Oct 7, 2021

The URL yes, but we can't ask it if its using the OpenMarket API or not

@kittykat
Copy link
Contributor Author

kittykat commented Oct 7, 2021

Can the check only be done when the phone number is submitted or when the settings panel is loaded? As far as we know, it only doesn't work with Sydent as far as we know right now.

@t3chguy
Copy link
Member

t3chguy commented Oct 7, 2021

We know the URL of the IS at any point.

@kittykat kittykat changed the title Hide "Phone number" section in settings until it's fixed Warn users that validating MSISDN with sydent does not work right now Oct 7, 2021
@kittykat kittykat changed the title Warn users that validating MSISDN with sydent does not work right now Validating MSISDN with vector.im does not work right now Oct 7, 2021
@babolivier
Copy link
Contributor

Hide phone number setting for vector.im users or give a warning that it's not possible to add a phone number right now (allow users to remove existing phone numbers?)

This wouldn't cover the case of a user signed up to a homeserver which delegates its msisdn verification to an identity server (which afaik clients can't know about), however it looks like Element automatically configures new users with vector.im as their IS (@t3chguy feel free to correct me if I'm wrong), which means the number of users in this case would probably be fairly minimal.

@novocaine
Copy link
Contributor

upstream issue matrix-org/sydent#411

@novocaine
Copy link
Contributor

@kittykat can you re-test this please

@kittykat
Copy link
Contributor Author

I can add a phone number now.

I don't get any useful error messages for invalid (wrong number of digits) or numbers which don't exist (e.g. +441234567890) and the error message for an invalid verification code isn't passed on correctly (matrix-org/synapse/issues/11016)

I'm going to close this issue as it is now possible to add a phone number.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-3PIDs A-Identity-Server A-User-Settings O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround T-Defect X-Needs-Community-Testing X-Regression Z-Community-Testing
Projects
None yet
Development

No branches or pull requests

4 participants