Re: [389-devel] Please Review: Managed Entries Plug-in

[Nathan Kinder]

On 03/30/2010 03:44 PM, Rich Megginson wrote:
>  Nathan Kinder wrote:
>     
> > New patch attached.
> >      
>  Nice!
>      
Thanks for the review!
>  mep.c:491 - value is not freed if it is not a valid filter - also
> would
>  be nice to have the error message print value to show what the bogus
>  filter is
>      
I'll address this.
>  Do we plan on supporting binary attrs?  e.g.
>  mepstaticattr: blobattr:: base64value
>  not sure if we care about this for the first iteration
>      
This could be nice for static attributes.  I'm not really sure if it 
makes sense for mapped attributes or not.  I'll look into how easily 
this could be done, but it is something we could add later as well.

-NGK
>     
> > -NGK
> >
> > On 03/30/2010 01:24 PM, Nathan Kinder wrote:
> >      
> >> Please hold off on the review.  I noticed one other thing that I need
> >> to address.  A new patch will be sent out shortly.
> >>
> >> -NGK
> >>
> >> On 03/30/2010 01:18 PM, Nathan Kinder wrote:
> >>        
> >>>
> >>> --
> >>> 389-devel mailing list
> >>> 389-devel(a)lists.fedoraproject.org
> >>> https://admin.fedoraproject.org/mailman/listinfo/389-devel
> >>>          
> >>
> >> --
> >> 389-devel mailing list
> >> 389-devel(a)lists.fedoraproject.org
> >> https://admin.fedoraproject.org/mailman/listinfo/389-devel
> >>        
> > ------------------------------------------------------------------------
> >
> > --
> > 389-devel mailing list
> > 389-devel(a)lists.fedoraproject.org
> > https://admin.fedoraproject.org/mailman/listinfo/389-devel
> >      
>  --
>  389-devel mailing list
>  389-devel(a)lists.fedoraproject.org
>  https://admin.fedoraproject.org/mailman/listinfo/389-devel
>