Re: [squid-dev] [PATCH] Digest Auth support for LDAP HA1 attribute without realm

2017-06-22 Thread FUSTE Emmanuel
Le 09/02/2017 à 17:43, Amos Jeffries a écrit : > On 7/02/2017 10:33 p.m., FUSTE Emmanuel wrote: >> New patch attached. >> - Empty realm parameter is treated as no realm parameter supplied. >> - realm parameter is mandatory if empty delimiter is supplied >> - -f vs -F typo corrected in the help

Re: [squid-dev] [PATCH] Digest Auth support for LDAP HA1 attribute without realm

2017-02-09 Thread Amos Jeffries
On 7/02/2017 10:33 p.m., FUSTE Emmanuel wrote: > New patch attached. > - Empty realm parameter is treated as no realm parameter supplied. > - realm parameter is mandatory if empty delimiter is supplied > - -f vs -F typo corrected in the help text > Tested in real conditions. > Thanks. I have

Re: [squid-dev] [PATCH] Digest Auth support for LDAP HA1 attribute without realm

2017-02-07 Thread FUSTE Emmanuel
Hello, Le 30/01/2017 à 16:04, FUSTE Emmanuel a écrit : > Le 29/01/2017 à 14:33, Amos Jeffries a écrit : >> On 21/01/2017 2:05 a.m., FUSTE Emmanuel wrote: >>> Hello, >>> >>> We have to support many historic digest auth implementation for which >>> the realm is not included in the digest password

Re: [squid-dev] [PATCH] Digest Auth support for LDAP HA1 attribute without realm

2017-01-30 Thread FUSTE Emmanuel
Le 29/01/2017 à 14:33, Amos Jeffries a écrit : > On 21/01/2017 2:05 a.m., FUSTE Emmanuel wrote: >> Hello, >> >> We have to support many historic digest auth implementation for which >> the realm is not included in the digest password attribute: >> The password is effectively stored as "HA1"

Re: [squid-dev] [PATCH] Digest Auth support for LDAP HA1 attribute without realm

2017-01-29 Thread Amos Jeffries
On 21/01/2017 2:05 a.m., FUSTE Emmanuel wrote: > Hello, > > We have to support many historic digest auth implementation for which > the realm is not included in the digest password attribute: > The password is effectively stored as "HA1" instead of "REALM:HA1". > I would like to kill our own

Re: [squid-dev] [PATCH] Digest Auth support for LDAP HA1 attribute without realm

2017-01-23 Thread FUSTE Emmanuel
Le 20/01/2017 à 18:40, Alex Rousskov a écrit : > On 01/20/2017 09:41 AM, FUSTE Emmanuel wrote: >> Le 20/01/2017 à 16:36, Alex Rousskov a écrit : >>> 1. Rename notnulldelim to a positive name like haveDelimiter or, better, >>> remove that variable completely in favor of a direct *delimiter test.

Re: [squid-dev] [PATCH] Digest Auth support for LDAP HA1 attribute without realm

2017-01-20 Thread FUSTE Emmanuel
Le 20/01/2017 à 16:36, Alex Rousskov a écrit : > On 01/20/2017 06:05 AM, FUSTE Emmanuel wrote: > >> We have to support many historic digest auth implementation for which >> the realm is not included in the digest password attribute: >> The password is effectively stored as "HA1" instead of

Re: [squid-dev] [PATCH] Digest Auth support for LDAP HA1 attribute without realm

2017-01-20 Thread Alex Rousskov
On 01/20/2017 06:05 AM, FUSTE Emmanuel wrote: > We have to support many historic digest auth implementation for which > the realm is not included in the digest password attribute: > The password is effectively stored as "HA1" instead of "REALM:HA1". > I would like to kill our own homegrown