Skip to content

Harden URL and environment path handling - #2

Merged
ChanTso merged 1 commit into
mainfrom
agent/fix-codeql-highs
Jul 30, 2026
Merged

ChanTso merged 1 commit into
mainfrom
agent/fix-codeql-highs

Conversation

@ChanTso

@ChanTso ChanTso commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • parse and validate RedNote collection URLs by exact HTTPS hostname and /board/ path
  • reject deceptive substring hosts, embedded credentials, non-HTTPS URLs, and unexpected ports
  • replace filesystem resolution of CONDA_PREFIX with a pure lexical boundary check
  • add focused regression tests for both CodeQL findings

Validation

  • Ruff passes
  • 47 tests pass
  • compileall passes
  • privacy cross-audit remains at zero matches
  • Gitleaks reports zero findings

Closes CodeQL alerts #1 and #2 once the post-merge scan confirms the fixes.

@ChanTso
ChanTso merged commit aa58a7c into main Jul 30, 2026
4 checks passed
@ChanTso
ChanTso deleted the agent/fix-codeql-highs branch July 30, 2026 15:31
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