From a66e91cda6b728ba04250cffd3b5ed364cf1e666 Mon Sep 17 00:00:00 2001 From: Surendra Reddy Katam Date: Tue, 3 Apr 2018 14:56:53 +0530 Subject: [PATCH] Sonar critical issues Define and throw a dedicated exception instead of using a generic one Sonar link: https://sonar.onap.org/project/issues?assignees=surendraReddy&id=org.onap.dmaap.messagerouter.dmaapclient%3AdmaapClient&open=AV4-MbPQ32hFUzlqc5XH&resolved=false&severities=CRITICAL Location: src/main/java/com/att/nsa/mr/client/MRClientFactory.java Line No-605 Change-Id: I4d2b43d8ae1585f6cd17b32d0c15987f9f5fe514 Issue-ID: DMAAP-380 Signed-off-by: Surendra Reddy Katam --- src/main/java/com/att/nsa/mr/client/MRClientFactory.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/com/att/nsa/mr/client/MRClientFactory.java b/src/main/java/com/att/nsa/mr/client/MRClientFactory.java index 8513129..496f94e 100644 --- a/src/main/java/com/att/nsa/mr/client/MRClientFactory.java +++ b/src/main/java/com/att/nsa/mr/client/MRClientFactory.java @@ -602,7 +602,7 @@ public class MRClientFactory { try { sub = new MRConsumerImpl(MRConsumerImpl.stringToList(host), topic, group, id, i, j, null, null, null); } catch (MalformedURLException e) { - throw new RuntimeException(e); + throw new IllegalArgumentException(e); } sub.setUsername(username); sub.setPassword(password); -- 2.16.6