[wp-trac] Re: [WordPress Trac] #8641: Improvements to is_ssl()

[WordPress Trac]

#8641: Improvements to is_ssl()
-------------------------------------------+--------------------------------
 Reporter:  johnbillion                    |        Owner:  johnbillion
     Type:  enhancement                    |       Status:  new        
 Priority:  normal                         |    Milestone:  2.8        
Component:  Security                       |      Version:  2.7        
 Severity:  normal                         |   Resolution:             
 Keywords:  is_ssl, ssl, https, has-patch  |  
-------------------------------------------+--------------------------------
Changes (by johnbillion):

  * keywords:  is_ssl, ssl, https => is_ssl, ssl, https, has-patch

Comment:

 Patch.

 `is_ssl()` now returns true if `$_SERVER['HTTPS'] == '1'`.

 If the `$_SERVER['HTTPS']` environment variable is not set, it then checks
 the port. If `$_SERVER['SERVER_PORT'] == '443'` then the function returns
 true.

 Several other CMS' detect SSL like this, including Trac and CubeCart.

 Tested and confirmed as working on a server where the HTTPS environment
 variable was not set but the server was running on SSL (port 443).

 Left the milestone as 2.8 but could we get this into 2.7.1?

-- 
Ticket URL: <http://trac.wordpress.org/ticket/8641#comment:1>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software