From 4e159efc68474fb06b37b2fe4438d957629f9d44 Mon Sep 17 00:00:00 2001 From: Surendra Reddy Katam Date: Mon, 17 Sep 2018 15:53:10 +0530 Subject: 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&open=AV4-VaeT32hFUzlqc5rM&resolved=false&severities=MAJOR Location: src/main/java/com/att/dmf/mr/exception/DMaaPCambriaExceptionMapper.java Line No-49 Change-Id: Ib053c8e75175b7aa62d5ae48ff93852c8749d107 Issue-ID: DMAAP-780 Signed-off-by: Surendra Reddy Katam --- src/main/java/com/att/dmf/mr/exception/DMaaPCambriaExceptionMapper.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/main/java/com/att/dmf') diff --git a/src/main/java/com/att/dmf/mr/exception/DMaaPCambriaExceptionMapper.java b/src/main/java/com/att/dmf/mr/exception/DMaaPCambriaExceptionMapper.java index 84634a9..304c15b 100644 --- a/src/main/java/com/att/dmf/mr/exception/DMaaPCambriaExceptionMapper.java +++ b/src/main/java/com/att/dmf/mr/exception/DMaaPCambriaExceptionMapper.java @@ -46,7 +46,7 @@ public class DMaaPCambriaExceptionMapper implements ExceptionMapper Date: Mon, 17 Sep 2018 16:41:33 +0530 Subject: 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&open=AV4-VaiY32hFUzlqc5u4&resolved=false&severities=MAJOR Location: src/main/java/com/att/dmf/mr/security/DMaaPAuthenticatorImpl.java Line No-47 Change-Id: Id963a5fd2a8e0c75effa58e4aebfaadd916e52bf Issue-ID: DMAAP-781 Signed-off-by: Surendra Reddy Katam --- src/main/java/com/att/dmf/mr/security/DMaaPAuthenticatorImpl.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/main/java/com/att/dmf') diff --git a/src/main/java/com/att/dmf/mr/security/DMaaPAuthenticatorImpl.java b/src/main/java/com/att/dmf/mr/security/DMaaPAuthenticatorImpl.java index 1641901..4a83a1c 100644 --- a/src/main/java/com/att/dmf/mr/security/DMaaPAuthenticatorImpl.java +++ b/src/main/java/com/att/dmf/mr/security/DMaaPAuthenticatorImpl.java @@ -44,7 +44,7 @@ public class DMaaPAuthenticatorImpl implements DMaaPAuthent // Setting timeout to a large value for testing purpose. - // private static final long kDefaultRequestTimeWindow = 1000 * 60 * 10; // + // 10 minutes private static final long kDefaultRequestTimeWindow = 1000 * 60 * 10 * 10 * 10 * 10 * 10; -- cgit 1.2.3-korg From 25a928b80256f30e2c71ca20753f2fee0417f841 Mon Sep 17 00:00:00 2001 From: Surendra Reddy Katam Date: Mon, 17 Sep 2018 17:37:58 +0530 Subject: 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&open=AWXX4Dp7ligbnTmNIX6R&resolved=false&severities=MAJOR Location: src/main/java/com/att/dmf/mr/service/impl/TopicServiceImpl.java Line No-L350 L355 L505 L515 Change-Id: Iadb3f456fa170c2430704f752eb6b741e2c8e56e Issue-ID: DMAAP-782 Signed-off-by: Surendra Reddy Katam --- src/main/java/com/att/dmf/mr/service/impl/TopicServiceImpl.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/main/java/com/att/dmf') 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 29c7665..cef646e 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 @@ -347,12 +347,12 @@ public class TopicServiceImpl implements TopicService { if (user == null && null != dmaapContext.getRequest().getHeader("Authorization")) { LOGGER.info("Authenticating the user, as ACL authentication is not provided"); // String permission = - // "com.att.dmaap.mr.topic"+"|"+topicName+"|"+"manage"; + String permission = ""; String nameSpace = topicName.substring(0, topicName.lastIndexOf(".")); String mrFactoryVal = AJSCPropertiesMap.getProperty(CambriaConstants.msgRtr_prop, "msgRtr.topicfactory.aaf"); - // String tokens[] = topicName.split(".mr.topic."); + permission = mrFactoryVal + nameSpace + "|destroy"; DMaaPAAFAuthenticator aaf = new DMaaPAAFAuthenticatorImpl(); if (!aaf.aafAuthentication(dmaapContext.getRequest(), permission)) { @@ -502,7 +502,7 @@ public class TopicServiceImpl implements TopicService { // - // if(!aaf.aafAuthentication(dmaapContext.getRequest(), permission)) + // { // LOGGER.error("Failed to permit write access to producer [" + // producerId + "] for topic " + topicName @@ -512,7 +512,7 @@ public class TopicServiceImpl implements TopicService { // errorMessages.getNotPermitted1()+" - // throw new DMaaPAccessDeniedException(errRes); + // } // } -- cgit 1.2.3-korg