phptek / silverstripe-sentry

Flexible Sentry compatible bug aggregation client for Silverstripe applications.
BSD 3-Clause "New" or "Revised" License
12 stars 22 forks source link

Userdata and exception context #3

Closed lekoala closed 7 years ago

phptek commented 7 years ago

Thanks for your contributions! Can you take a look at the Travis build failures (Failing unit-test) and either fixup and push some changes, or alter the test as necessary?

I haven't had time to fully look at the change, but the increased "scope" of the backtrace is something I had noticed myself - so thank you for that :-)

lekoala commented 7 years ago

well, it's the test that checks for user data. I've moved the code to collect data upon sending, not on init, therefore the test fails.

i can update the test, or set some default data if available, but I believe setting user data upon init is not the best idea because it's simply not available if used in _config.php