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

Consider supporting head requests #166

Closed
dshanske opened this issue May 20, 2019 · 3 comments
Closed

Consider supporting head requests #166

dshanske opened this issue May 20, 2019 · 3 comments

Comments

@dshanske
Copy link

My code makes a head request to a URL. Granary doesn't support HEAD.

@snarfed
Copy link
Owner

snarfed commented May 20, 2019

sorry for the trouble!

the difficulty with this is that i need to make an external HTTP request, sometimes more than one, before i know which HTTP status code i can return. HEADs are supposed to be light, and granary already gets a lot of HEAD traffic (mostly from low quality bots) that i'd rather not start serving and absorbing that load for.

i'm open to ideas! having said that, all other feed readers seem to handle granary's lack of HEAD fine. if you use the content-type from it, consider just falling back to GET, and check its returned content type if you need to?

@dshanske
Copy link
Author

That's fine. I am already coding around it

@snarfed
Copy link
Owner

snarfed commented Jun 1, 2019

thanks! closing for now. feel free to reopen!

@snarfed snarfed closed this as completed Jun 1, 2019
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

2 participants