Hi Eric, comments inside..
2011/1/25 Eric Charles <[email protected]>: > Hi, > > I recently opened a JIRA-1126 (Disallow the creation of alias from > non-existing username). > Before opening other ones related to virtual user, I would like to discuss > how this aspect should be managed. > > 1.- The management API is addMapping(String user, String domain, String > mapping). I always need to look at the code to find the goal of the params. > I first tough changing it to addMapping(fromAccount, toAccount) which is > more easy to undersand. For virtualhosting enabled, the account is the > email, so this would mean all mails for fromEmail will go to the toEmail. > For virtualhosting disabled, this may be more problematic, because we may > want to say "user1" at "domain1" goes to "user2". Or maybe we could have two > different methods (one for virtualhosting enabled, one for virtualhosting > disabled)... > I agree with you.. the virtualusertable api can be confusing. I'm open to every change which will make it more "self-documented". > 2.- Since we enabled the SMTP ValidRcptHandler, we need to really create the > user, before using it as an virtual user. This was not the case before. The > need to previously create a user does not worry me, but I find strange that, > depending on smtp configuration, we would or not need to create a user to > define a virtualuser. I think the need of having the existing user before add the mapping is a good think. We also would need to check if there is a mapping when deleting a user and remove it too. > > 3.- JIRA-1126 Disallow the creation of alias from non-existing username: I > think this is needed so we don't arrive to situation where mails are no more > delivered in case a bad usage of the API (reversing the method params). > +1 > tks, > - eric Bye, Norman --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
