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

Support HEAD request #1833

Closed
sindresorhus opened this issue Apr 30, 2019 · 5 comments · Fixed by #1875
Closed

Support HEAD request #1833

sindresorhus opened this issue Apr 30, 2019 · 5 comments · Fixed by #1875

Comments

@sindresorhus
Copy link

See #924 (comment)

@alexander-akait
Copy link
Member

We already support difference methods (https://github.com/webpack/webpack-dev-middleware#methods), by defaults we supports only GET. What is problem? Support by default all possible? Or support only HEAD? Or no documentation about this?

@sindresorhus
Copy link
Author

What's the purpose of limiting it?

@alexander-akait
Copy link
Member

alexander-akait commented Apr 30, 2019

@sindresorhus historical reason, previously we support only GET request for webpack-dev-middleware so we support only GET for webpack-dev-server, now we support all possible requests for webpack-dev-middleware, but by default we still support only GET. We can change this, because i can't see any problems to allowing any requests. Anyway can you provide case, just for information and future improvements, thanks.

@sindresorhus
Copy link
Author

I use HEAD request to check whether the server is running.

Another use-case here: #924 (comment)

@alexander-akait
Copy link
Member

Thanks, mark as feature, feel free to send a PR if you want this asap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment