Move the authorize method to api_controller
authorAndy Allan <git@gravitystorm.co.uk>
Wed, 20 Mar 2019 13:36:43 +0000 (14:36 +0100)
committerAndy Allan <git@gravitystorm.co.uk>
Wed, 20 Mar 2019 13:39:17 +0000 (14:39 +0100)
app/controllers/api_controller.rb
app/controllers/application_controller.rb

index 8ddb724..cb3d71d 100644 (file)
@@ -1,6 +1,19 @@
 class ApiController < ApplicationController
   skip_before_action :verify_authenticity_token
 
+  def authorize(realm = "Web Password", errormessage = "Couldn't authenticate you")
+    # make the current_user object from any auth sources we have
+    setup_user_auth
+
+    # handle authenticate pass/fail
+    unless current_user
+      # no auth, the user does not exist or the password was wrong
+      response.headers["WWW-Authenticate"] = "Basic realm=\"#{realm}\""
+      render :plain => errormessage, :status => :unauthorized
+      return false
+    end
+  end
+
   def deny_access(_exception)
     if current_token
       set_locale
index 485c30b..c5545c0 100644 (file)
@@ -112,19 +112,6 @@ class ApplicationController < ActionController::Base
     end
   end
 
-  def authorize(realm = "Web Password", errormessage = "Couldn't authenticate you")
-    # make the current_user object from any auth sources we have
-    setup_user_auth
-
-    # handle authenticate pass/fail
-    unless current_user
-      # no auth, the user does not exist or the password was wrong
-      response.headers["WWW-Authenticate"] = "Basic realm=\"#{realm}\""
-      render :plain => errormessage, :status => :unauthorized
-      return false
-    end
-  end
-
   def check_database_readable(need_api = false)
     if Settings.status == "database_offline" || (need_api && Settings.status == "api_offline")
       if request.xhr?