[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#6647) Overlay sssvlv is always loaded



--=-sj86jeXTHEFBrrKIl07g
Content-Type: text/plain; charset=utf-8
Content-ID: <1286271588.1952.3.camel@Nokia-N900-51-1>
Content-Transfer-Encoding: 8bit

----- Message d'origine -----
> 
> > Please try this (nearly blind) patch
> > 
> > <ftp://ftp.openldap.org/incoming/pierangelo-masarati-2010-09-14-sssvlv.1.patch>
> > 
> > and report through the ITS.   Thanks, p.
> 
> I note that I have messed things up a bit in cleanup; however, this
> should not affect the testing of the functionality, except if you delete
> the overlay from the configuration using back-config.   If the patch works
> fine, I'll fix that part.
> 


Hi,

sorry for this late reply. I tried the patch on 2.4.23 source code, the second hunk was not working, so I deleted it to apply the patch. I am not using cn=config anyway.

After recompiling with your patch and --enable-overlays, all works fine: a search with sss extention returns success (and fails if the control is critical).

So I think you can commit the code!

Thanks,

Clément.
--=-sj86jeXTHEFBrrKIl07g
Content-Type: text/html; charset=utf-8
Content-ID: <1286271588.1952.5.camel@Nokia-N900-51-1>
Content-Transfer-Encoding: 8bit

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd";>
<html><head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
    <meta name="generator" content="Osso Notes">
    <title></title></head>
<body>
<p>----- Message d'origine -----
<br>&gt; 
<br>&gt; &gt; Please try this (nearly blind) patch
<br>&gt; &gt; 
<br>&gt; &gt; &lt;<a href="ftp://ftp.openldap.org/incoming/pierangelo-masarati-2010-09-14-sssvlv.1.patch&gt";>ftp://ftp.openldap.org/incoming/pierangelo-masarati-2010-09-14-sssvlv.1.patch&gt</a>;
<br>&gt; &gt; 
<br>&gt; &gt; and report through the ITS.&nbsp; &#32;Thanks, p.
<br>&gt; 
<br>&gt; I note that I have messed things up a bit in cleanup; however, this
<br>&gt; should not affect the testing of the functionality, except if you delete
<br>&gt; the overlay from the configuration using back-config.&nbsp; &#32;If the patch works
<br>&gt; fine, I'll fix that part.
<br>&gt; 
<br>
<br>
<br>Hi,
<br>
<br>sorry for this late reply. I tried the patch on 2.4.23 source code, the second hunk was not working, so I deleted it to apply the patch. I am not using cn=config anyway.
<br>
<br>After recompiling with your patch and --enable-overlays, all works fine: a search with sss extention returns success (and fails if the control is critical).
<br>
<br>So I think you can commit the code!
<br>
<br>Thanks,
<br>
<br>Clément.</p>
</body>
</html>

--=-sj86jeXTHEFBrrKIl07g--