illacloud / deploy-illa-manually

Deploy illa utils manually. Docker, docker compose and k8s are all avaliable in this repo.
Apache License 2.0
37 stars 20 forks source link

Fix invalid email syntax insertion of root user #6

Closed Andrew-Chen-Wang closed 1 year ago

Andrew-Chen-Wang commented 1 year ago

The sign in page only displays email, but not username. The email field in the record gives an invalid email that the frontend will not validate due to an incorrect syntax.

karminski commented 1 year ago

@Andrew-Chen-Wang Hi, the illa-builder (frontend) treats the root as the special user. it will bypass the email format validator. The "root" as an email login is legal.

Andrew-Chen-Wang commented 1 year ago

Can you reproduce this on your end?

On my end:

  1. docker-compose -f docker-compose-with-official-images.yml up --build
  2. Open localhost:3000
  3. Try entering root for email and password for password. Unable to submit due to a validation error on email field stating the email is an invalid syntax
karminski commented 1 year ago

@Andrew-Chen-Wang The default user is just for the SELF_HOST mode (use all-in-one docker image). They are:

The docker-compose and k8s are CLOUD mode. They do not have a default user for safety reasons.

So please try deploying as SELF_HOST mode, or signup a new user in CLOUD mode.