feature/verify-alg #8

Merged

sezieru merged 13 commits from feature/verify-alg into main

2025-10-16 01:43:47 +00:00

sezieru commented

2025-10-16 01:42:25 +00:00

Owner

Adds a verify feature via -V flag which can be used to verify a specified hash given a key (with -k). If a key is not specified, an error message is provided and exit code is set appropriately. Otherwise, crypt() is used to verify the hash with the provided key and output "OK"/"FAIL" & exit codes as appropriate.

Adds a **verify** feature via **`-V`** flag which can be used to verify a specified hash given a key (with `-k`). If a key is not specified, an error message is provided and exit code is set appropriately. Otherwise, `crypt()` is used to verify the hash with the provided key and output "OK"/"FAIL" & exit codes as appropriate.

sezieru self-assigned this

2025-10-16 01:42:25 +00:00

sezieru added 13 commits

2025-10-16 01:42:25 +00:00

feat(verify): [TEMPORARY] add -V to verify hash with given key ce8d35c810

This is intended to show a naive implementation of hash verification.
This method is intended to be used for instructional purposes only,
and will be replaced by a far more optimal solution, which will also
serve as a valuable educational lesson.

refactor(core): refactor select_alg to no longer require heap allocation & strncpy() 60da99e622

fix(comment): updated comments to reflect new select_alg() structure; PR-ready d72061b736

Merge remote-tracking branch 'origin/refactor/alt-select' into feature/verify-alg f51000a084

fix(pfx): adjust to handle new select_alg() construction; avoid free(pfx) 0534ce1948

fix(verify): **vastly** improve hash verification bb54b67595

- uses provided hash as 'setting' for call to crypt_r(), allowing salt & tunings to be reused
- simplifies code dramatically, and permits greater opportunity for optimization
- eliminates lots of potential bugs & complexities in manual salt & tunings retrieval
- promotes better design paradigm for teaching & KISS principles

docs(comment): add comments to detail the verification change & motivation c6d9a67a9e

style(verify): adjust message output during verification 72412ec860

style(debug): adjust options message for debug output b6543619f6

test(verify): add OK/FAIL TAP cases; add T10 descriptive comment 5d2ef6210e

fix(debug): fix test compilation line for debug target in Makefile a61c6ad798

fix(docs): fixed an error in TESTING which referred to old structure name f079d70862

docs(verify): document -V option usage, examples, and exit codes/output e9902be377

sezieru merged commit 83e5562a53 into main

2025-10-16 01:43:47 +00:00

sezieru referenced this pull request from a commit

2025-10-16 01:43:49 +00:00

Merge pull request 'feature/verify-alg' (#8) from feature/verify-alg into main