Normalize account login parameter to lowercase in accounts controller

ecosyste-ms · Dec 7, 2024 · e9a3eed · e9a3eed
1 parent 7a356e8
commit e9a3eed
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/app/controllers/accounts_controller.rb b/app/controllers/accounts_controller.rb
@@ -5,7 +5,7 @@ def index
   end
 
   def show
-    @account = Account.find_by_login(params[:id])
+    @account = Account.find_by_login(params[:id].downcase)
   end
 
   def sponsors

diff --git a/app/controllers/api/v1/accounts_controller.rb b/app/controllers/api/v1/accounts_controller.rb
@@ -5,7 +5,7 @@ def index
   end
 
   def show
-    @account = Account.find_by_login(params[:id])
+    @account = Account.find_by_login(params[:id].downcase)
     raise ActiveRecord::RecordNotFound unless @account
   end
 
@@ -16,15 +16,15 @@ def sponsors
   end
 
   def sponsorships
-    @account = Account.find_by_login(params[:account_id])
+    @account = Account.find_by_login(params[:account_id].downcase)
     raise ActiveRecord::RecordNotFound unless @account
     scope = @account.sponsorships_as_maintainer.order('created_at DESC').includes(:maintainer, :funder)
     @pagy, @sponsorships = pagy(scope)
     render 'api/v1/sponsorships/index'
   end
 
   def account_sponsors
-    @account = Account.find_by_login(params[:account_id])
+    @account = Account.find_by_login(params[:account_id].downcase)
     raise ActiveRecord::RecordNotFound unless @account
     scope = @account.sponsorships_as_funder.order('created_at DESC').includes(:maintainer, :funder)
     @pagy, @sponsorships = pagy(scope)