You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2017/12/24 22:55:49 UTC
[directory-ldap-api] branch shared-value updated: Added a (first)
version of SASL CRAM-MD5 and DIGEST-MD5 dedicated bind function. Note that
the DIGESTMD5 does not have enough parameters.
This is an automated email from the ASF dual-hosted git repository.
elecharny pushed a commit to branch shared-value
in repository https://gitbox.apache.org/repos/asf/directory-ldap-api.git
The following commit(s) were added to refs/heads/shared-value by this push:
new e9445b1 Added a (first) version of SASL CRAM-MD5 and DIGEST-MD5 dedicated bind function. Note that the DIGESTMD5 does not have enough parameters.
e9445b1 is described below
commit e9445b1466ea171e6e4c24a53db09be36705d64e
Author: Emmanuel Lécharny <el...@symas.com>
AuthorDate: Sun Dec 24 23:55:36 2017 +0100
Added a (first) version of SASL CRAM-MD5 and DIGEST-MD5 dedicated bind
function. Note that the DIGESTMD5 does not have enough parameters.
---
.../ldap/client/api/LdapNetworkConnection.java | 36 ++++++++++++++++++++++
1 file changed, 36 insertions(+)
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
index e43a8f2..90f99fc 100644
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
@@ -1522,6 +1522,42 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
/**
+ * Bind to the server using the SASL CRAM-MD5 mechanism.
+ *
+ * @param userName The user name
+ * @param credentials The user credentials
+ * @return A LdapResponse containing the result
+ * @throws LdapException if some error occurred
+ */
+ public BindResponse bindSaslCramMd5( String userName, String credentials ) throws LdapException
+ {
+ SaslCramMd5Request request = new SaslCramMd5Request();
+ request.setUsername( userName );
+ request.setCredentials( "secret" );
+
+ return bind( request );
+ }
+
+
+ /**
+ * Bind to the server using the SASL DIGEST-MD5 mechanism.
+ *
+ * @param userName The user name
+ * @param credentials The user credentials
+ * @return A LdapResponse containing the result
+ * @throws LdapException if some error occurred
+ */
+ public BindResponse bindSaslDigestMd5( String userName, String credentials ) throws LdapException
+ {
+ SaslDigestMd5Request request = new SaslDigestMd5Request();
+ request.setUsername( userName );
+ request.setCredentials( "secret" );
+
+ return bind( request );
+ }
+
+
+ /**
* Bind to the server using a CramMd5Request object.
*
* @param request The CramMd5Request POJO containing all the needed parameters
--
To stop receiving notification emails like this one, please contact
['"commits@directory.apache.org" <co...@directory.apache.org>'].
Re: [directory-ldap-api] branch shared-value updated: Added a (first)
version of SASL CRAM-MD5 and DIGEST-MD5 dedicated bind function. Note that
the DIGESTMD5 does not have enough parameters.
Posted by Emmanuel Lécharny <el...@gmail.com>.
Ah, my bad, with a fresh git clone, the master is correct !
Ok, fine, I'll apply my last commit onto this branch.
Thanks and sorry for yhe noise :-)
Le 25/12/2017 à 13:54, Emmanuel Lécharny a écrit :
>
>
> Le 25/12/2017 à 07:23, Stefan Seelmann a écrit :
>> On 12/25/2017 12:57 AM, Emmanuel Lécharny wrote:
>>>
>>>
>>> Le 25/12/2017 à 00:39, Stefan Seelmann a écrit :
>>>> On 12/24/2017 11:55 PM, elecharny@apache.org wrote:
>>>>> This is an automated email from the ASF dual-hosted git repository.
>>>>>
>>>>> elecharny pushed a commit to branch shared-value
>>>>> in repository https://gitbox.apache.org/repos/asf/directory-ldap-api.git
>>>>
>>>> Hi Emmanuel,
>>>>
>>>> your last 2 commits went to the "value" branch, I guess they should go
>>>> to master?
>>>
>>> Well, I checked teh master, and it lacks some of the 'value' branch
>>> updates, so I guess the merge from the value branch into the master
>>> branch hasn't worked as expected :/
>>
>> Oh :-(
>>
>> Do you have an example of a missing change?
>
> Typically, the Value class is missing in trunk.
>
>
>>
>> Last change in value branch was 16b5e0edaec53bc8bbc2eb6598d372f7ef89b046
>> which is in master.
>>
>>> So I applied my changes into the value branch first, and I'm now trying
>>> to merge it back to master (which is not easy, with 300+ conflicts ;-).
>>
>> I dont' understand why conflicts happen. The two changes you committed
>> can be merged to master without problem:
>> https://github.com/apache/directory-ldap-api/compare/shared-value
>
>
> I have to redo that from scratch (ie, git clone the repo, and see what I
> get in master and shared-value branches).
>
> I'll tell you tonite :-)
>
--
Emmanuel Lecharny
Symas.com
directory.apache.org
Re: [directory-ldap-api] branch shared-value updated: Added a (first)
version of SASL CRAM-MD5 and DIGEST-MD5 dedicated bind function. Note that
the DIGESTMD5 does not have enough parameters.
Posted by Emmanuel Lécharny <el...@gmail.com>.
Le 25/12/2017 à 13:54, Emmanuel Lécharny a écrit :
>
>
> Le 25/12/2017 à 07:23, Stefan Seelmann a écrit :
>> On 12/25/2017 12:57 AM, Emmanuel Lécharny wrote:
>>>
>>>
>>> Le 25/12/2017 à 00:39, Stefan Seelmann a écrit :
>>>> On 12/24/2017 11:55 PM, elecharny@apache.org wrote:
>>>>> This is an automated email from the ASF dual-hosted git repository.
>>>>>
>>>>> elecharny pushed a commit to branch shared-value
>>>>> in repository https://gitbox.apache.org/repos/asf/directory-ldap-api.git
>>>>
>>>> Hi Emmanuel,
>>>>
>>>> your last 2 commits went to the "value" branch, I guess they should go
>>>> to master?
>>>
>>> Well, I checked teh master, and it lacks some of the 'value' branch
>>> updates, so I guess the merge from the value branch into the master
>>> branch hasn't worked as expected :/
>>
>> Oh :-(
>>
>> Do you have an example of a missing change?
>
> Typically, the Value class is missing in trunk.
>
>
>>
>> Last change in value branch was 16b5e0edaec53bc8bbc2eb6598d372f7ef89b046
>> which is in master.
>>
>>> So I applied my changes into the value branch first, and I'm now trying
>>> to merge it back to master (which is not easy, with 300+ conflicts ;-).
>>
>> I dont' understand why conflicts happen. The two changes you committed
>> can be merged to master without problem:
>> https://github.com/apache/directory-ldap-api/compare/shared-value
>
>
> I have to redo that from scratch (ie, git clone the repo, and see what I
> get in master and shared-value branches).
>
> I'll tell you tonite :-)
ok, I think teh prolem has been solved. I have pushed the modifications
to master successfully.
Note that while checking the 300+ conflicts, I found 2 bugs :-)
--
Emmanuel Lecharny
Symas.com
directory.apache.org
Re: [directory-ldap-api] branch shared-value updated: Added a (first)
version of SASL CRAM-MD5 and DIGEST-MD5 dedicated bind function. Note that
the DIGESTMD5 does not have enough parameters.
Posted by Emmanuel Lécharny <el...@gmail.com>.
Le 25/12/2017 à 07:23, Stefan Seelmann a écrit :
> On 12/25/2017 12:57 AM, Emmanuel Lécharny wrote:
>>
>>
>> Le 25/12/2017 à 00:39, Stefan Seelmann a écrit :
>>> On 12/24/2017 11:55 PM, elecharny@apache.org wrote:
>>>> This is an automated email from the ASF dual-hosted git repository.
>>>>
>>>> elecharny pushed a commit to branch shared-value
>>>> in repository https://gitbox.apache.org/repos/asf/directory-ldap-api.git
>>>
>>> Hi Emmanuel,
>>>
>>> your last 2 commits went to the "value" branch, I guess they should go
>>> to master?
>>
>> Well, I checked teh master, and it lacks some of the 'value' branch
>> updates, so I guess the merge from the value branch into the master
>> branch hasn't worked as expected :/
>
> Oh :-(
>
> Do you have an example of a missing change?
Typically, the Value class is missing in trunk.
>
> Last change in value branch was 16b5e0edaec53bc8bbc2eb6598d372f7ef89b046
> which is in master.
>
>> So I applied my changes into the value branch first, and I'm now trying
>> to merge it back to master (which is not easy, with 300+ conflicts ;-).
>
> I dont' understand why conflicts happen. The two changes you committed
> can be merged to master without problem:
> https://github.com/apache/directory-ldap-api/compare/shared-value
I have to redo that from scratch (ie, git clone the repo, and see what I
get in master and shared-value branches).
I'll tell you tonite :-)
--
Emmanuel Lecharny
Symas.com
directory.apache.org
Re: [directory-ldap-api] branch shared-value updated: Added a (first)
version of SASL CRAM-MD5 and DIGEST-MD5 dedicated bind function. Note that
the DIGESTMD5 does not have enough parameters.
Posted by Stefan Seelmann <ma...@stefan-seelmann.de>.
On 12/25/2017 12:57 AM, Emmanuel Lécharny wrote:
>
>
> Le 25/12/2017 à 00:39, Stefan Seelmann a écrit :
>> On 12/24/2017 11:55 PM, elecharny@apache.org wrote:
>>> This is an automated email from the ASF dual-hosted git repository.
>>>
>>> elecharny pushed a commit to branch shared-value
>>> in repository https://gitbox.apache.org/repos/asf/directory-ldap-api.git
>>
>> Hi Emmanuel,
>>
>> your last 2 commits went to the "value" branch, I guess they should go
>> to master?
>
> Well, I checked teh master, and it lacks some of the 'value' branch
> updates, so I guess the merge from the value branch into the master
> branch hasn't worked as expected :/
Oh :-(
Do you have an example of a missing change?
Last change in value branch was 16b5e0edaec53bc8bbc2eb6598d372f7ef89b046
which is in master.
> So I applied my changes into the value branch first, and I'm now trying
> to merge it back to master (which is not easy, with 300+ conflicts ;-).
I dont' understand why conflicts happen. The two changes you committed
can be merged to master without problem:
https://github.com/apache/directory-ldap-api/compare/shared-value
> Most of the conflicts are trivial, some are a bit more complex, but I
> think it's going to be completed in the next couple fo days.
>
> And, yes, we should delete the 'value' branch once done.
>
Re: [directory-ldap-api] branch shared-value updated: Added a (first)
version of SASL CRAM-MD5 and DIGEST-MD5 dedicated bind function. Note that
the DIGESTMD5 does not have enough parameters.
Posted by Emmanuel Lécharny <el...@gmail.com>.
Le 25/12/2017 à 00:39, Stefan Seelmann a écrit :
> On 12/24/2017 11:55 PM, elecharny@apache.org wrote:
>> This is an automated email from the ASF dual-hosted git repository.
>>
>> elecharny pushed a commit to branch shared-value
>> in repository https://gitbox.apache.org/repos/asf/directory-ldap-api.git
>
> Hi Emmanuel,
>
> your last 2 commits went to the "value" branch, I guess they should go
> to master?
Well, I checked teh master, and it lacks some of the 'value' branch
updates, so I guess the merge from the value branch into the master
branch hasn't worked as expected :/
So I applied my changes into the value branch first, and I'm now trying
to merge it back to master (which is not easy, with 300+ conflicts ;-).
Most of the conflicts are trivial, some are a bit more complex, but I
think it's going to be completed in the next couple fo days.
And, yes, we should delete the 'value' branch once done.
--
Emmanuel Lecharny
Symas.com
directory.apache.org
Re: [directory-ldap-api] branch shared-value updated: Added a (first)
version of SASL CRAM-MD5 and DIGEST-MD5 dedicated bind function. Note that
the DIGESTMD5 does not have enough parameters.
Posted by Stefan Seelmann <ma...@stefan-seelmann.de>.
On 12/24/2017 11:55 PM, elecharny@apache.org wrote:
> This is an automated email from the ASF dual-hosted git repository.
>
> elecharny pushed a commit to branch shared-value
> in repository https://gitbox.apache.org/repos/asf/directory-ldap-api.git
Hi Emmanuel,
your last 2 commits went to the "value" branch, I guess they should go
to master?
I suggest (after you merged) we delete the value branches and the still
existing "trunk" branch.
Kind Regards,
Stefan