russ / bugsnag Goto Github PK

In this commit release_stage was changed to Bool, but the Bugsnag API actually wants a String here. This should be the name of the release stage (i.e. "staging", "production", "development", etc...)

27c6ba8

The event.app is nilable

For now, you can either not_nil! or cast .as(Bugsnag::App)... but we can probably just make this required and have Event build it or something.

For Crystal 0.35.1, this needs to be

request.remote_address.as(Socket::IPAddress).address || "127.0.0.1"

We should use the Crystal built-in Log instead of just puts. Then you can tone down/up the log as you'd need

The Bugsnag.report method requires an HTTP::Server::Context to be passed in which pulls data for the Bugsnag::Request, but if you wanted to use this in say Mosquito, there's no context.

Maybe in this case, there could be an overload that doesn't take context, and it builds a fake one?