|
|
_0.png) |
|
|
|
|
Issue Type:
|
 Bug
|
|
Affects Versions:
|
current |
|
Assignee:
|
Nicolas De Loof
|
|
Components:
|
git, plugin |
|
Created:
|
28/Jun/12 5:17 PM
|
|
Description:
|
Create a job that polls a git repo and emails committers
Commit to repo using some git-config name and email address
Check "People" page. A new user is created. Checking the user configure page shows that:
- Their username is the git-config email without the @domain.com
- Everything else is left blank (the email field is not automatically populated)
|
|
Environment:
|
Jenkins 1.472 running in Ubuntu 11.04 on VMWare Fusion.
|
|
Fix Versions:
|
current |
|
Project:
|
Jenkins
|
|
Labels:
|
git
plugin
jenkins
|
|
Priority:
|
 Major
|
|
Reporter:
|
Thomas Kelly
|
|
|
|
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
|
|
Locked
I believe that a change to GitChangeSet.java introduced this problem.
commit 222647bfb3a2bce31f0e1ea9193f4b804e7811fe
Author: Kohsuke Kawaguchi <[hidden email]>
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 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.