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

explode should be made only by colon (:) and not colon+space (: ) #5250

Closed
wants to merge 1 commit into from
Closed

explode should be made only by colon (:) and not colon+space (: ) #5250

wants to merge 1 commit into from

Conversation

Repkit
Copy link

@Repkit Repkit commented Oct 9, 2013

I think that was a typo :)
Added protected property $value

@Repkit
explode should be made only by colon (:) and not colon+space (: )
55c7555 
I think that was a typo :)
Added protected property $value
@samsonasik
Copy link
Contributor

@Repkit I think you should close #5249 and #5248 , they combined at this PR ?

@Maks3w
Copy link
Member

Maks3w commented Oct 20, 2013

I don't know how I miss this header when I wrote #5299. @Repkit Can you update this PR and apply the call to GenericHeader::splitHeaderLine($headerLine);

@ghost ghost assigned Maks3w Oct 20, 2013
@Maks3w Maks3w mentioned this pull request Oct 21, 2013
Closed
gianarb pushed a commit to zendframework/zend-http that referenced this pull request May 15, 2015
Merge branch 'Maks3w-hotfix/http-explode'
c2783a1 
Close zendframework/zendframework#5311
Close zendframework/zendframework#5250
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@Maks3w Maks3w

Labels
Http
Projects
None yet
Milestone
2.2.5
Development

Successfully merging this pull request may close these issues.
3 participants
@Repkit @samsonasik @Maks3w