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

Handle 416 HTTP response codes properly #7

Open
pmonks opened this issue May 10, 2019 · 0 comments
Open

Handle 416 HTTP response codes properly #7

pmonks opened this issue May 10, 2019 · 0 comments
Assignees
Labels
Milestone

Comments

@pmonks
Copy link
Owner

pmonks commented May 10, 2019

Currently unfurl blindly issues an HTTP range request without checking if the server supports them or not. Although servers are supposed to return the full content with a 200 status code if they do not support HTTP range requests, some return a 416 status code instead, and in these cases unfurl should retry the request without the Range header.

(see also the discussion in issue #6)

@pmonks pmonks added the bug label May 10, 2019
@pmonks pmonks added this to the 1.0.0 milestone May 10, 2019
@pmonks pmonks self-assigned this May 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant