kenpratt / erlbrake

Erlang Airbrake notification client
http://github.com/kenpratt/erlbrake
MIT License
32 stars 21 forks source link

Log more details when notifying hoptoad #1

Closed zeisss closed 12 years ago

zeisss commented 14 years ago

The Hoptoad API v2 support more options that we currently support. See [1] for details.

We need to find a way to easily log these as well.

[1] http://help.hoptoadapp.com/faqs/api-2/notifier-api-v2

zeisss commented 14 years ago

Current interface:

notify(Type, Reason, Message, Module, Line)
notify(Type, Reason, Message, Module, Line, Stacktrace)

Maybe we can add a function like this for full support:

%%%
% Type = atom(), gets dropped
% Reason = atom() | list(), the error class (api), not seen in the UI, but used for grouping errors in hoptoad
% Message = string()
% Stacktrace = [Element]
% Element = {file, line} | {file, method, line}
% ProjectRoot = string()
% Request = {request, Url, Vars} |
%                 {request, Url, Component, Vars} |
%                 {request, Url, Component, Action, Vars}
% Vars = [{VarKey, VarValue}]
%%
notify(Type, Reason, Message, Module, Line, Stacktrace, ProjectRoot)
notify(Type, Reason, Message, Module, Line, Stacktrace, ProjectRoot, Request)
zeisss commented 14 years ago

I added this on my fork. As far as I can see, the only missing elements from the API is the session elements inside the request. WIP ;)

kenpratt commented 12 years ago

Closing this ticket, as I merged in your changes earlier this year (and forgot to close this).

You might want to pull from me, as I've updated the code to point to the new URL for Airbrake.

Thanks!

-Ken