3.0.5 breaks Airbrake.notify(hash)

[msepcot]

3.0.5 breaks Airbrake.notify(hash)

[der-flo]

Jep, I can confirm this.

[seanmcoleman]

This is also happening to me.

[iurimatias]

same here, please see #19

[ekampf]

same here

[iurimatias]

begin
  raise "theexception"
rescue Exception => e
  Airbrake.notify(
    :error_class   => e.class.name,
    :error_message => e.try(:message),
    :parameters    => params,
    :backtrace     => e.try(:backtrace),
    :session       => session
  )
end

results in NoMethodError (undefined method `backtrace' for #Hash:0x0000010ba8c0a8)

[jcode]

confirmed for me using Rails 2.3.14 and Ruby 1.8.7

[humandoing]

This is a bug that I found as well. After looking at the code, it looks like the Airbrake.notify method now expects two parameters, the first being an Error object and the 2nd being a Hash. I haven't tested this extensively, but here is what should be an easy workaround (passing a new RuntimeError instance as the first parameter in the call to Airbrake.notify)

Airbrake.notify( RuntimeError.new,
:error_class => e.class.name,
:error_message => e.try(:message),
:parameters => params,
:backtrace => e.try(:backtrace),
:session => session
)

[msepcot]

It's not necessary to pass a RuntimeError to Airbrake.notify, simply passing nil as the first parameter is an easy workaround, but there is still a mismatch between what is expected from the documentation and what is implemented.

[humandoing]

Yup, fair enough. I just figured since it seems to be expecting an Error object, might as well give it one.

[stuartchaney]

Guys thanks for raising this. Fix on its way.

[charleseff]

+1, waiting for the fix

[apolzon]

+1, looking forward to fix, thanks for tracking this guys!