Skip to content

Comments

Fix: Apply audit fixes (fixes #52)#53

Merged
taylortom merged 4 commits intomasterfrom
issue/52
Feb 20, 2026
Merged

Fix: Apply audit fixes (fixes #52)#53
taylortom merged 4 commits intomasterfrom
issue/52

Conversation

@taylortom
Copy link
Collaborator

Fix

  • Set has* properties to false when request attributes are falsy in addExistenceProps
  • Handle double-slash case in Router path getter when both parent path and child root have slashes

Testing

  1. Verify req.hasBody is false when req.body is null
  2. Verify router paths don't contain double slashes

🤖 Generated with Claude Code

taylortom and others added 4 commits February 20, 2026 20:10
@taylortom taylortom merged commit 06ed115 into master Feb 20, 2026
2 checks passed
@taylortom taylortom deleted the issue/52 branch February 20, 2026 21:59
github-actions bot pushed a commit that referenced this pull request Feb 20, 2026
## [1.4.2](v1.4.1...v1.4.2) (2026-02-20)

### Chore

* Fix falsy attr test to expect false instead of true (refs #52) ([25cd565](25cd565)), closes [#52](#52)
* Update tests to match corrected behaviour (refs #52) ([58aa7a9](58aa7a9)), closes [#52](#52)

### Fix

* Apply audit fixes (fixes #52) (#53) ([06ed115](06ed115)), closes [#52](#52) [#53](#53)
* Prevent double slash in Router path concatenation (fixes #52) ([8fceaef](8fceaef)), closes [#52](#52)
* Set has* to false when request attribute is falsy (refs #52) ([cb78aae](cb78aae)), closes [#52](#52)
@github-actions
Copy link

🎉 This PR is included in version 1.4.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant