Roger Geere commented on Bug JENKINS-14250

I believe that a change to GitChangeSet.java introduced this problem.

commit 222647bfb3a2bce31f0e1ea9193f4b804e7811fe
Author: Kohsuke Kawaguchi <k...@kohsuke.org>
Date: Wed Mar 21 13:27:40 2012 -0700

if (user == null)
user = User.get(csAuthorEmail.split("@")[0], true);

  • // set email address for user if needed
  • if (fixEmpty(csAuthorEmail) != null) {
    + // set email address for user if none is already available
    + if (fixEmpty(csAuthorEmail) != null && user.getProperty(Mailer.UserProperty.class)==null) { try { user.addProperty(new Mailer.UserProperty(csAuthorEmail)); } catch (IOException e) {

In the case where the call to User.get() causes a new user to be created it seems that the Mailer.UserProperty is already set with a Null string. This is not what I would expect from the documentation but I am not a java expert. So the email address from the commit does not get added to user. At any later stage a call to Mailer.UserProperty.gerAddress() seems to cause an abort. The visible result of this is that there is no creation of a directory and config.xml for the user in the jenkins /users directory.

Reverting this single change in the current code version shows that the directory and file start to be created again.

I am actually hitting a similar problem with the repo plugin which causes problems for the extended e-mail plugin and suspect it may be a similar issue.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to