GitHub Advisoryadvisory · Go
GHSA-g636-q5fc-4pr7 accounts: Hash account number using Salt
LOW
Official page ↗
- Ecosystem
- Go
- Packages
- github.com/moov-io/customers
- Severity
- Low
- Reviewed
- 2021-05-21
Affected packages
• Go github.com/moov-io/customers: < 0.5.0 → fixed in 0.5.0
Description
@alovak found that currently when we build hash of account number we do not "salt" it. Which makes it vulnerable to rainbow table attack.
What did you expect to see?
I expected salt (some random number from configuration) to be used in hash.AccountNumber
I would generate salt per tenant at least (maybe per organization).