Use a password input for HTTP Basic Auth in repositories
Summary: See D902. As @abdul notes, a password input is probably more appropraite here. Test Plan: Mashed stuff into it, got bullets instead of text. Reviewers: abdul, jungejason, nh, tuomaspelkonen, aran Reviewed By: nh CC: aran, nh Differential Revision: 913
This commit is contained in:
@@ -453,7 +453,7 @@ class PhabricatorRepositoryEditController
|
||||
->setLabel('HTTP Basic Login')
|
||||
->setValue($repository->getDetail('http-login')))
|
||||
->appendChild(
|
||||
id(new AphrontFormTextControl())
|
||||
id(new AphrontFormPasswordControl())
|
||||
->setName('http-pass')
|
||||
->setLabel('HTTP Basic Password')
|
||||
->setValue($repository->getDetail('http-pass')));
|
||||
|
||||
@@ -17,6 +17,7 @@ phutil_require_module('phabricator', 'applications/repository/storage/repository
|
||||
phutil_require_module('phabricator', 'infrastructure/env');
|
||||
phutil_require_module('phabricator', 'view/control/table');
|
||||
phutil_require_module('phabricator', 'view/form/base');
|
||||
phutil_require_module('phabricator', 'view/form/control/password');
|
||||
phutil_require_module('phabricator', 'view/form/control/select');
|
||||
phutil_require_module('phabricator', 'view/form/control/static');
|
||||
phutil_require_module('phabricator', 'view/form/control/submit');
|
||||
|
||||
Reference in New Issue
Block a user