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

fasthttp.Server GetOnly should also support HEAD requests #1455

Closed
TheKigen opened this issue Dec 20, 2022 · 1 comment
Closed

fasthttp.Server GetOnly should also support HEAD requests #1455

TheKigen opened this issue Dec 20, 2022 · 1 comment

Comments

@TheKigen
Copy link

Hi,

While GetOnly does suggest only GET requests should be supported I think it'd be way more useful of a setting if HEAD requests were also allowed with GetOnly set. HEAD requests are functionally similar to GET with the exception that the body of the reply is not expected. HEAD requests are useful for monitoring when you just want to see if the HTTP status is 200. And the main purpose I see behind GetOnly is to block DoS via pushing request bodies and random methods that various bots spam HTTP servers with.

@TheKigen
Copy link
Author

Closing this due to the PR being merged.

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

No branches or pull requests

1 participant