X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/c8f9387420233bd7e845cb205c3b68ede1f90024..95d9ae83abe22879e4630674e5355c9751cf9974:/app/controllers/application.rb diff --git a/app/controllers/application.rb b/app/controllers/application.rb index 82ef80920..792792f83 100644 --- a/app/controllers/application.rb +++ b/app/controllers/application.rb @@ -2,46 +2,64 @@ # Likewise, all the methods added will be available for all controllers. class ApplicationController < ActionController::Base + def authorize_web + @user = User.find_by_token(session[:token]) + end + def require_user + redirect_to :controller => 'user', :action => 'login', :referer => request.request_uri unless @user + end - # HTTP AUTH stuff for the API - - def authorize(realm='Web Password', errormessage="Could't authenticate you") - username, passwd = get_auth_data - # check if authorized - # try to get user - if @user = User.authenticate(username, passwd) + 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 @user # user exists and password is correct ... horray! - if @user.methods.include? 'lastlogin' - # note last login + if @user.methods.include? 'lastlogin' # note last login @session['lastlogin'] = user.lastlogin @user.last.login = Time.now @user.save() @session["User.id"] = @user.id end else - # the user does not exist or the password was wrong - @response.headers["Status"] = "Unauthorized" - @response.headers["WWW-Authenticate"] = "Basic realm=\"#{realm}\"" - render_text(errormessage, 401) + # no auth, the user does not exist or the password was wrong + response.headers["Status"] = "Unauthorized" + response.headers["WWW-Authenticate"] = "Basic realm=\"#{realm}\"" + render_text(errormessage, 401) # :unauthorized end end + # 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 private def get_auth_data - user, pass = '', '' - # extract authorisation credentials - if request.env.has_key? 'X-HTTP_AUTHORIZATION' - # try to get it where mod_rewrite might have put it - authdata = @request.env['X-HTTP_AUTHORIZATION'].to_s.split - elsif request.env.has_key? 'HTTP_AUTHORIZATION' - # this is the regular location - authdata = @request.env['HTTP_AUTHORIZATION'].to_s.split + if request.env.has_key? 'X-HTTP_AUTHORIZATION' # where mod_rewrite might have put it + authdata = request.env['X-HTTP_AUTHORIZATION'].to_s.split + elsif request.env.has_key? 'HTTP_AUTHORIZATION' # regular location + authdata = request.env['HTTP_AUTHORIZATION'].to_s.split end - - # at the moment we only support basic authentication + # only basic authentication supported if authdata and authdata[0] == 'Basic' - user, pass = Base64.decode64(authdata[1]).split(':')[0..1] + user, pass = Base64.decode64(authdata[1]).split(':',2) end return [user, pass] end