Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cherry-pick "Tests: Use .invalid TLD for invalid requests" #25134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nico
Copy link
Contributor

@nico nico commented Oct 18, 2024

RFC2606 and RFC6761 define .invalid as a guaranteed-invalid TLD, so let's use it. Theoretically this should make the request fail much faster.

(cherry picked from commit ea95e325390cefd4e1289e8c3de6b56d2c34473f)


LadybirdBrowser/ladybird#1597

@nico nico marked this pull request as ready for review October 18, 2024 01:33
@github-actions github-actions bot added the 👀 pr-needs-review PR needs review from a maintainer or community member label Oct 18, 2024
Copy link

stale bot commented Nov 9, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions!

@stale stale bot added the stale label Nov 9, 2024
RFC2606 and RFC6761 define .invalid as a guaranteed-invalid TLD, so
let's use it. Theoretically this should make the request fail much
faster.

(cherry picked from commit ea95e325390cefd4e1289e8c3de6b56d2c34473f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👀 pr-needs-review PR needs review from a maintainer or community member stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants