Skip to content

ETag must be an exact match#826

Closed
da2x wants to merge 1 commit intotextpattern:devfrom
da2x:bugfix_feed_etag_validation
Closed

ETag must be an exact match#826
da2x wants to merge 1 commit intotextpattern:devfrom
da2x:bugfix_feed_etag_validation

Conversation

@da2x
Copy link
Copy Markdown
Contributor

@da2x da2x commented Oct 26, 2016

Fixes these issues:

  • Partial string match will incorrectly report "304" when the rss_how_many setting is increased. (Or a ?limit=value is masked and increased through mod_rewrite.)
  • Apache appends a "-<encoding>" suffix to etags (e.g. ETag: "etagvalue-gzip") when it’s been processed through an encoding output filter for which Textpattern shouldn’t reply 304. Breaks caching when two non-equivalent resources (different encodings) report to have the same ETag.

You can see the headers not updating in response to content negotiation here: https://redbot.org/?uri=http%3A%2F%2Ftextpattern.com%2Fatom%2F%3Fsection%3Dweblog (or see a snapshot from archive.org if you’re from the future.)

@bloatware
Copy link
Copy Markdown
Member

What was the original reason for strstr(), I wonder?

@da2x
Copy link
Copy Markdown
Contributor Author

da2x commented Nov 2, 2016

What was the original reason for strstr(), I wonder?

A bug. I can’t find any historical reason for this in older revisions.

@da2x da2x closed this Nov 3, 2016
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