Skip to content

Commit acec820

Browse files
committed
Bug #55403: Don't set $_SERVER['HTTPS'] on unsecure connection
1 parent 9e9d2a9 commit acec820

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

NEWS

+3
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@ PHP NEWS
1212
. Added Shift_JIS Emoji (pictograms) support. (rui)
1313
. Ill-formed UTF-8 check for security enhancements. (rui)
1414

15+
- Improved NSAPI SAPI: (Uwe Schindler)
16+
. Don't set $_SERVER['HTTPS'] on unsecure connection (bug #55403).
17+
1518
04 Aug 2011, PHP 5.4.0 Alpha 3
1619
- Added features:
1720
. Short array syntax, see UPGRADING guide for full details (rsky0711 at gmail

sapi/nsapi/nsapi.c

+3-1
Original file line numberDiff line numberDiff line change
@@ -723,7 +723,9 @@ static void sapi_nsapi_register_server_variables(zval *track_vars_array TSRMLS_D
723723
nsapi_free(value);
724724

725725
php_register_variable("SERVER_SOFTWARE", system_version(), track_vars_array TSRMLS_CC);
726-
php_register_variable("HTTPS", (security_active ? "ON" : "OFF"), track_vars_array TSRMLS_CC);
726+
if (security_active) {
727+
php_register_variable("HTTPS", "ON", track_vars_array TSRMLS_CC);
728+
}
727729
php_register_variable("GATEWAY_INTERFACE", "CGI/1.1", track_vars_array TSRMLS_CC);
728730

729731
/* DOCUMENT_ROOT */

0 commit comments

Comments
 (0)