Skip to content

Fix for issue #492: Support UTF8 characters in attributes values for … #493

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

naorbar
Copy link

@naorbar naorbar commented Sep 12, 2018

…LDIF's DefaultAttributeValidationPolicy in order to comply with RFC2849

…tes values for LDIF's DefaultAttributeValidationPolicy in order to comply with RFC2849
@pivotal-issuemaster
Copy link

@naorbar Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@naorbar Thank you for signing the Contributor License Agreement!

@rwinch
Copy link
Member

rwinch commented Sep 12, 2018

@naorbar Thanks for the PR! Unfortunately one of the tests is now failing in Travis


org.springframework.ldap.ldif.DefaultAttributeValidationPolicyTest > parseAttribute[28] FAILED
    java.lang.AssertionError at DefaultAttributeValidationPolicyTest.java:176

Can you push a fix to the code?

@rwinch rwinch self-requested a review September 12, 2018 16:29
Copy link
Member

@rwinch rwinch left a comment

Choose a reason for hiding this comment

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

Please fix the build before we can merge this.

@naorbar
Copy link
Author

naorbar commented Sep 13, 2018

Rob,
I changed the encoding on my IDE to UTF8 to comply with Travis's encoding, and saved again the source file.
The build passed.
Thanks,
Naor

@rwinch rwinch merged commit f09881c into spring-projects:master Sep 13, 2018
@rwinch
Copy link
Member

rwinch commented Sep 13, 2018

@naorbar Thanks for fixing the build! This is now merged into master

@rwinch rwinch added this to the 2.3.3 milestone Sep 13, 2018
@naorbar
Copy link
Author

naorbar commented Sep 13, 2018

With pleasure. Let me know if I can help in the future...

@naorbar naorbar deleted the naorbar_fix_for_issue#492_support_UTF8_characters_in_LDIFParser branch September 13, 2018 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants