Skip to content
This repository has been archived by the owner on Aug 26, 2022. It is now read-only.

accounts: Hash account number using Salt

Low
adamdecaf published GHSA-g636-q5fc-4pr7 Oct 22, 2020

Package

customers

Affected versions

< v0.5.0

Patched versions

v0.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).

Severity

Low

CVE ID

No known CVE

Weaknesses

No CWEs