authgear / authgear/authgear-server

In correct error message when Passwordless SMS is enabled but to disable it

Open
#1,440 0 comments 0 reactions 0 assignees View on GitHub
bug/minor
Dominant language
Go
Stars
2k
Forks
125
Avg merge
2d 17h
Merged PRs (30d)
32

Description

`Can't save the config. At least one authenticator for type email is needed.`

Step to reproduct, at staging server:
1. Setup: Enabled Passwordless for SMS, Phone No
2. Action: Disable Phone No, Enable Email as Login ID

Show the above error message, which is not relevant; The correct course of action is to disable Passwordless for SMS, and step 2 is okay now.

![authgear-save-err](https://user-images.githubusercontent.com/387241/128656658-de4d7c96-cfd1-415d-b89f-21f9fd3509e7.jpg)

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the staging configuration flow: enable Passwordless for SMS and phone number, then disable phone number and enable email as the login ID. Trace the configuration save validation that produces the email-authenticator error; done means this combination no longer shows the irrelevant message and the reported Passwordless setting guidance is correct.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
authentication
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.