JENKINS-43268 make message clearer

todo unit test
jenkinsci · Jun 29, 2017 · 4cb744c · 4cb744c
1 parent 6d4cbac
commit 4cb744c
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/main/java/hudson/plugins/emailext/plugins/recipients/RecipientProviderUtilities.java b/src/main/java/hudson/plugins/emailext/plugins/recipients/RecipientProviderUtilities.java
@@ -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;
                             }
                         }