From 956d0cb0424277340e675bb14fef41ae2e35b495 Mon Sep 17 00:00:00 2001 From: Ethan <39577870+ethanndickson@users.noreply.github.com> Date: Thu, 1 Aug 2024 13:30:10 +1000 Subject: [PATCH] fix: block creating oidc users when oidc has not been configured (#14064) --- coderd/users.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/coderd/users.go b/coderd/users.go index 565aeca1cb..bfa81f8662 100644 --- a/coderd/users.go +++ b/coderd/users.go @@ -464,6 +464,12 @@ func (api *API) postUser(rw http.ResponseWriter, r *http.Request) { } loginType = database.LoginTypePassword case codersdk.LoginTypeOIDC: + if api.OIDCConfig == nil { + httpapi.Write(ctx, rw, http.StatusBadRequest, codersdk.Response{ + Message: "You must configure OIDC before creating OIDC users.", + }) + return + } loginType = database.LoginTypeOIDC case codersdk.LoginTypeGithub: loginType = database.LoginTypeGithub @@ -471,6 +477,7 @@ func (api *API) postUser(rw http.ResponseWriter, r *http.Request) { httpapi.Write(ctx, rw, http.StatusBadRequest, codersdk.Response{ Message: fmt.Sprintf("Unsupported login type %q for manually creating new users.", req.UserLoginType), }) + return } user, _, err := api.CreateUser(ctx, api.Database, CreateUserRequest{