lib/exception_loggable_controller_mixin.rb @@ -23,46 +23,35 @@ Now add a new route to your routes.rb: After that, visit /logged_exceptions in your application to manage the exceptions. -It's understandable that you may want to require authentication. Add this to your config/environments/production.rb: - - # config/environments/production.rb - config.after_initialize do - require 'application' unless Object.const_defined?(:ApplicationController) - LoggedExceptionsController.class_eval do - # set the same session key as the app - session :session_key => '_beast_session_id' - - # include any custom auth modules you need - include AuthenticationSystem - - before_filter :login_required +It's understandable that you may want to require authentication. You can do this by creating your own controller extending your ApplicationController (thereby leveraging all the existing authorization rules you may have) and include the ExceptionLoggableControllerMixin module. For example: + + class LoggedExceptionsController < ApplicationController + include ExceptionLoggableControllerMixin + self.application_name = "My Application Name" # this will show up in the title of the Rss feed + + # add your regular permission checking here, e.g.: + protected + # only allow admins + # this obviously depends on how your auth system works + def authorized? + current_user.is_a?(Admin) + end - # optional, sets the application name for the rss feeds - self.application_name = "Beast" - - protected - # only allow admins - # this obviously depends on how your auth system works - def authorized? - current_user.is_a?(Admin) - end - - # assume app's login required doesn't use http basic - def login_required_with_basic - respond_to do |accepts| - # alias_method_chain will alias the app's login_required to login_required_without_basic - accepts.html { login_required_without_basic } - - # access_denied_with_basic_auth is defined in LoggedExceptionsController - # get_auth_data returns back the user/password pair - accepts.rss do - access_denied_with_basic_auth unless self.current_user = User.authenticate(*get_auth_data) - end + # assume app's login required doesn't use http basic + def login_required_with_basic + respond_to do |accepts| + # alias_method_chain will alias the app's login_required to login_required_without_basic + accepts.html { login_required_without_basic } + + # access_denied_with_basic_auth is defined in ExceptionLoggableControllerMixin + # get_auth_data returns back the user/password pair + accepts.rss do + access_denied_with_basic_auth unless self.current_user = User.authenticate(*get_auth_data) end end - - alias_method_chain :login_required, :basic - end + end + + alias_method_chain :login_required, :basic end The exact code of course depends on the specific needs of your application. @@ -71,4 +60,4 @@ CREDITS Jamis Buck - original exception_notification plugin Rick Olson - model/controller code -Josh Goebel - design +Josh Goebel - design \ No newline at end of file README @@ -1,3 +1,3 @@ require 'will_paginate' unless Kernel.const_defined? 'WillPaginate' WillPaginate.enable -LoggedExceptionsController.view_paths = [File.join(directory, 'views')] +ExceptionLoggableControllerMixin.custom_view_paths = [File.join(directory, 'views')] init.rb @@ -1,71 +1,3 @@ class LoggedExceptionsController < ActionController::Base - cattr_accessor :application_name - layout nil - - def index - @exception_names = LoggedException.find_exception_class_names - @controller_actions = LoggedException.find_exception_controllers_and_actions - query - end - - def query - conditions = [] - parameters = [] - unless params[:id].blank? - conditions << 'id = ?' - parameters << params[:id] - end - unless params[:query].blank? - conditions << 'message LIKE ?' - parameters << "%#{params[:query]}%" - end - unless params[:date_ranges_filter].blank? - conditions << 'created_at >= ?' - parameters << params[:date_ranges_filter].to_f.days.ago.utc - end - unless params[:exception_names_filter].blank? - conditions << 'exception_class = ?' - parameters << params[:exception_names_filter] - end - unless params[:controller_actions_filter].blank? - conditions << 'controller_name = ? AND action_name = ?' - parameters += params[:controller_actions_filter].split('/').collect(&:downcase) - end - @exceptions = LoggedException.paginate :order => 'created_at desc', :per_page => 30, - :conditions => conditions.empty? ? nil : parameters.unshift(conditions * ' and '), :page => params[:page] - - respond_to do |format| - format.html { redirect_to :action => 'index' unless action_name == 'index' } - format.js { render :action => 'query.rjs' } - format.rss { render :action => 'query.rxml' } - end - end - - def show - @exc = LoggedException.find params[:id] - end - - def destroy - LoggedException.destroy params[:id] - end - - def destroy_all - LoggedException.delete_all ['id in (?)', params[:ids]] unless params[:ids].blank? - query - end - - private - def access_denied_with_basic_auth - headers["Status"] = "Unauthorized" - headers["WWW-Authenticate"] = %(Basic realm="Web Password") - render :text => "Could't authenticate you", :status => '401 Unauthorized' - end - - @@http_auth_headers = %w(X-HTTP_AUTHORIZATION HTTP_AUTHORIZATION Authorization) - # gets BASIC auth info - def get_auth_data - auth_key = @@http_auth_headers.detect { |h| request.env.has_key?(h) } - auth_data = request.env[auth_key].to_s.split unless auth_key.blank? - return auth_data && auth_data[0] == 'Basic' ? Base64.decode64(auth_data[1]).split(':')[0..1] : [nil, nil] - end -end + include ExceptionLoggableControllerMixin +end \ No newline at end of file lib/logged_exceptions_controller.rb c25f64a95fb20c705bf9a0ddfb6a1f22016bafce Lawrence Pit lawrence.pit@gmail.com http://github.com/lawrencepit/exception_logger/commit/4828f97a6385637675381a324c07f221074c81ce 4828f97a6385637675381a324c07f221074c81ce 2008-05-06T02:08:31-07:00 2008-05-06T02:08:31-07:00 Ability to create your own LoggedExceptionsController, thereby allowing easier deployment in dev environments and better leverage of existing (authentication) code in ApplicationController, i.e. no need to replicate in config.after_initialize 6a8347f76b90508fab8ea206f4abe484fb3fa4bf Lawrence Pit lawrence.pit@gmail.com