@@ -9,7 +9,7 @@ Request information: <% if @user_name && @user_email -%> * User: <%= @user_name %> <<%= @user_email %>> <% end -%> - * URL: <%= request.protocol %><%= @host %><%= request.request_uri %> + * URL: <%= request.protocol %><%= request.host_with_port %><%= request.request_uri %> * Parameters: <%= request.parameters.inspect %> * Rails root: <%= @rails_root %> * Hostname: <%= @exception_hostname %> app/views/exception_handler/exception_template.html.erb @@ -34,6 +34,13 @@ class ReportsExceptionsTest < Test::Unit::TestCase assert ActionController::Base.respond_to?(:saves_exceptions), 'ActionController::Base should respond to :saves_exceptions' end + def test_saves_exceptions_with_URL + TestController.saves_exceptions(:hostname=>'foo', :exceptions_dir=>EXCEPTIONS_DIR) + get :do_raise, :ex=>'Exception', :msg=>'test exception' + lines = read_file(Exception).join + assert_match 'URL: http://test.host/raise?ex=Exception&msg=test+exception', lines, "URL should be emitted" + end + def test_saves_exceptions_with_hostname TestController.saves_exceptions(:hostname=>'foo', :exceptions_dir=>EXCEPTIONS_DIR) get :do_raise, :ex=>'Exception', :msg=>'test exception' @@ -96,7 +103,6 @@ class ReportsExceptionsTest < Test::Unit::TestCase end - private def read_file(name) test/reports_exceptions_test.rb fbdf83dc5bafeddd307e9ee5cd3c483cc8fd7da3 Scott Steadman ss@stdmn.com http://github.com/ss/exception_reporter/commit/1c7a2d253dc8e35cf9a73bb160d1ea751a061f1a 1c7a2d253dc8e35cf9a73bb160d1ea751a061f1a 2008-09-08T13:51:00-07:00 2008-09-08T13:51:00-07:00 Fixed bug in template. Template references request.host_with_port rather than @host. b28f11b37e8504af8d97547e364250091df74d61 Scott Steadman ss@stdmn.com