From 0b0c8587ae611777a64ffdd23043a75afca2882e Mon Sep 17 00:00:00 2001 From: songtianlun Date: Sat, 18 Jan 2025 14:04:12 +0800 Subject: [PATCH] fix: handle user creation failure - Add @user.destroy to ensure proper cleanup on failure This change prevents incomplete user records from being left in the database when user creation fails due to an error. --- app/controllers/users_controller.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb index 55c6243..32b8f5a 100644 --- a/app/controllers/users_controller.rb +++ b/app/controllers/users_controller.rb @@ -35,6 +35,7 @@ class UsersController < ApplicationController rescue => e logger.error "User creation failed: #{e.message}" flash[:danger] = "Something went wrong. Please try again." + @user.destroy render "new", status: :unprocessable_entity end else