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

fix(http): correct HTTP 418 description and coverage #823

Merged
merged 3 commits into from
Dec 6, 2024

Conversation

blackshadev
Copy link
Contributor

Fixes #778 .

Somebody has to fix the most important and widely used HTTP statuscodes out there.

For anybody who doesn't know, 418 is a official HTTP status code introduces as an RFC during april fools by the official networking working committee at that time. I don't know why and how, but it was accepted and it stuck. Recently there was a movement to remove it, but surprise surprise , after more than 10 years not only did people use it. People loved this small easter egg in a rigid spec of the foundation of our internet.

Read the RFC if you have the time, it is a proper laugh.

@blackshadev blackshadev changed the title fix(router): Just fixing some minor casing issues in HTTP 418 fix(router): fix minor issues in HTTP 418 and missing coverage Dec 6, 2024
@blackshadev blackshadev force-pushed the fix-april-fools-statuscode branch from 5bf5065 to 29651c8 Compare December 6, 2024 21:57
src/Tempest/Http/src/Status.php Outdated Show resolved Hide resolved
@innocenzi innocenzi changed the title fix(router): fix minor issues in HTTP 418 and missing coverage fix(http): correct HTTP 418 description and coverage Dec 6, 2024
@innocenzi innocenzi merged commit dbad109 into tempestphp:main Dec 6, 2024
60 checks passed
@aidan-casey aidan-casey added the Perfect Storm This label is awarded to issues that provide excellent detail and/or a failing test in a PR. label Dec 7, 2024
@aidan-casey
Copy link
Member

aidan-casey commented Dec 7, 2024

Thank you, @blackshadev! This was driving me nuts and I'm awarding this the "Perfect Storm" label for adding context. 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Perfect Storm This label is awarded to issues that provide excellent detail and/or a failing test in a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Http Status Code 418 has the wrong description
3 participants