On Jun 4, 6:32 pm, "Michael Koziarski" <[EMAIL PROTECTED]> wrote:
> If it's just about making the method protected you should be able to change:
> @controller.filter_parameters(parameters)
> to
> @controller.send(:filter_parameters, parameters)

Thanks for your reply, Koz. This strategy you suggested appears to fix
the issue of the plugin not sending emails when exceptions are raised
in an app using Rails 2.1:

http://github.com/trevorturk/exception_notification/commit/9b3b0eca97ca27af6844ffc81ee6190ce022b364

Perhaps someone with commit access to the Rails github would consider
making this change in the official plugin?

Here is a ticket about the issue of the plugin not being compatible
with Rails 2.1:
http://rails.lighthouseapp.com/projects/8995/tickets/15-rails-2-1-0-filter_parameter_logging-crashes-exception_notifiable

Here is a ticket about the (separate?) issue of the (still) failing
test:
http://rails.lighthouseapp.com/projects/8995/tickets/11-exception_notifiable-test-failure

===
Just to be clear for others that may be impacted by this issue: the
exception_notification plugin does not appear to work with Rails 2.1
applications!
===

Thanks again,
- Trevor
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Core" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to