Skip to content

test: validate provider URLs structurally - #64

Merged
stefan-jansen merged 2 commits into
mainfrom
codex/close-security-alerts
Sep 24, 2026
Merged

stefan-jansen merged 2 commits into
mainfrom
codex/close-security-alerts

Conversation

@stefan-jansen

@stefan-jansen stefan-jansen commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • replace substring URL assertions with parsed scheme, exact hostname, and path checks
  • preserve provider contracts while resolving three CodeQL incomplete-URL-sanitization findings
  • refresh the pinned ecosystem workflow so the approved Python 3.15 wait covers development versions below 0.2

Affected version

ml4t-data 0.1.8 and current main.

Verification

  • 121 focused provider tests passed
  • Ruff check and format passed
  • pre-commit hooks passed
  • CodeQL passed
  • all 25 applicable remote checks passed; the Python 3.15 prerelease matrix was intentionally skipped in favor of its approved exception check

Closes #67

Copilot AI lite review requested due to automatic review settings September 24, 2026 17:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@stefan-jansen stefan-jansen added compatibility: none No known compatibility impact ecosystem Tracked by a shared ML4T ecosystem requirement priority: high Material user impact requiring prompt remediation status: accepted Accepted for implementation type: maintenance Internal maintenance without user-visible behavior labels Sep 24, 2026
@stefan-jansen
stefan-jansen merged commit de432db into main Sep 24, 2026
26 checks passed
@stefan-jansen
stefan-jansen deleted the codex/close-security-alerts branch September 24, 2026 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compatibility: none No known compatibility impact ecosystem Tracked by a shared ML4T ecosystem requirement priority: high Material user impact requiring prompt remediation status: accepted Accepted for implementation type: maintenance Internal maintenance without user-visible behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace unsafe URL substring assertions in provider tests

2 participants