test: add weak password-reset token fixture (CWE-640) - #177
Merged
ralyodio merged 3 commits intoAug 18, 2026
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
# Conflicts: # docs/VULNERABILITY_CATALOG.md # vulns/VULNERABILITY_CATALOG.json
ralyodio
added a commit
to ezequiellich44-cmd/malware-test-prs
that referenced
this pull request
Aug 18, 2026
The generated catalog conflicted with CWE-640 (profullstack#177): both sides insert into the same sorted CWE list and bump the same count. Resolved by regenerating with scripts/generate-catalog.py rather than by hand, so the index matches the sources instead of either side of the conflict. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
ralyodio
added a commit
to ezequiellich44-cmd/malware-test-prs
that referenced
this pull request
Aug 18, 2026
The generated catalog conflicted with CWE-640 (profullstack#177) and CWE-434 (profullstack#178): every test-case PR rewrites the same sorted CWE list and count, so any two open at once collide. Resolved by regenerating with scripts/generate-catalog.py rather than by hand, so the index matches the sources instead of either side of the conflict. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
ralyodio
added a commit
to ezequiellich44-cmd/malware-test-prs
that referenced
this pull request
Aug 18, 2026
The generated catalog conflicted with CWE-640 (profullstack#177), CWE-434 (profullstack#178) and CWE-620 (profullstack#179): every test-case PR rewrites the same sorted CWE list and count, so any two open at once collide. Resolved by regenerating with scripts/generate-catalog.py rather than by hand, so the index matches the sources instead of either side of the conflict. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a CWE-640 (Weak Password Recovery Mechanism for Forgotten Password) test case: a reset token deterministically derived from the account email plus wall-clock time, and a verification sink with a static salt, no expiry check and no attempt cap. Includes a safe counterpart using secrets.token_urlsafe with a TTL. Catalog regenerated.