@@ -55,16 +55,16 @@ module OpenidEnabled identifier = params[:openid_url] if identifier.nil? flash[:error] = "OpenID URL not given" - redirect_to failed_login_redirect + redirect_to ( params[:failed_login_redirects_to] || failed_login_redirect ) return end oidreq = consumer.begin(identifier) rescue OpenID::OpenIDError => e flash[:error] = "Discovery failed for #{identifier}: #{e}" - redirect_to failed_login_redirect + redirect_to ( params[:failed_login_redirects_to] || failed_login_redirect ) return end - return_to = url_for :action => :complete_login + return_to = url_for :action => :complete_login, :login_redirect => ( params[:login_redirects_to] || login_redirect ), :failed_login_redirect => ( params[:failed_login_redirects_to] || failed_login_redirect ) realm = url_for :action => :index if oidreq.send_redirect?(realm, return_to, params[:immediate]) @@ -86,17 +86,21 @@ module OpenidEnabled else flash[:error] = "Verification failed: #{oidresp.message}" end + redirect_to params[:failed_login_redirect] when OpenID::Consumer::SUCCESS flash[:notice] = ("Verification of #{oidresp.display_identifier}"\ " succeeded.") session[openid_session_sym] = oidresp.display_identifier + redirect_to params[:login_redirect] when OpenID::Consumer::SETUP_NEEDED flash[:notice] = "Immediate request failed - Setup Needed" + redirect_to params[:failed_login_redirect] when OpenID::Consumer::CANCEL flash[:notice] = "OpenID transaction cancelled." + redirect_to params[:failed_login_redirect] else + redirect_to params[:login_redirect] end - redirect_to login_redirect end def consumer lib/openid_enabled.rb 74d23ab34fe96aa11f901385f446b835ac1d1bdf madnificent madnificent@gmail.com http://github.com/madnificent/openid_enabled/commit/c8fa4fce99f1c25519ae0182b0558028b23e5aa7 c8fa4fce99f1c25519ae0182b0558028b23e5aa7 2008-08-11T14:22:48-07:00 2008-08-11T14:22:48-07:00 Enabled a way to specify where a failed and a successfull login should redirect to on a per-commit basis. This works by adding hidden fields to the login-form. There are two meaningfull fields: login_redirects_to and failed_login_redirects_to. The names are self-explanatory and an example login-form looks like <% form_tag start_login_users_path, :method => :get do -%> <%= text_field_tag :openid_url %> <%= hidden_field_tag 'login_redirects_to', cities_path %> <%= hidden_field_tag 'failed_login_redirects_to', users_path %> <%= submit_tag 'OpenIDfy me!' %> <% end -%> da18b1a8f836a433befdb6fe20ef4bb4bc5c9f6d madnificent madnificent@gmail.com