On 01/14/2013 11:54 PM, Slide wrote:
Now that the Mailer is outside the core, I think that the User class should have an email field (or a list of emails or something). Mail address resolution should probably be moved back into the core as well.
Perhaps, assuming you can figure out the compatibility issues. The hack in HudsonPrivateSecurityRealm.createAccount is pretty ugly, though this could be addressed directly by adding a simple extension point in core for initializing the email address of a User, which mailer-plugin would implement. I guess the question is what was intended to be accomplished by splitting off mailer-plugin and will that still be accomplished if we have to move a bunch of code back into core, leaving only the actual task/action/command in the plugin?
