Patchwork D3434: tests: add tests demonstrating ISE for HTTP 304 responses with hgwebdir

login
register
mail settings
Submitter phabricator
Date May 1, 2018, 12:37 a.m.
Message ID <differential-rev-PHID-DREV-5v7c4awenhu6thgbfgdi-req@phab.mercurial-scm.org>
Download mbox | patch
Permalink /patch/31247/
State Superseded
Headers show

Comments

phabricator - May 1, 2018, 12:37 a.m.
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  There are two separate failures here. One for the Content-Type header.
  Another for the Content-Security-Policy header.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D3434

AFFECTED FILES
  tests/test-hgweb-csp.t
  tests/test-hgweb.t

CHANGE DETAILS




To: indygreg, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/tests/test-hgweb.t b/tests/test-hgweb.t
--- a/tests/test-hgweb.t
+++ b/tests/test-hgweb.t
@@ -875,4 +875,25 @@ 
   Internal Server Error (no-eol)
 
   $ killdaemons.py
+
+HTTP 304 works with hgwebdir (issue5844)
+
+  $ cat > hgweb.conf << EOF
+  > [paths]
+  > /repo = $TESTTMP/test
+  > EOF
+
+  $ hg serve --web-conf hgweb.conf -p $HGPORT -d --pid-file hg.pid -E error.log
+  $ cat hg.pid >> $DAEMON_PIDS
+
+  $ get-with-headers.py --twice --headeronly localhost:$HGPORT 'repo/static/style.css' - date etag server
+  200 Script output follows
+  content-length: 2677
+  content-type: text/css
+  500 Internal Server Error
+  transfer-encoding: chunked
+  [1]
+
+  $ killdaemons.py
+
   $ cd ..
diff --git a/tests/test-hgweb-csp.t b/tests/test-hgweb-csp.t
--- a/tests/test-hgweb-csp.t
+++ b/tests/test-hgweb-csp.t
@@ -54,6 +54,12 @@ 
   200 Script output follows
   content-security-policy: script-src https://example.com/ 'unsafe-inline'
 
+  $ get-with-headers.py --twice --headeronly localhost:$HGPORT repo1/static/style.css content-security-policy
+  200 Script output follows
+  content-security-policy: script-src https://example.com/ 'unsafe-inline'
+  500 Internal Server Error
+  [1]
+
 repo page should send CSP by default, include etag w/o nonce
 
   $ get-with-headers.py --headeronly localhost:$HGPORT repo1 content-security-policy etag