Add before_process and transform hooks

[jondeandres]

No description provided.

[brianr]

@jondeandres It seems that the :scope is empty/missing in both the before_process and transform hooks. Repro case:

handler1 = proc do |options|
  puts "+++ in handler1"
  puts options
  puts options[:scope][:custom][:some_key]
  puts "+++"
end

handler2 = proc do |options|
  puts "--- in handler2"
  puts options
  puts "---"
end

Rollbar.configure do |config|
  config.access_token = 'token'
  config.before_process << handler1
  config.transform << handler2
end

Rollbar.scope(:custom => {:some_key => "some value"}).debug("a debug message")

When I do the above, the output is:

+++ in handler1
{:level=>"debug", :scope=>{}, :exception=>nil, :message=>"a debug message", :extra=>nil}
E, [2016-02-10T13:45:17.817247 #20761] ERROR -- : [Rollbar] Error calling the `before_process` hook: undefined method `[]' for nil:NilClass
--- in handler2
{:level=>"debug", :scope=>{}, :exception=>nil, :message=>"a debug message", :extra=>nil, :payload=>{"access_token"=>"token", "data"=>{:timestamp=>1455140717, :environment=>"unspecified", :level=>"debug", :language=>"ruby", :framework=>"Plain", :server=>{:host=>"Brians-MacBook-Pro-2.local", :pid=>20761}, :notifier=>{:name=>"rollbar-gem", :version=>"2.7.1"}, :body=>{:message=>{:body=>"a debug message"}}, :uuid=>"f74b9e4f-a144-4b3d-9b6c-6270efd1b0f5", :custom=>{:some_key=>"some value"}}}}
---
I, [2016-02-10T13:45:17.817682 #20761]  INFO -- : [Rollbar] Scheduling payload
I, [2016-02-10T13:45:17.817715 #20761]  INFO -- : [Rollbar] Sending payload
I, [2016-02-10T13:45:18.580879 #20761]  INFO -- : [Rollbar] Success
I, [2016-02-10T13:45:18.580976 #20761]  INFO -- : [Rollbar] Details: https://rollbar.com/instance/uuid?uuid=f74b9e4f-a144-4b3d-9b6c-6270efd1b0f5 (only available if report was successful)

[jondeandres]

@brianr the interface to access the scope is options[:scope].custom in that case. But using [] shouldn't give that behavior. I'll review this.

[jondeandres]

Forget it, it's a bug yes. I'm passing there configuration.payload_options instead of the scope_object. I'll fix it.

[jondeandres]

@brianr I've fixed and changed the interface in 3428eb1.

I think it'd be better if the users can access the scope as you wanted, scope[:request] instead of scope.request, since the scope is defined like Rollbar.scoped(request: some_data).