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 HTTP request method #165

Open
PaulAnnekov opened this issue Apr 3, 2016 · 7 comments
Open

Support HEAD HTTP request method #165

PaulAnnekov opened this issue Apr 3, 2016 · 7 comments
Labels
Stale No Activity 60 Days To be marked closed

Comments

@PaulAnnekov
Copy link
Contributor

I think that motioneye should support HEAD method. It will allow to detect fast (related to mobile network) if motioneye is still working. E.g. this mobile app uses HEAD request to check validate image url and it fails for motioneye :(.

I can't imagine any argument against HEAD support. Even google.com supports HEAD.

@ccrisan
Copy link
Collaborator

ccrisan commented Apr 4, 2016

Alright, HEAD support will be added to motionEye :)

@ccrisan
Copy link
Collaborator

ccrisan commented May 18, 2016

Added with commit 0d78dd0.

@ccrisan ccrisan closed this as completed May 18, 2016
@PaulAnnekov
Copy link
Contributor Author

Hm, you have added HEAD support only for main page, but not for https://example.com/picture/1/current/ one. And that mobile app still gets 405 when asking for a HEAD. I think it will be better to add HEAD support for all pages, because HEAD is used to validate that resource exists. I don't see any cons. Should I create new issue?

@ccrisan
Copy link
Collaborator

ccrisan commented Jul 2, 2016

It's done: d78a9c1 :).

@PaulAnnekov
Copy link
Contributor Author

Great man! Now it accepts HEAD, but return Content-Type: text/html; charset=UTF-8, that's wrong :(. GET returns Content-Type: image/jpeg and HEAD should return the same.

@ccrisan
Copy link
Collaborator

ccrisan commented Jul 2, 2016

Hmm that means I'd have to implement separate HEAD methods for each possible URIs. I never thought any piece of software written these days would rely on the existence (and the result) of the HEAD method. I guess I'd probably need to properly write HEAD methods for various URIs available.

@ccrisan ccrisan reopened this Jul 2, 2016
@PaulAnnekov
Copy link
Contributor Author

Yep, HEAD is still used. Not so popular, but still...

@starbasessd starbasessd added the Stale No Activity 60 Days To be marked closed label Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale No Activity 60 Days To be marked closed
Development

No branches or pull requests

3 participants