Sonar critical issues 97/42397/1
authorSurendra Reddy Katam <SK00524980@techmahindra.com>
Thu, 12 Apr 2018 05:35:55 +0000 (11:05 +0530)
committerSurendra Reddy Katam <SK00524980@techmahindra.com>
Thu, 12 Apr 2018 05:35:55 +0000 (11:05 +0530)
 Either re-interrupt this method or rethrow the InterruptedException
Sonar Link:
https://sonar.onap.org/project/issues?assignees=surendraReddy&id=org.onap.dmaap.messagerouter.msgrtr%3Amsgrtr&open=AV4-Vacn32hFUzlqc5o4&resolved=false&severities=CRITICAL
Location:
src/main/java/com/att/nsa/cambria/beans/DMaaPCambriaLimiter.java
Line No-144

Change-Id: I6c57315d9a4fc913a5e336feb63e2a11910a9ad8
Issue-ID: DMAAP-405
Signed-off-by: Surendra Reddy Katam <SK00524980@techmahindra.com>
src/main/java/com/att/nsa/cambria/beans/DMaaPCambriaLimiter.java

index 1b609b0..4e9fc02 100644 (file)
@@ -142,7 +142,8 @@ public class DMaaPCambriaLimiter {
                                        log.info(ri.getLabel() + ": " + "No sleep configured, just throwing error.");
                                }
                        } catch (InterruptedException e) {
-                               // ignore
+                               log.error(e.toString());
+                               Thread.currentThread().interrupt();
                        }
                        ErrorResponse errRes = new ErrorResponse(HttpStatusCodes.k429_tooManyRequests, 
                                        DMaaPResponseCode.TOO_MANY_REQUESTS.getResponseCode(),