Page MenuHomePhorge
Differential D25137 Diff 554 src/applications/base/controller/PhabricatorController.php

Changeset View

Standalone View

View Options

src/applications/base/controller/PhabricatorController.php

Show First 20 Lines • Show All 68 Lines • ▼ Show 20 Lines if ($request->getUser()) {
// permitted to do this. // permitted to do this.
PhabricatorTestCase::assertExecutingUnitTests(); PhabricatorTestCase::assertExecutingUnitTests();
$user = $request->getUser(); $user = $request->getUser();
} else { } else {
$user = new PhabricatorUser(); $user = new PhabricatorUser();
$session_engine = new PhabricatorAuthSessionEngine(); $session_engine = new PhabricatorAuthSessionEngine();
$phsid = $request->getCookie(PhabricatorCookies::COOKIE_SESSION); $phsid = $request->getCookie(PhabricatorCookies::COOKIE_SESSION);
if (strlen($phsid)) { if (phutil_nonempty_string($phsid)) {
valerio.bozzolanUnsubmitted
Done
Inline Actions

✅ The input value is probably safe to be assumed as string or NULL

valerio.bozzolan: ✅ The input value is probably safe to be assumed as `string` or NULL
$session_user = $session_engine->loadUserForSession( $session_user = $session_engine->loadUserForSession(
PhabricatorAuthSession::TYPE_WEB, PhabricatorAuthSession::TYPE_WEB,
$phsid); $phsid);
if ($session_user) { if ($session_user) {
$user = $session_user; $user = $session_user;
} }
} else { } else {
// If the client doesn't have a session token, generate an anonymous // If the client doesn't have a session token, generate an anonymous
▲ Show 20 Lines • Show All 566 Lines • Show Last 20 Lines
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0