Skip to content

Add url format guards - #7670

Merged
wliyongfeng merged 4 commits into
mainfrom
check-url-error
Nov 5, 2026
Merged

wliyongfeng merged 4 commits into
mainfrom
check-url-error

Conversation

@hyifeng

@hyifeng hyifeng commented Nov 5, 2026

Copy link
Copy Markdown
Member

No description provided.

hyifeng and others added 4 commits November 5, 2026 09:32
main stopped reporting 4xx client errors in the error page. This branch already
did that through `isReportableErrorStatus`, so the conflict keeps the shared
helper (now carrying main's rationale) and only adds the reason for it: skip the
profile lookup when nothing is going to be reported.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@hyifeng
hyifeng marked this pull request as ready for review November 5, 2026 01:56
@wliyongfeng
wliyongfeng merged commit f1e4361 into main Nov 5, 2026
1 check passed
@wliyongfeng
wliyongfeng deleted the check-url-error branch November 5, 2026 02:09
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.

2 participants