split airbrake javascript loader into script and configuration #108

Closed
wants to merge 2 commits into
from

2 participants

@TMaYaD

This allows the user to override them individually

@shime shime commented on the diff May 21, 2013
lib/airbrake/rails/controller_methods.rb
@@ -9,7 +9,7 @@ def airbrake_request_data
:controller => params[:controller],
:action => params[:action],
:url => airbrake_request_url,
- :cgi_data => airbrake_filter_if_filtering(request.env),
+ :cgi_data => ENV.to_hash.merge(airbrake_filter_if_filtering(request.env)),
@shime
shime added a line comment May 21, 2013

Could you please revert this change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@shime

Merged in c7cf4e1. Had to modify it slightly to make it work.

Thanks!

@shime shime closed this May 21, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment