Skip to content

Add Have I Been Pwned password requirements check - #426

Open
mgaffigan wants to merge 2 commits into
OpenIntegrationEngine:mainfrom
mgaffigan:feat/hibp-password-verifier
Open

Add Have I Been Pwned password requirements check#426
mgaffigan wants to merge 2 commits into
OpenIntegrationEngine:mainfrom
mgaffigan:feat/hibp-password-verifier

Conversation

@mgaffigan

Copy link
Copy Markdown
Contributor

NIST SP 800-63B §5.1.1.2 requires passwords to be checked against leak lists. This fixes that gap in OIE by adding support for checking HIBP's API. Confidentiality of the password is maintained by k-anonymity. The setting is default-enabled on new installations.

Testing can be accomplished by trying to change the password to "correct horse battery staple":

image

The functionality is default-disabled for upgrades, and can be enabled/disabled by setting/clearing the new password.breachedurl mirth.properties setting.

# password requirements
# ...snip...
# Check new passwords against the Have I Been Pwned range API, as recommended by
# NIST SP 800-63B (see https://haveibeenpwned.com/NIST).  Secured by k-anonymity.
password.breachedurl = https://api.pwnedpasswords.com/range/

@github-actions

github-actions Bot commented Aug 29, 2026

Copy link
Copy Markdown

Test Results

681 tests  +6   681 ✅ +6   2m 20s ⏱️ + 1m 26s
115 suites +1     0 💤 ±0 
115 files   +1     0 ❌ ±0 

Results for commit 7385ea7. ± Comparison against base commit 423df18.

♻️ This comment has been updated with latest results.

@mgaffigan mgaffigan changed the title Add Have-I-Been-Pwned password requirements check Add Have I Been Pwned password requirements check Aug 29, 2026
@mgaffigan
mgaffigan force-pushed the feat/hibp-password-verifier branch from 1170e49 to 89ce90f Compare August 29, 2026 19:25
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
mgaffigan force-pushed the feat/hibp-password-verifier branch from 89ce90f to 7385ea7 Compare August 29, 2026 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant