Re: Policy.conf - multiple @

2012-11-29 Thread A . L . M . Buxey
On Thu, Nov 29, 2012 at 09:16:48AM +, Ben Brown wrote: Looking at the master branch in git, this file has been replaced by a directory with several files (proxy.d). The example of filter_username in proxy.d/filter uses /@.*@/ which as we've discovered on the list is fine for the purpose

Re: Policy.conf - multiple @

2012-11-29 Thread Ben Brown
On 28 Nov 2012, at 15:08, alan buxey a.l.m.bu...@lboro.ac.uk wrote: Hi, That's not quite the same, as it would match more than the original regex. OK the parenthesis aren't needed, however consider the following username: bob@@realm that would be a wrong and illegal

RE: Policy.conf - multiple @

2012-11-28 Thread Alan Buxey
hi, ..theres also an error in v2.2 policy.conf # # Realm begins with a dot # e.g. u...@.site.com # if (User-Name !~ /@\\./) { update reply { Reply-Message+ =

RE: Policy.conf - multiple @

2012-11-28 Thread Alan Buxey
hi, if(User-Name =~ /@(.+)?@/i ) { = Would this not be simpler just by using '/@.*@/' ? its good to get feedback. yes, theres no need for case insensitive for this oneand also, we dont care about feeding the results to a following processso no need

Re: Policy.conf - multiple @

2012-11-28 Thread Ben Brown
On Wed, Nov 28, 2012 at 01:09:44PM +, John Horne wrote: # if(User-Name =~ /@(.+)?@/i ) { = Would this not be simpler just by using '/@.*@/' ? That's not quite the same, as it would match more than the original regex. OK the parenthesis aren't needed,

Re: Policy.conf - multiple @

2012-11-28 Thread alan buxey
Hi, That's not quite the same, as it would match more than the original regex. OK the parenthesis aren't needed, however consider the following username: bob@@realm that would be a wrong and illegal entry...so needs to be rejected too. alan - List info/subscribe/unsubscribe? See

Re: Policy.conf - multiple @

2012-11-28 Thread Arran Cudbard-Bell
On 28 Nov 2012, at 15:08, alan buxey a.l.m.bu...@lboro.ac.uk wrote: Hi, That's not quite the same, as it would match more than the original regex. OK the parenthesis aren't needed, however consider the following username: bob@@realm that would be a wrong and illegal entry...so needs