Skip to content

Fix isDisallowed invalid URL handling#45

Open
AnandChowdhary wants to merge 1 commit into
masterfrom
fix/daily-maintenance-20260512
Open

Fix isDisallowed invalid URL handling#45
AnandChowdhary wants to merge 1 commit into
masterfrom
fix/daily-maintenance-20260512

Conversation

@AnandChowdhary
Copy link
Copy Markdown
Collaborator

Summary

  • propagate undefined from isAllowed() through isDisallowed() for invalid/out-of-scope URLs instead of coercing it to true
  • add regression coverage for invalid URL handling across both methods
  • separate port-mismatch cases from explicitly disallowed URL cases in tests

Test plan

  • git diff --cached --check
  • added-line security scan: no findings
  • independent Claude Code review: passed; no security concerns or logic errors
  • npm test (44 passing, 100% coverage)
  • npm audit --omit=dev (0 vulnerabilities)

Refs #41

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.

1 participant