HomePhorge

Use OpaqueEnvelopes for all passwords in Phabricator

Description

Use OpaqueEnvelopes for all passwords in Phabricator

Summary:
See D2991 / T1526. Two major changes here:

  • PHP just straight-up logs passwords on ldap_bind() failures. Suppress that with "@" and keep them out of DarkConsole by enabling discard mode.
  • Use PhutilOpaqueEnvelope whenever we send a password into a call stack.

Test Plan:

  • Created a new account.
  • Reset password.
  • Changed password.
  • Logged in with valid password.
  • Tried to login with bad password.
  • Changed password via accountadmin.
  • Hit various LDAP errors and made sure nothing appears in the logs.

Reviewers: vrana, btrahan

Reviewed By: vrana

CC: aran

Differential Revision: https://secure.phabricator.com/D2993

Details

Provenance
epriestleyAuthored on Jul 17 2012, 12:06 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPae2e73ce809f: Add "stop on redirect" and "always profile" debugging options
Branches
Unknown
Tags
Unknown

Event Timeline