Skip to content

Improvement: Add URL scheme check for http to SanitizeService.#481

Merged
dansup merged 2 commits into
mainfrom
d2
Feb 7, 2026
Merged

Improvement: Add URL scheme check for http to SanitizeService.#481
dansup merged 2 commits into
mainfrom
d2

Conversation

@shleeable

@shleeable shleeable commented Feb 5, 2026

Copy link
Copy Markdown
Contributor 
    BaseValidator::isLocalObject() requires the URL scheme to be https
    SanitizeService::isLocalObject() ignores the URL scheme entirely

Your Name added 2 commits February 5, 2026 17:08
@dansup dansup merged commit b62af1f into main Feb 7, 2026
10 checks passed
@dansup dansup deleted the d2 branch February 7, 2026 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dansup dansup dansup approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@shleeable @dansup