Skip to content

fix: use Argon2id for password hashing - #65

Draft
cloudcarver wants to merge 1 commit into
mainfrom
agent/security-password-kdf
Draft

fix: use Argon2id for password hashing#65
cloudcarver wants to merge 1 commit into
mainfrom
agent/security-password-kdf

Conversation

@cloudcarver

Copy link
Copy Markdown
Owner

Summary

  • replace fast SHA-256 password storage with versioned Argon2id hashes and random salts
  • verify encoded hashes in constant time with bounded parameters
  • retain legacy SHA-256 verification and upgrade legacy records after successful login
  • use a compare-and-swap database update during automatic upgrades
  • add malformed, wrong-password, randomization, compatibility, and upgrade tests

Validation

  • go test ./...
  • go test -race ./pkg/utils ./pkg/service
  • go vet ./...

Closes #40

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.

[Security] Replace fast password hashing with a password KDF

1 participant