gitweb: refactor If-Modified-Since handling
Get series with b4
b4 am [email protected]Versions newest first
- v6 Latest3 patches 2 repliesIn review
Changes in v6 from the cover letter
* Use test_when_finished to unset HTTP_IF_MODIFIED_SINCE.
* Actually use the tree ID in the modification: tree snapshot test.
* Conditional last_modified argument for $cgi->header git_snapshot and git_feed.
* tree-ish -> trees in git_snapshot commit message and test name.
Patches in v6 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/3 | gitweb: add `status` headers to git_feed() responses.gitweb/ | +4 | 0 replies | |
| 2/3 | gitweb: refactor If-Modified-Since handlinggitweb/ t/ | +55 | 0 replies | |
| 3/3 | gitweb: add If-Modified-Since handling to git_snapshot().gitweb/ t/ | +43 | 1 reply |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v6 yet.
Cover letter [PATCH v6 0/3] gitweb: refactor If-Modified-Since handling
Changes since v5:
* Use test_when_finished to unset HTTP_IF_MODIFIED_SINCE.
* Actually use the tree ID in the modification: tree snapshot test.
* Conditional last_modified argument for $cgi->header git_snapshot and git_feed.
* tree-ish -> trees in git_snapshot commit message and test name.
Unchanged since v5:
* Patch 1/3, but I'm resending it anyway with git send-email. I could skip resending it with:
git send-email … HEAD^^
but that would mess up the patch numbering.
W. Trevor King (3):
gitweb: add status headers to git_feed() responses.
gitweb: refactor If-Modified-Since handling
gitweb: add If-Modified-Since handling to git_snapshot().
gitweb/gitweb.perl | 65 ++++++++++++++++++------------ t/t9501-gitweb-standalone-http-status.sh | 60 +++++++++++++++++++++++++++- 2 files changed, 98 insertions(+), 27 deletions(-)
Signature · 1 line
1.7.3.4