Filter cookies from events by default

[imjoehaines]

Goal

In other notifiers, the cookies header is filtered by default and we don't collect a 'Cookies' tab — this PR brings PHP inline

Cookies can still be recorded by a user by removing it as a filtered value and adding the cookie callback. The filter controls whether it shows in 'Request → Headers' and the callback adds the 'Cookies' tab

$bugsnag = \Bugsnag\Client::make();

$bugsnag->setFilters(['password']);
$bugsnag->registerCallback(
    new \Bugsnag\Callbacks\RequestCookies(
        new \Bugsnag\Request\BasicResolver()
    )
);

Changeset

• No longer register the RequestCookies callback by default
• Configuration::$filters now contains cookie, to filter the cookie header
• Added a test for the default callbacks, which also ensures cookies aren't collected as a separate tab and the cookie header is filtered

[kattrali]

😆