summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEylon Malin <eylon.malin@intl.att.com>2019-08-04 18:03:29 +0300
committerIttay Stern <ittay.stern@att.com>2019-08-04 15:11:25 +0000
commitc459c7efeafd556333c46c65fc4fe2616d3ef532 (patch)
tree74da52be5a3825bbea5c9c667ad3c5821af99bdd
parente8e72260254d2399279a1497c14abafc06018037 (diff)
Remove web-filter for incoming scheduler requests
authentication can be handled in a different way like using AAF (see 48b0c6ee) Issue-ID: VID-378 Signed-off-by: Eylon Malin <eylon.malin@intl.att.com> Change-Id: I68c664bc6a20ce4d8722558e00985b7638fed04b
-rwxr-xr-xepsdk-app-onap/src/main/webapp/WEB-INF/conf/system.properties3
-rwxr-xr-xepsdk-app-onap/src/main/webapp/WEB-INF/conf/system_template.properties3
-rw-r--r--vid-app-common/src/main/java/org/onap/vid/controller/filter/ClientCredentialsFilter.java82
-rw-r--r--vid-app-common/src/main/java/org/onap/vid/scheduler/SchedulerProperties.java3
-rw-r--r--vid-app-common/src/test/java/org/onap/vid/controller/ClientCredentialsFilterTest.java111
5 files changed, 2 insertions, 200 deletions
diff --git a/epsdk-app-onap/src/main/webapp/WEB-INF/conf/system.properties b/epsdk-app-onap/src/main/webapp/WEB-INF/conf/system.properties
index 96fa22082..0fe2f4e15 100755
--- a/epsdk-app-onap/src/main/webapp/WEB-INF/conf/system.properties
+++ b/epsdk-app-onap/src/main/webapp/WEB-INF/conf/system.properties
@@ -197,7 +197,6 @@ scheduler.server.url=http://BYO.scheduler:8989/scheduler
scheduler.submit.new.vnf.change=/v1/ChangeManagement/schedules/{scheduleId}/approvals
scheduler.get.schedules=/v1/ChangeManagement/schedules/scheduleDetails/
-scheduler.basic.auth=
features.set.filename=onap.features.properties
@@ -205,4 +204,4 @@ vid.asyncJob.howLongToKeepOldJobsInDays=7
# thread definition - count and timeout (in seconds)
vid.thread.count=50
-vid.thread.timeout=30 \ No newline at end of file
+vid.thread.timeout=30
diff --git a/epsdk-app-onap/src/main/webapp/WEB-INF/conf/system_template.properties b/epsdk-app-onap/src/main/webapp/WEB-INF/conf/system_template.properties
index 726b56f09..54359c2fb 100755
--- a/epsdk-app-onap/src/main/webapp/WEB-INF/conf/system_template.properties
+++ b/epsdk-app-onap/src/main/webapp/WEB-INF/conf/system_template.properties
@@ -153,7 +153,6 @@ mso.dme2.client.timeout=${MSO_DME2_CLIENT_TIMEOUT}
mso.dme2.client.read.timeout=${MSO_DME2_CLIENT_READ_TIMEOUT}
mso.dme2.server.url=${MSO_DME2_SERVER_URL}
mso.dme2.enabled=${MSO_DME2_ENABLED}
-scheduler.basic.auth=
features.set.filename=onap.features.properties
@@ -161,4 +160,4 @@ vid.asyncJob.howLongToKeepOldJobsInDays=7
# thread definition - count and timeout (in seconds)
vid.thread.count=50
-vid.thread.timeout=30 \ No newline at end of file
+vid.thread.timeout=30
diff --git a/vid-app-common/src/main/java/org/onap/vid/controller/filter/ClientCredentialsFilter.java b/vid-app-common/src/main/java/org/onap/vid/controller/filter/ClientCredentialsFilter.java
deleted file mode 100644
index f1129057f..000000000
--- a/vid-app-common/src/main/java/org/onap/vid/controller/filter/ClientCredentialsFilter.java
+++ /dev/null
@@ -1,82 +0,0 @@
-/*-
- * ============LICENSE_START=======================================================
- * VID
- * ================================================================================
- * Copyright (C) 2017 - 2019 AT&T Intellectual Property. All rights reserved.
- * ================================================================================
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * ============LICENSE_END=========================================================
- */
-
-package org.onap.vid.controller.filter;
-
-import org.apache.commons.lang3.StringUtils;
-import org.onap.vid.scheduler.SchedulerProperties;
-import org.onap.portalsdk.core.logging.logic.EELFLoggerDelegate;
-import org.onap.portalsdk.core.util.SystemProperties;
-import org.springframework.web.filter.GenericFilterBean;
-import javax.servlet.FilterChain;
-import javax.servlet.ServletException;
-import javax.servlet.ServletRequest;
-import javax.servlet.ServletResponse;
-import javax.servlet.annotation.WebFilter;
-import javax.servlet.http.HttpServletRequest;
-import javax.servlet.http.HttpServletResponse;
-import java.io.IOException;
-
-/**
- * Created by amichai on 13/05/2018.
- */
-@WebFilter(urlPatterns = "/change-management/workflow/*")
-public class ClientCredentialsFilter extends GenericFilterBean {
-
- private static final EELFLoggerDelegate filterLogger = EELFLoggerDelegate.getLogger(ClientCredentialsFilter.class);
-
-
- @Override
- public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain) throws IOException, ServletException {
-
- if (!(request instanceof HttpServletRequest) || !(response instanceof HttpServletResponse))
- return;
-
- String expectedAuthorization = SystemProperties.getProperty(SchedulerProperties.SCHEDULER_BASIC_AUTH);
- String actualAuthorization = ((HttpServletRequest)request).getHeader("Authorization");
-
- if (verifyClientCredentials(actualAuthorization, expectedAuthorization)) {
- filterLogger.warn(EELFLoggerDelegate.debugLogger,"Client credentials authenticated.");
- chain.doFilter(request, response);
- return;
- }
-
- filterLogger.warn(EELFLoggerDelegate.debugLogger,"Client did not provide the expected credentials.");
- ((HttpServletResponse) response).sendError(401);
- }
-
- public boolean verifyClientCredentials(String actualAuthorization, String expectedAuthorization)
- {
- if (StringUtils.isEmpty(expectedAuthorization))
- {
- filterLogger.warn(EELFLoggerDelegate.debugLogger,String.format("Expected Authorization is not configured (key: %s)", SchedulerProperties.SCHEDULER_BASIC_AUTH));
- return true;
- }
-
- if (StringUtils.isEmpty(actualAuthorization))
- {
- filterLogger.warn(EELFLoggerDelegate.debugLogger,"Authorization header is missing.");
- return false;
- }
-
- return actualAuthorization.equals(expectedAuthorization);
- }
-
-}
diff --git a/vid-app-common/src/main/java/org/onap/vid/scheduler/SchedulerProperties.java b/vid-app-common/src/main/java/org/onap/vid/scheduler/SchedulerProperties.java
index 8974032b3..400e8d942 100644
--- a/vid-app-common/src/main/java/org/onap/vid/scheduler/SchedulerProperties.java
+++ b/vid-app-common/src/main/java/org/onap/vid/scheduler/SchedulerProperties.java
@@ -35,7 +35,4 @@ public class SchedulerProperties extends SystemProperties {
public static final String SCHEDULER_DELETE_SCHEDULE = "scheduler.delete.schedule";
- public static final String SCHEDULER_BASIC_AUTH = "scheduler.basic.auth";
-
-
}
diff --git a/vid-app-common/src/test/java/org/onap/vid/controller/ClientCredentialsFilterTest.java b/vid-app-common/src/test/java/org/onap/vid/controller/ClientCredentialsFilterTest.java
deleted file mode 100644
index 37c02b2e0..000000000
--- a/vid-app-common/src/test/java/org/onap/vid/controller/ClientCredentialsFilterTest.java
+++ /dev/null
@@ -1,111 +0,0 @@
-/*-
- * ============LICENSE_START=======================================================
- * VID
- * ================================================================================
- * Copyright (C) 2017 - 2019 AT&T Intellectual Property. All rights reserved.
- * ================================================================================
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * ============LICENSE_END=========================================================
- */
-
-package org.onap.vid.controller;
-
-
-import org.junit.Assert;
-import org.mockito.Mockito;
-import org.onap.vid.controller.filter.ClientCredentialsFilter;
-import org.testng.annotations.DataProvider;
-import org.testng.annotations.Test;
-
-import javax.servlet.FilterChain;
-import javax.servlet.ServletException;
-import javax.servlet.http.HttpServletRequest;
-import javax.servlet.http.HttpServletResponse;
-import java.io.IOException;
-
-import static org.mockito.ArgumentMatchers.any;
-
-
-/**
- * Created by amichai on 16/05/2018.
- */
-public class ClientCredentialsFilterTest {
-
- @DataProvider
- public static Object[][] authorizedData() {
- return new Object[][] {
- {"Basic 123==", null},
- {null, null},
- {null, ""},
- {"Basic 123==", ""},
- {"Basic 123==", "Basic 123=="}
- };
- }
-
- @DataProvider
- public static Object[][] notAuthorizedData() {
- return new Object[][] {
- {null, "Basic 123=="},
- {"", "Basic 123=="},
- {"not null but not as expected", "Basic 123=="},
- {"basic 123==", "Basic 123=="}
- };
- }
-
- @DataProvider
- public static Object[][] clientVerified() {
- return new Object[][] {
- {true},
- {false}
- };
- }
-
- @Test(dataProvider = "authorizedData")
- public void givenAuthorizationHeader_Authorized(String actualAuth, String expectedAuth){
- ClientCredentialsFilter filter = new ClientCredentialsFilter();
- Assert.assertTrue(filter.verifyClientCredentials(actualAuth, expectedAuth));
- }
-
- @Test(dataProvider = "notAuthorizedData")
- public void givenAuthorizationHeader_NotAuthorized(String actualAuth, String expectedAuth){
- ClientCredentialsFilter filter = new ClientCredentialsFilter();
- Assert.assertFalse(filter.verifyClientCredentials(actualAuth, expectedAuth));
- }
-
- //@Test(dataProvider = "clientVerified")
- public void notAuthorized_return401(Boolean clientVerified) throws IOException, ServletException {
- ClientCredentialsFilter filter = Mockito.mock(ClientCredentialsFilter.class);
- HttpServletResponse response = Mockito.mock(HttpServletResponse.class);
- HttpServletRequest request = Mockito.mock(HttpServletRequest.class);
- FilterChain chain = Mockito.mock(FilterChain.class);
-
-
- Mockito.when(filter.verifyClientCredentials(any(),any())).thenReturn(clientVerified);
- Mockito.doNothing().when(response).sendError(401);
-
- Mockito.doCallRealMethod().when(filter).doFilter(request,response,chain);
- filter.doFilter(request,response,chain);
-
- if (clientVerified)
- {
- Mockito.verify(chain).doFilter(request,response);
-
- }
- else {
- Mockito.verify(response).sendError(401);
- }
-
- }
-
-
-}