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

HTTPCLIENT-2293 - Implement 'If-Range' request validation as per RFC9110 #485

Merged
merged 1 commit into from
Sep 23, 2023

Conversation

arturobernalg
Copy link
Member

@arturobernalg arturobernalg commented Sep 20, 2023

This pull request aims to introduce support for the "If-Range" header as specified in RFC 9110. The "If-Range" header provides a more efficient way to handle partial content retrieval and is a crucial step towards full support for Range requests in our HTTP client component.

Note: This is the first step towards comprehensive support for HTTP Range Requests in our project.

@arturobernalg
Copy link
Member Author

arturobernalg commented Sep 20, 2023

I would greatly appreciate your feedback, @reschke, on this implementation, particularly regarding its compliance with RFC standards and any best practices to consider.

…9110

- Ensure the presence of 'Range' header when 'If-Range' is specified.
- Enforce strong validator requirements when 'If-Range' is paired with a Date.
- Exit processing early if 'Last-Modified' header is missing, ensuring strong validation adherence.
@arturobernalg arturobernalg merged commit 16aa558 into apache:5.4.x Sep 23, 2023
7 checks passed
ok2c pushed a commit that referenced this pull request Dec 13, 2023
…9110 (#485)

- Ensure the presence of 'Range' header when 'If-Range' is specified.
- Enforce strong validator requirements when 'If-Range' is paired with a Date.
- Exit processing early if 'Last-Modified' header is missing, ensuring strong validation adherence.
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

Successfully merging this pull request may close these issues.

2 participants