fix: bug in registration.ErrorHandler

ory · Mar 31, 2022 · b37b16d · b37b16d
1 parent dbf11c4
commit b37b16d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/selfservice/flow/registration/error.go b/selfservice/flow/registration/error.go
@@ -98,8 +98,8 @@ func (s *ErrorHandler) WriteFlowError(
 		return
 	}
 
-	ds, err := s.d.Config(r.Context()).DefaultIdentityTraitsSchemaURL()
-	if err != nil {
+	ds, e := s.d.Config(r.Context()).DefaultIdentityTraitsSchemaURL()
+	if e != nil {
 		s.forward(w, r, f, err)
 		return
 	}

diff --git a/selfservice/strategy/code/registration_test.go b/selfservice/strategy/code/registration_test.go
@@ -60,7 +60,7 @@ func TestRegistration(t *testing.T) {
 				v.Set("traits.phone", identifier)
 			}
 			testhelpers.SubmitRegistrationFormWithFlow(t, true, hc, values,
-				false, http.StatusBadRequest, publicTS.URL+registration.RouteSubmitFlow, f)
+				false, http.StatusOK, publicTS.URL+registration.RouteSubmitFlow, f)
 
 			messages, err := reg.CourierPersister().NextMessages(context.Background(), 10)
 			assert.NoError(t, err, "Courier queue should not be empty.")
@@ -107,7 +107,7 @@ func TestRegistration(t *testing.T) {
 				v.Set("traits.phone", identifier)
 			}
 			body := testhelpers.SubmitRegistrationFormWithFlow(t, isAPI, hc, values,
-				isSPA, http.StatusBadRequest, expectReturnTo, f)
+				isSPA, http.StatusOK, expectReturnTo, f)
 
 			messages, err := reg.CourierPersister().NextMessages(context.Background(), 10)
 			assert.NoError(t, err, "Courier queue should not be empty.")