Page MenuHomePhabricator

D8572.id20341.diff
No OneTemporary

D8572.id20341.diff

diff --git a/src/applications/config/option/PhabricatorSecurityConfigOptions.php b/src/applications/config/option/PhabricatorSecurityConfigOptions.php
--- a/src/applications/config/option/PhabricatorSecurityConfigOptions.php
+++ b/src/applications/config/option/PhabricatorSecurityConfigOptions.php
@@ -52,8 +52,9 @@
"want (to any other string), but doing so will break existing ".
"sessions and CSRF tokens.")),
$this->newOption('security.require-https', 'bool', false)
+ ->setLocked(true)
->setSummary(
- pht("Force users to connect via https instead of http."))
+ pht("Force users to connect via HTTPS instead of HTTP."))
->setDescription(
pht(
"If the web server responds to both HTTP and HTTPS requests but ".
@@ -67,15 +68,13 @@
"balancer which terminates HTTPS connections and you can not ".
"reasonably configure more granular behavior there.\n\n".
- "NOTE: Phabricator determines if a request is HTTPS or not by ".
- "examining the PHP \$_SERVER['HTTPS'] variable. If you run ".
+ "IMPORTANT: Phabricator determines if a request is HTTPS or not ".
+ "by examining the PHP \$_SERVER['HTTPS'] variable. If you run ".
"Apache/mod_php this will probably be set correctly for you ".
"automatically, but if you run Phabricator as CGI/FCGI (e.g., ".
"through nginx or lighttpd), you need to configure your web ".
"server so that it passes the value correctly based on the ".
- "connection type. Alternatively, you can add a PHP snippet to ".
- "the top of this configuration file to directly set ".
- "\$_SERVER['HTTPS'] to the correct value."))
+ "connection type."))
->setBoolOptions(
array(
pht('Force HTTPS'),

File Metadata

Mime Type
text/plain
Expires
Sun, Jun 16, 10:13 AM (2 w, 1 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6288663
Default Alt Text
D8572.id20341.diff (1 KB)

Event Timeline