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

The telephone entry field is not disabled when telephone-to-account linkage is. #29395

Open
RokeJulianLockhart opened this issue Feb 28, 2025 · 1 comment
Labels
A-Error-Message A-User-Settings O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter

Comments

@RokeJulianLockhart
Copy link

RokeJulianLockhart commented Feb 28, 2025

Steps to reproduce

  1. Visit https://chat.fedoraproject.org/#/home.

  2. Select mx_UserMenu_userAvatar.

  3. Select mx_UserMenu_iconSettings.

  4. Navigate to "Phone number":

    <input type="text" label="Phone Number" autocomplete="tel-national" placeholder="Phone Number" id="mx_Field_17" value="" style="">
  5. Select +44.

  6. Enter 7840875616.

Outcome

What did you expect?

xor:

  1. The telephone number should be accepted.
  2. The entry form for a telephone number should be disabled or hidden.

What happened instead?

As @hoamer rather incomprehensibly reports at matrix-org/synapse/issues/4630, the user sees:

MatrixError: [403] Account phone numbers are not authorized on this server

Operating system

cpe:/o:fedoraproject:fedora:41

Browser information

firefox-135.0.1-1.fc41.x86_64

URL for webapp

chat.fedoraproject.org/#/home:~:text=Sign%20out-,Personal%20info,-Email%20addresses

Application version

Fedora Chat version: 1.11.92
Crypto version: Rust SDK 0.9.0 (5de53c7), Vodozemac 0.8.1

Homeserver

fedora.im

Will you send logs?

Yes

@t3chguy
Copy link
Member

t3chguy commented Feb 28, 2025

I don't see a Matrix API specific for MSISDN input. Only one for MSISDN+Email. So the client has no way of knowing it is disabled without you trying with a valid phone number which will receive a text message if it isn't disabled.

https://spec.matrix.org/v1.13/client-server-api/#m3pid_changes-capability

@t3chguy t3chguy added the X-Needs-Info This issue is blocked awaiting information from the reporter label Feb 28, 2025
@dosubot dosubot bot added A-Error-Message A-User-Settings O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist labels Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Error-Message A-User-Settings O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter
Projects
None yet
Development

No branches or pull requests

2 participants