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

Ldap Ldif Decoder bug fix #4361

Merged
merged 3 commits into from
May 1, 2013
Merged

Conversation

zefrog
Copy link

@zefrog zefrog commented Apr 30, 2013

This PR fixes the two issues described in #3895:

  • decoding an attribute containing a colon (which is valid for, e.g. URI)
  • decoding a base64 attribute that is followed by a multi line comment -> a flag was added to track the current status of the parser.

Tests have been written for the described issues. Fails before, passes after.

Regards.

@ghost ghost assigned weierophinney May 1, 2013
weierophinney added a commit that referenced this pull request May 1, 2013
weierophinney added a commit that referenced this pull request May 1, 2013
@weierophinney weierophinney merged commit d6bc445 into zendframework:develop May 1, 2013
weierophinney added a commit to zendframework/zend-ldap that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-ldap 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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants