Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 4:49 PM, Stefano Bagnara wrote: > Robert Burrell Donkin ha scritto: >> On Sun, Dec 14, 2008 at 4:12 PM, Stefano Bagnara wrote: >>> Robert Burrell Donkin ha scritto: On Sun, Dec 14, 2008 at 2:49 PM, Norman Maurer wrote: > Should I just go ahead an commit it ( and

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Stefano Bagnara
Robert Burrell Donkin ha scritto: > On Sun, Dec 14, 2008 at 4:12 PM, Stefano Bagnara wrote: >> Robert Burrell Donkin ha scritto: >>> On Sun, Dec 14, 2008 at 2:49 PM, Norman Maurer wrote: Should I just go ahead an commit it ( and open a jira issue) ? >>> if we follow stefano's design reasonin

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 4:12 PM, Stefano Bagnara wrote: > Robert Burrell Donkin ha scritto: >> On Sun, Dec 14, 2008 at 2:49 PM, Norman Maurer wrote: >>> Should I just go ahead an commit it ( and open a jira issue) ? >> >> if we follow stefano's design reasoning then really the VUT code needs >> t

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Stefano Bagnara
Robert Burrell Donkin ha scritto: > On Sun, Dec 14, 2008 at 2:49 PM, Norman Maurer wrote: >> Should I just go ahead an commit it ( and open a jira issue) ? > > if we follow stefano's design reasoning then really the VUT code needs > to be moved from LocalDelivery into a separate mailet I'm not f

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Stefano Bagnara
Robert Burrell Donkin ha scritto: > On Sun, Dec 14, 2008 at 2:49 PM, Norman Maurer wrote: >> Should I just go ahead an commit it ( and open a jira issue) ? > > if we follow stefano's design reasoning then really the VUT code needs > to be moved from LocalDelivery into a separate mailet A little

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Norman Maurer
I created an jira issue and attached some patch.. please review: https://issues.apache.org/jira/browse/JAMES-882 Cheers, Norman 2008/12/14 Norman Maurer : > I only moved it the UserRepositoryAliasForwarding mailet because of > some "shared" code. I think it would prolly be better to extract > sh

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Norman Maurer
I only moved it the UserRepositoryAliasForwarding mailet because of some "shared" code. I think it would prolly be better to extract shared code to an Abstract class and seperate the "vut store" and userrepository stuff. Cheers, Norman 2008/12/14 Robert Burrell Donkin : > On Sun, Dec 14, 2008 at

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 2:49 PM, Norman Maurer wrote: > Should I just go ahead an commit it ( and open a jira issue) ? if we follow stefano's design reasoning then really the VUT code needs to be moved from LocalDelivery into a separate mailet - robert --

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 2:49 PM, Norman Maurer wrote: > Should I just go ahead an commit it stefano? > ( and open a jira issue) ? i have a (failing) smoke test for VUT so opening a JIRA would be useful anyway - robert - To un

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Norman Maurer
Should I just go ahead an commit it ( and open a jira issue) ? Cheers, Norman 2008/12/14 Robert Burrell Donkin : > On Sun, Dec 14, 2008 at 2:15 PM, Norman Maurer wrote: >> I thought of something like this: > > > > definitely looks neater > > - robert > >

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 2:15 PM, Norman Maurer wrote: > I thought of something like this: definitely looks neater - robert - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: ser

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 2:18 PM, Stefano Bagnara wrote: > Robert Burrell Donkin ha scritto: >> On Sun, Dec 14, 2008 at 1:55 PM, Norman Maurer wrote: >>> Ah ok now I see what makes you wonder... thats right. isLocalEmail >>> works only after the vut mappings was done. Thats prolly a problem in >>>

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Stefano Bagnara
Robert Burrell Donkin ha scritto: > On Sun, Dec 14, 2008 at 1:55 PM, Norman Maurer wrote: >> Ah ok now I see what makes you wonder... thats right. isLocalEmail >> works only after the vut mappings was done. Thats prolly a problem in >> some cases.. > > it's the problem in my case :-) > > (the co

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Norman Maurer
I thought of something like this: Index: UsersRepositoryAliasingForwarding.java === --- UsersRepositoryAliasingForwarding.java (revision 726438) +++ UsersRepositoryAliasingForwarding.java (working copy) @@ -28,6 +28,7 @@ im

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 2:01 PM, Robert Burrell Donkin wrote: > On Sun, Dec 14, 2008 at 1:55 PM, Norman Maurer wrote: >> Ah ok now I see what makes you wonder... thats right. isLocalEmail >> works only after the vut mappings was done. Thats prolly a problem in >> some cases.. > > it's the problem

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 1:55 PM, Norman Maurer wrote: > Ah ok now I see what makes you wonder... thats right. isLocalEmail > works only after the vut mappings was done. Thats prolly a problem in > some cases.. it's the problem in my case :-) (the code's very messy since it's trying to ensure bac

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Norman Maurer
Ah ok now I see what makes you wonder... thats right. isLocalEmail works only after the vut mappings was done. Thats prolly a problem in some cases.. Cheers, Norman 2008/12/14 Robert Burrell Donkin : > On Sun, Dec 14, 2008 at 1:49 PM, Norman Maurer wrote: >> Hi Robert, >> >> I think it works li

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 1:49 PM, Norman Maurer wrote: > Hi Robert, > > I think it works like it should... if virtualhosting is not used it > just use the username to check if it is enable it use the complete > emailaddress. > Why you think it not works as aspected ? suppose supp...@example.org ma

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Norman Maurer
Hi Robert, I think it works like it should... if virtualhosting is not used it just use the username to check if it is enable it use the complete emailaddress. Why you think it not works as aspected ? Cheers, Norman 2008/12/14 Robert Burrell Donkin : > On Sun, Dec 14, 2008 at 1:38 PM, Norman Mau

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 1:38 PM, Norman Maurer wrote: > Ok I will try to fix it now.. Do you think we should create an extra > mailet or better include it in localDelivery ? > I think I would prolly add it to localDelivery UsersRepositoryAliasingForwarding is used by LocalDelivery and appears to

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Norman Maurer
Ok I will try to fix it now.. Do you think we should create an extra mailet or better include it in localDelivery ? I think I would prolly add it to localDelivery Cheers, Norman 2008/12/14 Robert Burrell Donkin : > On Sun, Dec 14, 2008 at 12:37 PM, Norman Maurer wrote: >> Sorry Robert I'm a bit

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 12:37 PM, Norman Maurer wrote: > Sorry Robert I'm a bit confused.. So after adding that config stuff it > should just be a Matter of coding the right Mailet to support the VUT > stuff ? i think that it should just work once the config is right - robert --

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Norman Maurer
Sorry Robert I'm a bit confused.. So after adding that config stuff it should just be a Matter of coding the right Mailet to support the VUT stuff ? Cheers, Norman 2008/12/14 Robert Burrell Donkin : > On Sun, Dec 14, 2008 at 12:24 PM, Norman Maurer wrote: >> Hi Robert, >> >> that's what I have d

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 12:24 PM, Norman Maurer wrote: > Hi Robert, > > that's what I have done ;-) i needed to add that to my test - robert - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional comma

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Norman Maurer
Hi Robert, that's what I have done ;-) Cheers, Norman 2008/12/14 Robert Burrell Donkin : > On Sun, Dec 14, 2008 at 11:27 AM, Norman Maurer wrote: >> Hi Robert, >> >> thx for digging a bit deeper. I think we just need a Mailet which do >> the "rewrite" of the recipients and add it before the Loc

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 11:27 AM, Norman Maurer wrote: > Hi Robert, > > thx for digging a bit deeper. I think we just need a Mailet which do > the "rewrite" of the recipients and add it before the LocalDelivery > mailet. > > Anythink I miss ? the same thing i missed ;-) the code's there but the

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Norman Maurer
Hi Robert, thx for digging a bit deeper. I think we just need a Mailet which do the "rewrite" of the recipients and add it before the LocalDelivery mailet. Anythink I miss ? Cheers, Norman 2008/12/14 Robert Burrell Donkin : > On Sun, Dec 14, 2008 at 10:02 AM, Robert Burrell Donkin > wrote: >>

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 10:02 AM, Robert Burrell Donkin wrote: > On Sun, Dec 14, 2008 at 9:44 AM, Robert Burrell Donkin > wrote: >> On Sat, Dec 13, 2008 at 8:42 PM, Robert Burrell Donkin >> wrote: >>> On Sat, Dec 13, 2008 at 10:57 AM, Robert Burrell Donkin >>> wrote: On Sat, Dec 13, 2008 a

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sun, Dec 14, 2008 at 9:44 AM, Robert Burrell Donkin wrote: > On Sat, Dec 13, 2008 at 8:42 PM, Robert Burrell Donkin > wrote: >> On Sat, Dec 13, 2008 at 10:57 AM, Robert Burrell Donkin >> wrote: >>> On Sat, Dec 13, 2008 at 10:24 AM, Norman Maurer wrote: Hi Robert, comments inside...

Re: VirtualUserTable and VirtualHosting

2008-12-14 Thread Robert Burrell Donkin
On Sat, Dec 13, 2008 at 8:42 PM, Robert Burrell Donkin wrote: > On Sat, Dec 13, 2008 at 10:57 AM, Robert Burrell Donkin > wrote: >> On Sat, Dec 13, 2008 at 10:24 AM, Norman Maurer wrote: >>> Hi Robert, >>> comments inside... >>> 2008/12/13 Robert Burrell Donkin : On Fri, Dec 12, 2008 at 4:5

Re: VirtualUserTable and VirtualHosting

2008-12-13 Thread Robert Burrell Donkin
On Sat, Dec 13, 2008 at 10:57 AM, Robert Burrell Donkin wrote: > On Sat, Dec 13, 2008 at 10:24 AM, Norman Maurer wrote: >> Hi Robert, >> comments inside... >> 2008/12/13 Robert Burrell Donkin : >>> On Fri, Dec 12, 2008 at 4:56 PM, Norman Maurer wrote: >> I think before the defunc it get calle

Re: VirtualUserTable and VirtualHosting

2008-12-13 Thread Robert Burrell Donkin
On Sat, Dec 13, 2008 at 10:24 AM, Norman Maurer wrote: > Hi Robert, > comments inside... > > > 2008/12/13 Robert Burrell Donkin : >> On Fri, Dec 12, 2008 at 4:56 PM, Norman Maurer wrote: >>> Hi Guys, >>> >>> after some long time I had some time to look at james trunk source >>> again. >> >> cool

Re: VirtualUserTable and VirtualHosting

2008-12-13 Thread Norman Maurer
Hi Robert, comments inside... 2008/12/13 Robert Burrell Donkin : > On Fri, Dec 12, 2008 at 4:56 PM, Norman Maurer wrote: >> Hi Guys, >> >> after some long time I had some time to look at james trunk source >> again. > > cool :-) > >> I tried to deploy a james install and use VirtualUserTable tod

Re: VirtualUserTable and VirtualHosting

2008-12-13 Thread Robert Burrell Donkin
On Fri, Dec 12, 2008 at 4:56 PM, Norman Maurer wrote: > Hi Guys, > > after some long time I had some time to look at james trunk source > again. cool :-) > I tried to deploy a james install and use VirtualUserTable todo > all the forwarding stuff. But It seems it get never caleld todo the > mapp