mirror of https://github.com/OpenIdentityPlatform/OpenDJ.git

Gaetan Boismal
16.41.2016 ff0653af9a57afaf256008ce4d39e7d6fc899446
refs
author Gaetan Boismal <gaetan.boismal@forgerock.com>
Friday, September 16, 2016 11:41 +0200
committer Gaetan Boismal <gaetan.boismal@forgerock.com>
Monday, October 31, 2016 15:00 +0100
commitff0653af9a57afaf256008ce4d39e7d6fc899446
tree d7043e784b76470aed6527f224c08a5b44204646 tree | zip | gz
parent 88496c3a54b4c6e969cb0dce5cf67e5da6846740 view | diff
OPENDJ-2937 Fix SDK SASL bind requests implementations

* For multi-stage (CRAM-MD5, DIGEST-MD5, GSSAPI and EXTERNAL) mechanisms
* Rely on the result of saslClient.evaluateChallenge(...) instead of saslClient.isComplete() to know if there is a need to send additinal data to the server (i.e if we are in a multi-stage SASL bind)
* Factorize code in parent class to prevent duplication.

* As SASL Plain is not multi-stage we do not need to override evaluateResult() in the bind request implementations, return true is enought.

* Also adapts test ldap server to not systematically attach a challenge to a bind result.
8 files modified
144 ■■■■■ changed files
opendj-core/src/main/java/org/forgerock/opendj/ldap/requests/CRAMMD5SASLBindRequestImpl.java 19 ●●●● diff | view | raw | blame | history
opendj-core/src/main/java/org/forgerock/opendj/ldap/requests/DigestMD5SASLBindRequestImpl.java 18 ●●●● diff | view | raw | blame | history
opendj-core/src/main/java/org/forgerock/opendj/ldap/requests/ExternalSASLBindRequestImpl.java 20 ●●●● diff | view | raw | blame | history
opendj-core/src/main/java/org/forgerock/opendj/ldap/requests/GSSAPISASLBindRequestImpl.java 20 ●●●● diff | view | raw | blame | history
opendj-core/src/main/java/org/forgerock/opendj/ldap/requests/PlainSASLBindRequestImpl.java 8 ●●●● diff | view | raw | blame | history
opendj-core/src/main/java/org/forgerock/opendj/ldap/requests/SASLBindClientImpl.java 46 ●●●●● diff | view | raw | blame | history
opendj-core/src/main/resources/com/forgerock/opendj/ldap/core.properties 3 ●●●● diff | view | raw | blame | history
opendj-core/src/test/java/org/forgerock/opendj/ldap/LDAPServer.java 10 ●●●● diff | view | raw | blame | history