SONAR-13341 improved error message

diff --git a/server/sonar-server/src/main/java/org/sonar/server/webhook/ws/WebhookSupport.java b/server/sonar-server/src/main/java/org/sonar/server/webhook/ws/WebhookSupport.java
index daa74e2b9c70efc672d2cc26eed2a5770fbe373a..3a6f842f715f10f107569fd428e26f6317172706 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/webhook/ws/WebhookSupport.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/webhook/ws/WebhookSupport.java
@@ -60,7 +60,7 @@ public class WebhookSupport {
       InetAddress address = InetAddress.getByName(okUrl.host());
       if (configuration.getBoolean(SONAR_VALIDATE_WEBHOOKS.getKey()).orElse(true)
         && (address.isLoopbackAddress() || address.isAnyLocalAddress())) {
-        throw new IllegalArgumentException("Invalid URL");
+        throw new IllegalArgumentException("Invalid URL: loopback and wildcard addresses are not allowed for webhooks.");
       }
     } catch (UnknownHostException e) {
       // if a host can not be resolved the deliveries will fail - no need to block it from being set

diff --git a/server/sonar-server/src/test/java/org/sonar/server/webhook/ws/WebhookSupportTest.java b/server/sonar-server/src/test/java/org/sonar/server/webhook/ws/WebhookSupportTest.java
index cca13f0a68c498a1060343eb316de6475885c7ed..49ff03b4be785b02e9706eeaf338f2d60525348a 100644 (file)
--- a/server/sonar-server/src/test/java/org/sonar/server/webhook/ws/WebhookSupportTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/webhook/ws/WebhookSupportTest.java
@@ -78,7 +78,7 @@ public class WebhookSupportTest {
   public void checkUrlPatternFailsForLoopbackAddress(String url) {
     assertThatThrownBy(() -> underTest.checkUrlPattern(url, "msg"))
       .isInstanceOf(IllegalArgumentException.class)
-      .hasMessage("Invalid URL");
+      .hasMessage("Invalid URL: loopback and wildcard addresses are not allowed for webhooks.");
   }
 
   @Test