Setting cookies when headers sent

[adamthehutt]

Recently Raygun started tracking users by setting unique id cookies. The problem is that sometimes an error occurs after output has already started (e.g., if the error is triggered in a template). This results in another error being triggered, since you can't set cookies after output has been sent to the browser.

A pretty easy fix would be to add a conditional:

if(!headers_sent()) 

in RaygunClient.php before calling setcookie().

[fundead]

Thanks for spotting this, I've managed to repro that and the fix you suggested does indeed work. You'll find the updated master branch with the change.

[CmdrKeen]

As an aside - if you're not setting the user until after you've started streaming the content then you will lose the benefits of session tracking (which, for now, are not obvious since we're just building out support for this in the Raygun app).

While we have the guard I'd suggest setting the user earlier if you want to take advantage of what this feature will evolve to 👍