search

jenssegers / laravel-rollbar

Rollbar error monitoring integration for Laravel projects
328 stars 98 forks source link

Send request info along with errors #14

Closed coryvirok closed 9 years ago

coryvirok commented 9 years ago

First off, thanks for building this!

It appears that the request information is not sent along with any errors/logs. I'm not very familiar with Laravel but it seems like you could just add it here: https://github.com/jenssegers/laravel-rollbar/blob/master/src/RollbarLogHandler.php#L79

It would be very helpful to send over all of the request headers.

jenssegers commented 9 years ago

The Rollbar client picks up the request headers.

mikebronner commented 9 years ago

Hi Jens,

I have been talking with Cory (from Rollbar) about this -- is there anything I need to configure to have Rollbar pick up the referrer? Using your package with Laravel 5 has not been sending referrers over to Rollbar for some reason.

I'll be happy to work on this and submit a pull request if you like, but I'm sure its something stupid on my end that I'm not seeing?

mikebronner commented 9 years ago

Update: OK, my apologies Cory and Jens. I see now where I went wrong. I never realized that the Rollbar occurrence page scrolled to the right to reveal more columns (my resolution somehow made it look like there weren't any more). Scrolling to the right I now see some occurrences have referrers, and some don't. Thanks! (Go ahead and close the issue.)

coryvirok commented 9 years ago

Cool. Sorry for the spam and I'm glad it's working. :)