Skip to content
This repository was archived by the owner on Nov 27, 2020. It is now read-only.

Commit f63bbbf

Browse files
phihagfabpot
authored andcommitted
Properly lock down app_dev.php
Currently, web/app_dev.php in the symphony-standard package limits requests to those coming from CLI and unproxied localhost. This prevents an attacker to execute this file in a production environment. IPv4 localhost requests can come from 127.0.0.0/8, so the current check is actually a little bit too restrictive, but safe. IPv6 localhost requests come from ::1/128. The address fe80::1, however, is a link-local IPv6 address like any other. Anyone in the local network can take it and communicate with it. Therefore, it does not belong here.
1 parent 1b197e6 commit f63bbbf

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

web/app_dev.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
// Feel free to remove this, extend it, or make something more sophisticated.
1313
if (isset($_SERVER['HTTP_CLIENT_IP'])
1414
|| isset($_SERVER['HTTP_X_FORWARDED_FOR'])
15-
|| !(in_array(@$_SERVER['REMOTE_ADDR'], array('127.0.0.1', 'fe80::1', '::1')) || php_sapi_name() === 'cli-server')
15+
|| !(in_array(@$_SERVER['REMOTE_ADDR'], array('127.0.0.1', '::1')) || php_sapi_name() === 'cli-server')
1616
) {
1717
header('HTTP/1.0 403 Forbidden');
1818
exit('You are not allowed to access this file. Check '.basename(__FILE__).' for more information.');

0 commit comments

Comments
 (0)