Was accidentally turned into an always-true boolean in
5f343617,
due to confusion with JavaScript's default '||' operator.
Change-Id: I24071e22f8bb7a296ae138303f63acdb8ea4950a
// XXX Is it ok to put untrusted data into log??
'csp-report' => $report,
'method' => __METHOD__,
- 'user_id' => $this->getUser()->getId() || 'logged-out',
+ 'user_id' => $this->getUser()->getId() ?: 'logged-out',
'user-agent' => $userAgent,
'source' => $this->getParameter( 'source' ),
] );
'<https://blocked.test> blocked from being loaded on <https://doc.test/path>:4',
[
'method' => 'ApiCSPReport::execute',
- // FIXME
- 'user_id' => true,
+ 'user_id' => 'logged-out',
'user-agent' => 'Test/0.0',
'source' => 'internal'
]