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

Header\ContentType: remove empty values from parsed header #6491

Closed
wants to merge 1 commit into from

Conversation

olivier1980
Copy link

Fix bug where a header string ending in ';' will add an empty element in array, triggering an 'Undefined offset' error later.

'Content-Type: multipart/alternative; boundary="Apple-Mail=_1B852F10-F9C6-463D-AADD-CD503A5428DD";'
);
$params = $contentTypeHeader->getParameters();
$this->assertEquals($params,array('boundary' => 'Apple-Mail=_1B852F10-F9C6-463D-AADD-CD503A5428DD'));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing space after ,

@Ocramius Ocramius added this to the 2.3.2 milestone Aug 4, 2014
Ocramius added a commit that referenced this pull request Aug 6, 2014
Ocramius added a commit that referenced this pull request Aug 6, 2014
Ocramius added a commit that referenced this pull request Aug 6, 2014
@Ocramius Ocramius self-assigned this Aug 6, 2014
@Ocramius Ocramius closed this in 6808ce7 Aug 6, 2014
@Ocramius
Copy link
Member

Ocramius commented Aug 6, 2014

@olivier1980 I fixed the CS issues on merge :-)

weierophinney pushed a commit to zendframework/zend-mail that referenced this pull request May 14, 2015
weierophinney pushed a commit to zendframework/zend-mail that referenced this pull request May 14, 2015
weierophinney pushed a commit to zendframework/zend-mail that referenced this pull request May 14, 2015
weierophinney pushed a commit to zendframework/zend-mail that referenced this pull request May 14, 2015
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants