zitadel icon indicating copy to clipboard operation
zitadel copied to clipboard

[Bug]: Second factor SMS OTP can't be added on the org level

Open eliobischof opened this issue 1 year ago • 3 comments

Preflight Checklist

  • [X] I could not find a solution in the documentation, the existing issues or discussions
  • [X] I have joined the ZITADEL chat

Environment

Self-hosted

Version

latest

Database

None

Database Version

Postgres latest

Describe the problem caused by this bug

SMS OTP is unusable on org level.

To reproduce

  1. Spin up a fresh environment from scratch
  2. add SMS OTP via console to the org policy
  3. Repeat, as it won't be listed
  4. Note the already exists error

Screenshots

No response

Expected behavior

No response

Operating System

No response

Relevant Configuration

No response

Additional Context

No response

eliobischof avatar Nov 29 '23 10:11 eliobischof

@eliobischof as you already reproduced it, was it a frontend or backend issue?

hifabienne avatar Nov 29 '23 12:11 hifabienne

@eliobischof as you already reproduced it, was it a frontend or backend issue?

Backend I think.

Sth is probably wrong with the login policy projection

eliobischof avatar Nov 29 '23 12:11 eliobischof

It seems that the issue also happens trying to add a Multifactor SMTP OPT for an org

image

Maybe you can assign this to me @hifabienne

doncicuto avatar Feb 12 '24 18:02 doncicuto

@doncicuto thanks for offering your help with this! I assigned you.

eliobischof avatar Feb 27 '24 11:02 eliobischof

Hi @eliobischof and @hifabienne, I've tried to reproduce the issue again before working on the issue, but it seems that with the latest main code this is not happening anymore. Here's a video with a fresh backend compile and a new org:

https://github.com/zitadel/zitadel/assets/30386061/38ae7fb2-db4b-4409-adf1-1e1325f3afd1

If you can reproduce it again ping me :wink: I'm moving to another issue for now

Cheers!

doncicuto avatar Mar 02 '24 07:03 doncicuto

Couldn't reproduce either. Will close the issue for now, if it reappears we can reopen. @doncicuto Thanks for looking into it.

hifabienne avatar Mar 04 '24 08:03 hifabienne