Clean up unnecessary comments and references

- Remove vague TODO comment, clarify legacy SHA256 support purpose
- Make CONTRIBUTING.md more generic (not GitHub-specific)
- Remove Git from prerequisites (only needed if cloning)
This commit is contained in:
fraggle 2026-01-11 19:04:42 -04:00
parent 46a22138b9
commit 05bab4b9c9
3 changed files with 2 additions and 3 deletions

View File

@ -70,4 +70,4 @@ Requires `pandoc` or `go-md2man` to be installed.
## Questions?
Feel free to open an issue for questions or discussions.
Feel free to open an issue or start a discussion in the repository.

View File

@ -20,7 +20,6 @@ A modern password manager using AGE encryption.
- **Go 1.21 or later** - Required for building
- **Linux/Unix system** - Designed for Unix-like operating systems
- **Git** - For cloning the repository
- **pandoc** or **go-md2man** (optional) - For building manpages
- **xclip** or **wl-clipboard** - For clipboard support (X11/Wayland)

View File

@ -171,7 +171,7 @@ func verifyMasterPassword(storeDir string, password []byte) (bool, error) {
}
// verifyLegacySHA256 verifies against old SHA256 hashes (for migration)
// TODO: Maybe remove this eventually? But backwards compat is nice...
// Kept for backwards compatibility with stores created before Argon2id migration
func verifyLegacySHA256(password []byte, storedHash string) (bool, error) {
// Old stores used SHA256 - not ideal but we support it for migration
hash := sha256Sum(password)