Skip to content
Permalink
Browse files
JENKINS-43268 make message clearer
todo unit test
  • Loading branch information
davidvanlaatum committed Jun 29, 2017
1 parent 6d4cbac commit 4cb744c3d9d031bd6ef9d95a76d35703525f56c2
Showing 1 changed file with 8 additions and 3 deletions.
@@ -42,6 +42,7 @@

import javax.annotation.CheckForNull;
import javax.mail.internet.InternetAddress;
import java.io.IOException;
import java.lang.reflect.Field;
import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;
@@ -200,9 +201,13 @@ private static void addUsers(final Set<User> users, final TaskListener listener,
listener.getLogger().printf("Warning: %s is not a recognized user, but sending mail anyway%n", userAddress);
} else {
listener.getLogger().printf("Not sending mail to unregistered user %s because your SCM"
+ " claimed this was associated with a user ID ‘%s’ which your security realm"
+ " does not recognize; you may need changes in your SCM plugin%n", userAddress,
user.getId());
+ " claimed this was associated with a user ID ‘", userAddress);
try {
listener.hyperlink('/' + user.getUrl(), user.getDisplayName());
} catch (IOException ignored) {
}
listener.getLogger().printf("' which your security realm does not recognize; you may need" +
" changes in your SCM plugin%n");
continue;
}
}

0 comments on commit 4cb744c

Please sign in to comment.