diff options
author | Surendra Reddy Katam <SK00524980@techmahindra.com> | 2018-09-10 15:47:21 +0530 |
---|---|---|
committer | Surendra Reddy Katam <SK00524980@techmahindra.com> | 2018-09-10 15:47:21 +0530 |
commit | bd630a48890f7f3a565f07968fe9f87218c539fa (patch) | |
tree | dfc114191e20ae72eee7ac88cb4503d6f47a7217 | |
parent | ef5fa212032f5aef7891ad87e41ee086f4501522 (diff) |
Sonar Major issues
This block of commented-out lines of code should be removed
Sonar Link:
https://sonar.onap.org/project/issues?assignees=surendraReddy&id=org.onap.dmaap.messagerouter.msgrtr%3Amsgrtr&resolved=false&severities=MAJOR
Location:
src/main/java/com/att/dmf/mr/service/impl/TopicServiceImpl.java
Line No- L514 L524 L579
Change-Id: Ic54488da6e4114e8942e13dfb257e806faeca886
Issue-ID: DMAAP-761
Signed-off-by: Surendra Reddy Katam <SK00524980@techmahindra.com>
-rw-r--r-- | src/main/java/com/att/dmf/mr/service/impl/TopicServiceImpl.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/main/java/com/att/dmf/mr/service/impl/TopicServiceImpl.java b/src/main/java/com/att/dmf/mr/service/impl/TopicServiceImpl.java index 42a7089..3a707c4 100644 --- a/src/main/java/com/att/dmf/mr/service/impl/TopicServiceImpl.java +++ b/src/main/java/com/att/dmf/mr/service/impl/TopicServiceImpl.java @@ -511,7 +511,7 @@ public class TopicServiceImpl implements TopicService { // - // String permission = aaf.aafPermissionString(topicName, "manage"); + // if(!aaf.aafAuthentication(dmaapContext.getRequest(), permission)) // { // LOGGER.error("Failed to permit write access to producer [" + @@ -521,7 +521,7 @@ public class TopicServiceImpl implements TopicService { // DMaaPResponseCode.ACCESS_NOT_PERMITTED.getResponseCode(), // errorMessages.getNotPermitted1()+" <Grant publish permissions> - // LOGGER.info(errRes); + // throw new DMaaPAccessDeniedException(errRes); // } // } @@ -576,7 +576,7 @@ public class TopicServiceImpl implements TopicService { // DMaaPResponseCode.ACCESS_NOT_PERMITTED.getResponseCode(), // errorMessages.getNotPermitted1()+" <Revoke publish permissions> - // LOGGER.info(errRes); + // throw new DMaaPAccessDeniedException(errRes); // |