@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Minor, fix access log issue with non-auth methods.
+5
-4
+5
-4
src/applications/conduit/controller/api/PhabricatorConduitAPIController.php
+5
-4
src/applications/conduit/controller/api/PhabricatorConduitAPIController.php
···
113
113
114
114
$allow_unguarded_writes = false;
115
115
$auth_error = null;
116
+
$conduit_username = '-';
116
117
if ($method_handler->shouldRequireAuthentication()) {
117
118
$metadata['scope'] = $method_handler->getRequiredScope();
118
119
$auth_error = $this->authenticateUser($api_request, $metadata);
···
123
124
if (isset($metadata['actAsUser'])) {
124
125
$this->actAsUser($api_request, $metadata['actAsUser']);
125
126
}
126
-
}
127
127
128
-
$access_log = PhabricatorAccessLog::getLog();
129
-
if ($access_log) {
130
-
$conduit_username = '-';
131
128
$conduit_user = $api_request->getUser();
132
129
if ($conduit_user && $conduit_user->getPHID()) {
133
130
$conduit_username = $conduit_user->getUsername();
134
131
}
132
+
}
133
+
134
+
$access_log = PhabricatorAccessLog::getLog();
135
+
if ($access_log) {
135
136
$access_log->setData(
136
137
array(
137
138
'u' => $conduit_username,