X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/d07277efba0c0069705bbefa68f2624a9c6c9ff0..7901811752faea37a9fb0f5f05405a5cfbd63b7e:/app/controllers/application.rb?ds=sidebyside diff --git a/app/controllers/application.rb b/app/controllers/application.rb index a4f65cd54..792792f83 100644 --- a/app/controllers/application.rb +++ b/app/controllers/application.rb @@ -6,18 +6,22 @@ class ApplicationController < ActionController::Base @user = User.find_by_token(session[:token]) end - def authorize(realm='Web Password', errormessage="Could't authenticate you") - username, passwd = get_auth_data # parse from headers + def require_user + redirect_to :controller => 'user', :action => 'login', :referer => request.request_uri unless @user + end + + 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 @@ -34,17 +38,18 @@ class ApplicationController < ActionController::Base end end - def get_xml_doc - doc = XML::Document.new - doc.encoding = 'UTF-8' - root = XML::Node.new 'osm' - root['version'] = API_VERSION - root['generator'] = 'OpenStreetMap server' - doc.root = root - return doc + # 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 => :bad_request + # 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 + # 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 @@ -54,8 +59,8 @@ class ApplicationController < ActionController::Base end # only basic authentication supported if authdata and authdata[0] == 'Basic' - user, pass = Base64.decode64(authdata[1]).split(':')[0..1] - end + user, pass = Base64.decode64(authdata[1]).split(':',2) + end return [user, pass] end