HomePhabricator

Fix a potentially lax hash comparison

Description

Fix a potentially lax hash comparison

Summary: Via HackerOne. See D14025. I missed this comparison when making the original change.

Test Plan:

  • Used cat mail.txt | scripts/mail/mail_handler.php --process-duplicates to pipe mail in a whole lot of times.
  • Tried bad hashes, saw rejections.
  • Tried good hash, saw mail accepted.

Reviewers: chad

Reviewed By: chad

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

Details

Provenance
epriestleyAuthored on
epriestleyPushed on Nov 10 2015, 6:22 PM
Reviewer
chad
Differential Revision
D14455: Fix a potentially lax hash comparison
Parents
rP64ad44cffb24: Always override `auth.email-domains` when running unit tests
Branches
Unknown
Tags
Unknown
Build Status
Buildable 8762
Build 10193: Run Core Tests