X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/668f8020ccbf544de89822415e342c1c60826dfe..83beee6d6a49c7403ac21e0603b923905f56d527:/app/controllers/application.rb

diff --git a/app/controllers/application.rb b/app/controllers/application.rb
index 9d4749a0d..fd6ea7866 100644
--- a/app/controllers/application.rb
+++ b/app/controllers/application.rb
@@ -10,18 +10,18 @@ class ApplicationController < ActionController::Base
     redirect_to :controller => 'user', :action => 'login' unless @user
   end
 
-  def authorize(realm='Web Password', errormessage="Could't authenticate you") 
-    username, passwd = get_auth_data # parse from headers
+  def authorize(realm='Web Password', errormessage="Couldn't authenticate you") 
+    username, passwd = get_auth_data # parse from headers
     # authenticate per-scheme
-    if username.nil?
-      @user = nil # no authentication provided - perhaps first connect (client should retry after 401)
-    elsif username == 'token' 
-      @user = User.authenticate_token(passwd) # preferred - random token for user from db, passed in basic auth
-    else
-      @user = User.authenticate(username, passwd) # basic auth
-    end
-    
-    # handle authenticate pass/fail
+    if username.nil?
+      @user = nil # no authentication provided - perhaps first connect (client should retry after 401)
+    elsif username == 'token' 
+      @user = User.authenticate_token(passwd) # preferred - random token for user from db, passed in basic auth
+    else
+      @user = User.authenticate(username, passwd) # basic auth
+    end
+    
+    # handle authenticate pass/fail
     if @user
       # user exists and password is correct ... horray! 
       if @user.methods.include? 'lastlogin'         # note last login 
@@ -38,7 +38,18 @@ class ApplicationController < ActionController::Base
     end 
   end 
 
-  # extract authorisation credentials from headers, returns user = nil if none
+  # Report and error to the user
+  # (If anyone ever fixes Rails so it can set a http status "reason phrase",
+  #  rather than only a status code and having the web engine make up a 
+  #  phrase from that, we can also put the error message into the status
+  #  message. For now, rails won't let us)
+  def report_error(message)
+    render :nothing => true, :status => 400
+    # Todo: some sort of escaping of problem characters in the message
+    response.headers['Error'] = message
+  end
+
+  # extract authorisation credentials from headers, returns user = nil if none
   private 
   def get_auth_data 
     if request.env.has_key? 'X-HTTP_AUTHORIZATION'          # where mod_rewrite might have put it 
@@ -49,7 +60,7 @@ class ApplicationController < ActionController::Base
     # only basic authentication supported
     if authdata and authdata[0] == 'Basic' 
       user, pass = Base64.decode64(authdata[1]).split(':')[0..1] 
-    end 
+    end 
     return [user, pass] 
   end