Page MenuHomePhabricator

Provide `auth lock` and `auth unlock` to restrict authentication provider management to the CLI
Closed, ResolvedPublic

Description

See some discussion in T6755. Providing auth lock and auth unlock commands would moderately increase the security of installs in the face of a compromised administrative account. There is normally very little need to edit authentication configuration after initial setup, and it necessarily has a wide array of access implications.

With respect to T6755, specifically, it is potentially an SSRF vector. It's somewhat hard to imagine this being a meaningful component of a practical attack, but not wholly absurd.

Revisions and Commits

Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision

Event Timeline

epriestley raised the priority of this task from to Low.
epriestley updated the task description. (Show Details)
epriestley added projects: Security, Auth.
epriestley added a subscriber: epriestley.

After T13222, this is more relevant:

  • Administrator accounts may disable all MFA from the web UI.
  • Auth guidance is a more compelling target for content injection than most injectable channels (e.g., the login screen can be made to say "go to evil.ru and type in ur password qt ;)", which seems authoritative-ish).
epriestley added a revision: Restricted Differential Revision.Jan 25 2019, 6:45 PM
epriestley added a commit: Restricted Diffusion Commit.Jan 28 2019, 5:44 PM
amckinley added a revision: Restricted Differential Revision.Apr 10 2019, 10:35 PM
amckinley added a commit: Restricted Diffusion Commit.Apr 10 2019, 11:15 PM
amckinley added a revision: Restricted Differential Revision.Apr 11 2019, 8:16 PM
amckinley added a commit: Restricted Diffusion Commit.Apr 17 2019, 6:08 PM
epriestley added a revision: Restricted Differential Revision.Apr 18 2019, 12:24 AM
epriestley added a commit: Restricted Diffusion Commit.Apr 18 2019, 2:54 AM

This could be made slightly cleaner with a setSummary() to set a shorter summary:

Screen Shot 2019-04-18 at 7.01.55 AM.png (251×916 px, 51 KB)

epriestley added a revision: Restricted Differential Revision.Apr 18 2019, 2:05 PM
epriestley added a commit: Restricted Diffusion Commit.Apr 18 2019, 8:08 PM
epriestley added a commit: Restricted Diffusion Commit.Apr 22 2019, 5:38 AM
epriestley added a commit: Restricted Diffusion Commit.Apr 22 2019, 2:00 PM
amckinley added a revision: Restricted Differential Revision.Jul 10 2019, 3:05 PM
amckinley closed this task as Resolved by committing Restricted Diffusion Commit.Jul 15 2019, 6:53 PM
amckinley claimed this task.
amckinley added a commit: Restricted Diffusion Commit.