RFC8246 HTTP Immutable Responses#462
Merged
arturobernalg merged 1 commit intoapache:5.3.xfrom Jul 24, 2023
Merged
Conversation
ok2c
changed the title
ok2c
reviewed
Jul 4, 2023
|
|
||
| // calculate freshness lifetime | ||
| final Duration freshnessLifetime = calculateFreshnessLifetime(cacheControl, response); | ||
|
|
Member
There was a problem hiding this comment.
@arturobernalg I cannot find immutable cache control directive mentioned anywhere in the RFC 9111. Am I missing something?
Member
Author
Member
@arturobernalg Could you please reflect this in the PR title and the commit message? |
arturobernalg
changed the title
Member
Author
@ok2c |
Member
|
@arturobernalg Looks good to me, but can we drop |
This commit adds the functionality to handle the 'immutable' directive in the Cache-Control header as per the RFC8246 specifications. Key changes include: - The 'immutable' directive is checked in the Cache-Control of an HTTP response, indicating that the origin server will not update the resource representation during the response's freshness lifetime. - If the 'immutable' directive is present and the response is still fresh, the response is considered cacheable without further validation. - Ignoring any arguments with the 'immutable' directive, as per RFC stipulations. - Treating multiple instances of the 'immutable' directive as equivalent to one.
Member
Author
@ok2c Removed. |
ok2c
approved these changes
Jul 23, 2023
ok2c
pushed a commit
that referenced
this pull request
Aug 20, 2023
This commit adds the functionality to handle the 'immutable' directive in the Cache-Control header as per the RFC8246 specifications. Key changes include: - The 'immutable' directive is checked in the Cache-Control of an HTTP response, indicating that the origin server will not update the resource representation during the response's freshness lifetime. - If the 'immutable' directive is present and the response is still fresh, the response is considered cacheable without further validation. - Ignoring any arguments with the 'immutable' directive, as per RFC stipulations. - Treating multiple instances of the 'immutable' directive as equivalent to one.
ok2c
pushed a commit
that referenced
this pull request
Dec 13, 2023
This commit adds the functionality to handle the 'immutable' directive in the Cache-Control header as per the RFC8246 specifications. Key changes include: - The 'immutable' directive is checked in the Cache-Control of an HTTP response, indicating that the origin server will not update the resource representation during the response's freshness lifetime. - If the 'immutable' directive is present and the response is still fresh, the response is considered cacheable without further validation. - Ignoring any arguments with the 'immutable' directive, as per RFC stipulations. - Treating multiple instances of the 'immutable' directive as equivalent to one.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This commit adds the functionality to handle the 'immutable' directive in the Cache-Control header as per the RFC specifications.
Key changes include: