summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
-rw-r--r--test/test_php_application.py20
-rw-r--r--test/test_php_targets.py2
2 files changed, 21 insertions, 1 deletions
diff --git a/test/test_php_application.py b/test/test_php_application.py
index f442f551..c554200c 100644
--- a/test/test_php_application.py
+++ b/test/test_php_application.py
@@ -664,6 +664,26 @@ opcache.preload_user = %(user)s
resp['headers']['Location'] == 'http://foo/path/'
), 'Location with custom Host over UDS'
+ def test_php_application_forbidden(self, temp_dir):
+ new_root = temp_dir + "/php-root"
+ os.makedirs(new_root + '/path')
+ os.chmod(new_root + '/path', 0o000)
+
+ assert 'success' in self.conf(
+ {
+ "listeners": {"*:7080": {"pass": "applications/php-path"}},
+ "applications": {
+ "php-path": {
+ "type": self.get_application_type(),
+ "processes": {"spare": 0},
+ "root": new_root,
+ }
+ },
+ }
+ ), 'forbidden directory'
+
+ assert self.get(url='/path/')['status'] == 403, 'access forbidden'
+
def test_php_application_extension_check(self, temp_dir):
self.load('phpinfo')
diff --git a/test/test_php_targets.py b/test/test_php_targets.py
index eec1846f..2cf39c1c 100644
--- a/test/test_php_targets.py
+++ b/test/test_php_targets.py
@@ -45,7 +45,7 @@ class TestPHPTargets(TestApplicationPHP):
assert self.get(url='/1')['body'] == '1'
assert self.get(url='/2')['body'] == '2'
- assert self.get(url='/blah')['status'] == 503 # TODO 404
+ assert self.get(url='/blah')['status'] == 404
assert self.get(url='/')['body'] == 'index'
assert self.get(url='/1.php?test=test.php/')['body'] == '1'