Skip to content

fix(config): read ALLOW_REGISTRATION_EMAIL for the registration whitelist (#2731) - #2735

Open
PiedPiper911 wants to merge 1 commit into
coze-dev:mainfrom
PiedPiper911:fix/config-registration-whitelist-2731
Open

PiedPiper911 wants to merge 1 commit into
coze-dev:mainfrom
PiedPiper911:fix/config-registration-whitelist-2731

Conversation

@PiedPiper911

Copy link
Copy Markdown

@C:/Users/PiedPiper888/WorkBuddy/2026-08-03-14-30-56/pr_body_coze2731.md

…list (coze-dev#2731)

getBasicConfigurationFromOldConfig() populated AllowRegistrationEmail from the wrong
env var (consts.DisableUserRegistration), so with DISABLE_USER_REGISTRATION=true the
whitelist became ["true"] and every registration (including whitelisted emails) was
rejected. Read consts.AllowRegistrationEmail instead, matching AdminAuthMW.

Signed-off-by: PiedPiper911 <PiedPiper911@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant