revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-22 05:20:56 +01:00
Code Issues Releases Wiki Activity

Allow new Conduit clients by default.

Browse source 
Summary:
There's no reason to default-reject clients since they can just
pretend to be arc anyway. If they're speaking the right protocol, let them
communicate over Conduit.

Test Plan:
Changed arc to identify as 'arczsdba', ran an arc command.

Reviewed By: simpkins
Reviewers: simpkins
CC: simpkins
Differential Revision: 132
This commit is contained in:
epriestley 2011-04-13 18:56:32 -07:00
parent 2bd51fd125
commit d4fb7f578e
2 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/applications/conduit/method/conduit/connect/ConduitAPI_conduit_connect_Method.php
View file

@ -102,7 +102,8 @@ class ConduitAPI_conduit_connect_Method extends ConduitAPIMethod {
}
break;
default:
throw new ConduitException('ERR-UNKNOWN-CLIENT');
// Allow new clients by default.
break;
}
$token = $request->getValue('authToken');

1
src/applications/conduit/method/conduit/connect/__init__.php
View file

@ -10,6 +10,7 @@ phutil_require_module('phabricator', 'applications/conduit/method/base');
phutil_require_module('phabricator', 'applications/conduit/protocol/exception');
phutil_require_module('phabricator', 'applications/conduit/storage/connectionlog');
phutil_require_module('phabricator', 'applications/people/storage/user');
phutil_require_module('phabricator', 'infrastructure/env');
phutil_require_module('phabricator', 'storage/queryfx');
phutil_require_module('phutil', 'utils');