Don't auto-verify mobile emails anymore

gallery-so · May 2, 2024 · 6a86766 · 6a86766
1 parent a02cbf5
commit 6a86766
Showing 1 changed file with 4 additions and 11 deletions.
diff --git a/publicapi/user.go b/publicapi/user.go
@@ -597,17 +597,10 @@ func (api UserAPI) CreateUser(ctx context.Context, authenticator auth.Authentica
 		return "", "", err
 	}
 
-	// Temporary fix for the mobile app
-	if gc.GetHeader("X-Platform") == "Mobile" {
-		if email != nil {
-			createUserParams.EmailStatus = persist.EmailVerificationStatusVerified
-		}
-	} else {
-		if createUserParams.EmailStatus == persist.EmailVerificationStatusUnverified && email != nil {
-			if err := emails.RequestVerificationEmail(ctx, userID); err != nil {
-				// Just the log the error since the user can verify their email later
-				logger.For(ctx).Warnf("failed to send verification email: %s", err)
-			}
+	if createUserParams.EmailStatus == persist.EmailVerificationStatusUnverified && email != nil {
+		if err := emails.RequestVerificationEmail(ctx, userID); err != nil {
+			// Just the log the error since the user can verify their email later
+			logger.For(ctx).Warnf("failed to send verification email: %s", err)
 		}
 	}