From f15306304a747922990eeabbedea63ae6e277149 Mon Sep 17 00:00:00 2001 From: burdziak Date: Mon, 16 Sep 2019 12:56:44 +0200 Subject: Personalized client certificate authentication via FTPes in 5G Bulk PM usecase doesn't work Issue-ID: DCAEGEN2-1754 Signed-off-by: burdziak Change-Id: Idec3efce8412dd175ecf75f55790a24cec651104 --- datafile-app-server/pom.xml | 2 +- .../collectors/datafile/model/FileData.java | 6 +++++ .../collectors/datafile/model/FileDataTest.java | 27 ++++++++++++++++++++++ pom.xml | 2 +- version.properties | 4 ++-- 5 files changed, 37 insertions(+), 4 deletions(-) diff --git a/datafile-app-server/pom.xml b/datafile-app-server/pom.xml index 6b0ebe4d..5f9474a0 100644 --- a/datafile-app-server/pom.xml +++ b/datafile-app-server/pom.xml @@ -25,7 +25,7 @@ org.onap.dcaegen2.collectors datafile - 1.2.2-SNAPSHOT + 1.3.0-SNAPSHOT org.onap.dcaegen2.collectors.datafile diff --git a/datafile-app-server/src/main/java/org/onap/dcaegen2/collectors/datafile/model/FileData.java b/datafile-app-server/src/main/java/org/onap/dcaegen2/collectors/datafile/model/FileData.java index 8cafd0c4..4805cb47 100644 --- a/datafile-app-server/src/main/java/org/onap/dcaegen2/collectors/datafile/model/FileData.java +++ b/datafile-app-server/src/main/java/org/onap/dcaegen2/collectors/datafile/model/FileData.java @@ -130,6 +130,12 @@ public abstract class FileData { String[] userAndPassword = userInfoString.split(":"); if (userAndPassword.length == 2) { return Optional.of(userAndPassword); + }else if(userAndPassword.length == 1)//if just user + { + String[] tab = new String[2]; + tab[0] = userAndPassword[0]; + tab[1] = "";//add empty password + return Optional.of(tab); } } return Optional.empty(); diff --git a/datafile-app-server/src/test/java/org/onap/dcaegen2/collectors/datafile/model/FileDataTest.java b/datafile-app-server/src/test/java/org/onap/dcaegen2/collectors/datafile/model/FileDataTest.java index 35894f9c..10a84e76 100644 --- a/datafile-app-server/src/test/java/org/onap/dcaegen2/collectors/datafile/model/FileDataTest.java +++ b/datafile-app-server/src/test/java/org/onap/dcaegen2/collectors/datafile/model/FileDataTest.java @@ -37,6 +37,8 @@ public class FileDataTest { private static final int PORT_22 = 22; private static final String LOCATION_WITH_USER = FTPES_SCHEME + USER + ":" + PWD + "@" + SERVER_ADDRESS + ":" + PORT_22 + REMOTE_FILE_LOCATION; + private static final String LOCATION_WITH_USER_NO_PASSWORD = + FTPES_SCHEME + USER + "@" + SERVER_ADDRESS + ":" + PORT_22 + REMOTE_FILE_LOCATION; private static final String LOCATION_WITHOUT_USER = FTPES_SCHEME + SERVER_ADDRESS + ":" + PORT_22 + REMOTE_FILE_LOCATION; @@ -65,6 +67,18 @@ public class FileDataTest { .build(); } + private FileData properFileDataWithUserNoPassword() { + return ImmutableFileData.builder() // + .name("name") // + .location(LOCATION_WITH_USER_NO_PASSWORD) // + .compression("comp") // + .fileFormatType("type") // + .fileFormatVersion("version") // + .scheme(Scheme.FTPS) // + .messageMetaData(messageMetaData()) // + .build(); + } + private FileData properFileDataWithoutUser() { return ImmutableFileData.builder() // .name("name") // @@ -90,6 +104,19 @@ public class FileDataTest { assertEquals(expectedFileServerData, actualFileServerData); } + @Test + public void fileServerData_properLocationWithUserNoPassword() { + ImmutableFileServerData expectedFileServerData = ImmutableFileServerData.builder() // + .serverAddress(SERVER_ADDRESS) // + .port(PORT_22) // + .userId(USER) // + .password("") // + .build(); + + FileServerData actualFileServerData = properFileDataWithUserNoPassword().fileServerData(); + assertEquals(expectedFileServerData, actualFileServerData); + } + @Test public void fileServerData_properLocationWithoutUser() { ImmutableFileServerData expectedFileServerData = ImmutableFileServerData.builder() // diff --git a/pom.xml b/pom.xml index 31d2df39..50d2de9d 100644 --- a/pom.xml +++ b/pom.xml @@ -30,7 +30,7 @@ org.onap.dcaegen2.collectors datafile - 1.2.2-SNAPSHOT + 1.3.0-SNAPSHOT dcaegen2-collectors.datafile datafile collector diff --git a/version.properties b/version.properties index e562846c..f520c971 100644 --- a/version.properties +++ b/version.properties @@ -1,6 +1,6 @@ major=1 -minor=2 -patch=2 +minor=3 +patch=0 base_version=${major}.${minor}.${patch} release_version=${base_version} snapshot_version=${base_version}-SNAPSHOT -- cgit 1.2.3-korg