Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Fix for https cases for Apache on IBM i #6896

Closed
wants to merge 1 commit into from

Conversation

amitdar
Copy link
Contributor

@amitdar amitdar commented Nov 19, 2014

On Apache on IBM i the $_SERVER['HTTPS'] is set to OFF instead of off (lowercase).
I've added the strtolower to prevent cases like that.

On Apache on IBM i the `$_SERVER['HTTPS']` is set to `OFF` instead of `off` (lowercase).
I've added the strtolower to prevent cases like that.
@Ocramius
Copy link
Member

@amitdar please add a test case to avoid regressions

@Ocramius Ocramius added this to the 2.3.4 milestone Nov 22, 2014
@Ocramius Ocramius self-assigned this Nov 22, 2014
Ocramius added a commit that referenced this pull request Dec 16, 2014
Ocramius added a commit that referenced this pull request Dec 16, 2014
Ocramius added a commit that referenced this pull request Dec 16, 2014
@Ocramius Ocramius closed this in 0314c58 Dec 16, 2014
@Ocramius
Copy link
Member

Added tests and manually merged. Thanks for spotting the problem, @amitdar :-)

master: 0314c58
develop: 7440246

gianarb pushed a commit to zendframework/zend-http that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-http that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-http that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-http that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants