From 038bd748c7ed26580d8741a0d3119342b3c63f28 Mon Sep 17 00:00:00 2001 From: skyflow-vivek Date: Tue, 10 Dec 2024 17:23:30 +0530 Subject: [PATCH 01/10] SK-1737 Fix inconsistencies in Java SDK --- src/main/java/com/skyflow/Skyflow.java | 18 +++++++--- .../java/com/skyflow/utils/HttpUtility.java | 7 ++-- .../utils/validations/Validations.java | 7 ++-- .../connection/InvokeConnectionResponse.java | 24 +++++++------ .../controller/ConnectionController.java | 5 ++- src/test/java/com/skyflow/SkyflowTests.java | 34 +++++++++++++++++++ .../connection/InvokeConnectionTests.java | 17 ++++++---- .../controller/ConnectionControllerTests.java | 2 +- 8 files changed, 86 insertions(+), 28 deletions(-) diff --git a/src/main/java/com/skyflow/Skyflow.java b/src/main/java/com/skyflow/Skyflow.java index 2a2e42cc..6866bd43 100644 --- a/src/main/java/com/skyflow/Skyflow.java +++ b/src/main/java/com/skyflow/Skyflow.java @@ -82,13 +82,23 @@ public LogLevel getLogLevel() { return this.builder.logLevel; } - public VaultController vault() { - String vaultId = (String) this.builder.vaultClientsMap.keySet().toArray()[0]; + public VaultController vault() throws SkyflowException { + Object[] array = this.builder.vaultClientsMap.keySet().toArray(); + if (array.length < 1) { + LogUtil.printErrorLog(ErrorLogs.VAULT_CONFIG_DOES_NOT_EXIST.getLog()); + throw new SkyflowException(ErrorCode.INVALID_INPUT.getCode(), ErrorMessage.VaultIdNotInConfigList.getMessage()); + } + String vaultId = (String) array[0]; return this.vault(vaultId); } - public VaultController vault(String vaultId) { - return this.builder.vaultClientsMap.get(vaultId); + public VaultController vault(String vaultId) throws SkyflowException { + VaultController controller = this.builder.vaultClientsMap.get(vaultId); + if (controller == null) { + LogUtil.printErrorLog(ErrorLogs.VAULT_CONFIG_DOES_NOT_EXIST.getLog()); + throw new SkyflowException(ErrorCode.INVALID_INPUT.getCode(), ErrorMessage.VaultIdNotInConfigList.getMessage()); + } + return controller; } public ConnectionController connection() { diff --git a/src/main/java/com/skyflow/utils/HttpUtility.java b/src/main/java/com/skyflow/utils/HttpUtility.java index 93cbdd31..664cc9da 100644 --- a/src/main/java/com/skyflow/utils/HttpUtility.java +++ b/src/main/java/com/skyflow/utils/HttpUtility.java @@ -13,14 +13,13 @@ public final class HttpUtility { + private static final String LINE_FEED = "\r\n"; private static String requestID; public static String getRequestID() { return requestID; } - private static final String LINE_FEED = "\r\n"; - public static String sendRequest(String method, URL url, JsonObject params, Map headers) throws IOException, SkyflowException { HttpURLConnection connection = null; @@ -55,7 +54,7 @@ public static String sendRequest(String method, URL url, JsonObject params, Map< input = formatJsonToFormEncodedString(params).getBytes(StandardCharsets.UTF_8); } else if (requestContentType.contains("multipart/form-data")) { input = formatJsonToMultiPartFormDataString(params, boundary).getBytes(StandardCharsets.UTF_8); - }else { + } else { input = params.toString().getBytes(StandardCharsets.UTF_8); } @@ -66,7 +65,7 @@ public static String sendRequest(String method, URL url, JsonObject params, Map< int status = connection.getResponseCode(); String requestID = connection.getHeaderField("x-request-id"); - HttpUtility.requestID = requestID; + HttpUtility.requestID = requestID.split(",")[0]; Reader streamReader; if (status > 299) { diff --git a/src/main/java/com/skyflow/utils/validations/Validations.java b/src/main/java/com/skyflow/utils/validations/Validations.java index 67853bab..2876c15d 100644 --- a/src/main/java/com/skyflow/utils/validations/Validations.java +++ b/src/main/java/com/skyflow/utils/validations/Validations.java @@ -1,5 +1,7 @@ package com.skyflow.utils.validations; +import com.google.gson.Gson; +import com.google.gson.JsonObject; import com.skyflow.config.ConnectionConfig; import com.skyflow.config.Credentials; import com.skyflow.config.VaultConfig; @@ -132,8 +134,9 @@ public static void validateInvokeConnectionRequest(InvokeConnectionRequest invok } if (requestBody != null) { - Map requestBodyMap = (Map) requestBody; - if (requestBodyMap.isEmpty()) { + Gson gson = new Gson(); + JsonObject bodyObject = gson.toJsonTree(requestBody).getAsJsonObject(); + if (bodyObject.isEmpty()) { LogUtil.printErrorLog(Utils.parameterizedString( ErrorLogs.EMPTY_REQUEST_BODY.getLog(), InterfaceName.INVOKE_CONNECTION.getName())); throw new SkyflowException(ErrorCode.INVALID_INPUT.getCode(), ErrorMessage.EmptyRequestBody.getMessage()); diff --git a/src/main/java/com/skyflow/vault/connection/InvokeConnectionResponse.java b/src/main/java/com/skyflow/vault/connection/InvokeConnectionResponse.java index 76f8823f..a30a44c9 100644 --- a/src/main/java/com/skyflow/vault/connection/InvokeConnectionResponse.java +++ b/src/main/java/com/skyflow/vault/connection/InvokeConnectionResponse.java @@ -1,25 +1,29 @@ package com.skyflow.vault.connection; import com.google.gson.Gson; +import com.google.gson.GsonBuilder; import com.google.gson.JsonObject; -import com.google.gson.JsonParser; public class InvokeConnectionResponse { - private JsonObject response; + private final JsonObject data; + private final JsonObject metadata; - public InvokeConnectionResponse(JsonObject response) { - this.response = response; + public InvokeConnectionResponse(JsonObject data, JsonObject metadata) { + this.data = data; + this.metadata = metadata; } - public JsonObject getResponse() { - return response; + public JsonObject getData() { + return data; + } + + public JsonObject getMetadata() { + return metadata; } @Override public String toString() { - Gson gson = new Gson(); - JsonObject responseObject = JsonParser.parseString(gson.toJson(this)).getAsJsonObject(); - responseObject = responseObject.remove("response").getAsJsonObject(); - return responseObject.toString(); + Gson gson = new GsonBuilder().serializeNulls().create(); + return gson.toJson(this); } } diff --git a/src/main/java/com/skyflow/vault/controller/ConnectionController.java b/src/main/java/com/skyflow/vault/controller/ConnectionController.java index 66720521..0bb2251e 100644 --- a/src/main/java/com/skyflow/vault/controller/ConnectionController.java +++ b/src/main/java/com/skyflow/vault/controller/ConnectionController.java @@ -64,7 +64,10 @@ public InvokeConnectionResponse invoke(InvokeConnectionRequest invokeConnectionR } String response = HttpUtility.sendRequest(requestMethod.name(), new URL(filledURL), requestBody, headers); - connectionResponse = new InvokeConnectionResponse((JsonObject) JsonParser.parseString(response)); + JsonObject data = JsonParser.parseString(response).getAsJsonObject(); + JsonObject metadata = new JsonObject(); + metadata.addProperty("requestId", HttpUtility.getRequestID()); + connectionResponse = new InvokeConnectionResponse(data, metadata); LogUtil.printInfoLog(InfoLogs.INVOKE_CONNECTION_REQUEST_RESOLVED.getLog()); } catch (IOException e) { LogUtil.printErrorLog(ErrorLogs.INVOKE_CONNECTION_REQUEST_REJECTED.getLog()); diff --git a/src/test/java/com/skyflow/SkyflowTests.java b/src/test/java/com/skyflow/SkyflowTests.java index f7385af1..8ccff81d 100644 --- a/src/test/java/com/skyflow/SkyflowTests.java +++ b/src/test/java/com/skyflow/SkyflowTests.java @@ -198,6 +198,40 @@ public void testGettingNonExistentVaultConfigInSkyflowClient() { } } + @Test + public void testGettingAlreadyRemovedVaultFromEmptyConfigs() { + try { + Skyflow skyflowClient = Skyflow.builder().build(); + skyflowClient.vault(); + Assert.fail(EXCEPTION_NOT_THROWN); + } catch (SkyflowException e) { + Assert.assertEquals(ErrorCode.INVALID_INPUT.getCode(), e.getHttpCode()); + Assert.assertEquals(ErrorMessage.VaultIdNotInConfigList.getMessage(), e.getMessage()); + } + } + + @Test + public void testGettingAlreadyRemovedVaultFromNonEmptyConfigs() { + try { + VaultConfig primaryConfig = new VaultConfig(); + primaryConfig.setVaultId(vaultID); + primaryConfig.setClusterId(clusterID); + primaryConfig.setEnv(Env.SANDBOX); + + VaultConfig secondaryConfig = new VaultConfig(); + secondaryConfig.setVaultId(vaultID + "123"); + secondaryConfig.setClusterId(clusterID); + secondaryConfig.setEnv(Env.SANDBOX); + Skyflow skyflowClient = Skyflow.builder().addVaultConfig(primaryConfig).addVaultConfig(secondaryConfig).build(); + skyflowClient.removeVaultConfig(vaultID); + skyflowClient.vault(vaultID); + Assert.fail(EXCEPTION_NOT_THROWN); + } catch (SkyflowException e) { + Assert.assertEquals(ErrorCode.INVALID_INPUT.getCode(), e.getHttpCode()); + Assert.assertEquals(ErrorMessage.VaultIdNotInConfigList.getMessage(), e.getMessage()); + } + } + @Test public void testAddingInvalidConnectionConfigInSkyflowBuilder() { diff --git a/src/test/java/com/skyflow/vault/connection/InvokeConnectionTests.java b/src/test/java/com/skyflow/vault/connection/InvokeConnectionTests.java index 6ac020e1..90bc4c1e 100644 --- a/src/test/java/com/skyflow/vault/connection/InvokeConnectionTests.java +++ b/src/test/java/com/skyflow/vault/connection/InvokeConnectionTests.java @@ -420,12 +420,17 @@ public void testEmptyRequestBodyInInvokeConnectionRequestValidations() { @Test public void testInvokeConnectionResponse() { try { - JsonObject responseObject = new JsonObject(); - responseObject.addProperty("test_key_1", "test_value_1"); - responseObject.addProperty("test_key_2", "test_value_2"); - InvokeConnectionResponse connectionResponse = new InvokeConnectionResponse(responseObject); - Assert.assertEquals(2, connectionResponse.getResponse().size()); - Assert.assertEquals(responseObject.toString(), connectionResponse.toString()); + JsonObject data = new JsonObject(); + data.addProperty("test_key_1", "test_value_1"); + data.addProperty("test_key_2", "test_value_2"); + JsonObject metadata = new JsonObject(); + metadata.addProperty("requestId", "12345"); + InvokeConnectionResponse connectionResponse = new InvokeConnectionResponse(data, metadata); + String responseString = "{\"data\":{\"test_key_1\":\"test_value_1\",\"test_key_2\":\"test_value_2\"}," + + "\"metadata\":{\"requestId\":\"12345\"}}"; + Assert.assertEquals(2, connectionResponse.getData().size()); + Assert.assertEquals(responseString, connectionResponse.toString()); + Assert.assertEquals(1, connectionResponse.getMetadata().size()); } catch (Exception e) { Assert.fail(INVALID_EXCEPTION_THROWN); } diff --git a/src/test/java/com/skyflow/vault/controller/ConnectionControllerTests.java b/src/test/java/com/skyflow/vault/controller/ConnectionControllerTests.java index 102eb9d7..b121280a 100644 --- a/src/test/java/com/skyflow/vault/controller/ConnectionControllerTests.java +++ b/src/test/java/com/skyflow/vault/controller/ConnectionControllerTests.java @@ -39,7 +39,7 @@ public static void setup() { @Test public void testInvalidRequestInInvokeConnectionMethod() { try { - HashMap requestBody = new HashMap<>(); + HashMap requestBody = new HashMap<>(); InvokeConnectionRequest connectionRequest = InvokeConnectionRequest.builder().requestBody(requestBody).build(); Skyflow skyflowClient = Skyflow.builder().setLogLevel(LogLevel.DEBUG).addConnectionConfig(connectionConfig).build(); skyflowClient.connection().invoke(connectionRequest); From 7938fa6403c2c7253e60ba84508ca94cca6b3044 Mon Sep 17 00:00:00 2001 From: skyflow-vivek Date: Tue, 10 Dec 2024 12:27:28 +0000 Subject: [PATCH 02/10] [AUTOMATED] Private Release 1.15.0-dev-8339ce7 --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index 2801b615..c56da584 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.skyflow skyflow-java - 1.15.0-dev.4ea53e6 + 1.15.0-dev.8339ce7 jar ${project.groupId}:${project.artifactId} From f146b51a64c31242e78fd57828444a8043e958a2 Mon Sep 17 00:00:00 2001 From: skyflow-vivek Date: Wed, 11 Dec 2024 12:39:22 +0530 Subject: [PATCH 03/10] SK-1737 Update workflow to fix build issue --- .github/workflows/pr.yml | 10 +++++----- .github/workflows/shared-build-and-deploy.yml | 18 ++++++++++-------- 2 files changed, 15 insertions(+), 13 deletions(-) diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml index ff48495f..5554efe3 100644 --- a/.github/workflows/pr.yml +++ b/.github/workflows/pr.yml @@ -11,11 +11,11 @@ jobs: uses: gsactions/commit-message-checker@v1 with: pattern: '(\[?[A-Z]{1,5}-[1-9][0-9]*)|(\[AUTOMATED\])|(Merge)|(Release).+$' - flags: 'gm' - excludeDescription: 'true' - checkAllCommitMessages: 'true' + flags: "gm" + excludeDescription: "true" + checkAllCommitMessages: "true" accessToken: ${{ secrets.PAT_ACTIONS }} - error: 'One of your your commit messages is not matching the format with JIRA ID Ex: ( SDK-123 commit message )' + error: "One of your your commit messages is not matching the format with JIRA ID Ex: ( SDK-123 commit message )" Test: runs-on: ubuntu-latest steps: @@ -40,7 +40,7 @@ jobs: echo TEST_REUSABLE_TOKEN=${{ secrets.TEST_REUSABLE_TOKEN }} >> .env - name: Build & Run tests with Maven - run: mvn -B package -DTEST_EXPIRED_TOKEN=${{ secrets.TEST_EXPIRED_TOKEN }} -DTEST_DATA_CREDENTIALS_FILE=${{ secrets.TEST_DATA_CREDENTIALS_FILE }} -f pom.xml + run: mvn -B package -f pom.xml - name: Codecov uses: codecov/codecov-action@v2.1.0 diff --git a/.github/workflows/shared-build-and-deploy.yml b/.github/workflows/shared-build-and-deploy.yml index e235a83d..b8a3186f 100644 --- a/.github/workflows/shared-build-and-deploy.yml +++ b/.github/workflows/shared-build-and-deploy.yml @@ -3,22 +3,22 @@ on: workflow_call: inputs: ref: - description: 'Git reference to use (e.g., main or branch name)' + description: "Git reference to use (e.g., main or branch name)" required: true type: string is-internal: - description: 'Flag for internal release' + description: "Flag for internal release" required: true type: boolean server-id: - description: 'Id of the repository' + description: "Id of the repository" required: true type: string - + profile: - description: 'Profile to pick from pom.xml' + description: "Profile to pick from pom.xml" required: true type: string secrets: @@ -34,6 +34,9 @@ on: gpg-passphrase: required: true +env: + TEST_REUSABLE_TOKEN: ${{ secrets.TEST_REUSABLE_TOKEN }} + jobs: publish: runs-on: ubuntu-latest @@ -46,8 +49,8 @@ jobs: - name: Set up maven or jfrog repository uses: actions/setup-java@v1 with: - java-version: '1.8' - distribution: 'adopt' + java-version: "1.8" + distribution: "adopt" server-id: ${{ inputs.server-id }} server-username: SERVER_USERNAME server-password: SERVER_PASSWORD @@ -89,7 +92,6 @@ jobs: touch .env echo SKYFLOW_CREDENTIALS=${{ secrets.SKYFLOW_CREDENTIALS }} >> .env echo TEST_EXPIRED_TOKEN=${{ secrets.TEST_EXPIRED_TOKEN }} >> .env - echo TEST_REUSABLE_TOKEN=${{ secrets.TEST_REUSABLE_TOKEN }} >> .env - name: Create credentials json id: create-json From 09fb1e8bf11290732c1c60dc4dd1dbce9a7c881e Mon Sep 17 00:00:00 2001 From: skyflow-vivek Date: Wed, 11 Dec 2024 07:10:59 +0000 Subject: [PATCH 04/10] [AUTOMATED] Private Release 1.15.0-dev-f146b51 --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index c56da584..50021123 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.skyflow skyflow-java - 1.15.0-dev.8339ce7 + 1.15.0-dev.f146b51 jar ${project.groupId}:${project.artifactId} From ad9b710af44ec93891d24f725b894337032a9b1d Mon Sep 17 00:00:00 2001 From: skyflow-vivek Date: Wed, 11 Dec 2024 12:49:44 +0530 Subject: [PATCH 05/10] SK-1737 add stack trace to debug failures --- src/test/java/com/skyflow/ConnectionClientTests.java | 1 + src/test/java/com/skyflow/VaultClientTests.java | 1 + 2 files changed, 2 insertions(+) diff --git a/src/test/java/com/skyflow/ConnectionClientTests.java b/src/test/java/com/skyflow/ConnectionClientTests.java index 4a69120c..ab514844 100644 --- a/src/test/java/com/skyflow/ConnectionClientTests.java +++ b/src/test/java/com/skyflow/ConnectionClientTests.java @@ -58,6 +58,7 @@ public void testSetBearerToken() { // re-use scenario connectionClient.setBearerToken(); } catch (Exception e) { + e.printStackTrace(); Assert.fail(INVALID_EXCEPTION_THROWN); } } diff --git a/src/test/java/com/skyflow/VaultClientTests.java b/src/test/java/com/skyflow/VaultClientTests.java index 8a02caaf..66495140 100644 --- a/src/test/java/com/skyflow/VaultClientTests.java +++ b/src/test/java/com/skyflow/VaultClientTests.java @@ -264,6 +264,7 @@ public void testSetBearerToken() { HttpBearerAuth auth = (HttpBearerAuth) vaultClient.getApiClient().getAuthentication("Bearer"); Assert.assertEquals(bearerToken, auth.getBearerToken()); } catch (Exception e) { + e.printStackTrace(); Assert.fail(INVALID_EXCEPTION_THROWN); } } From deddbcffc79e0d6d634a01621a0731ea92373794 Mon Sep 17 00:00:00 2001 From: skyflow-vivek Date: Wed, 11 Dec 2024 07:20:16 +0000 Subject: [PATCH 06/10] [AUTOMATED] Private Release 1.15.0-dev-ad9b710 --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index 50021123..10c37658 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.skyflow skyflow-java - 1.15.0-dev.f146b51 + 1.15.0-dev.ad9b710 jar ${project.groupId}:${project.artifactId} From d2e7155415d71a392f9bd273c0c8cb0c8a42ccb1 Mon Sep 17 00:00:00 2001 From: skyflow-vivek Date: Wed, 11 Dec 2024 13:02:18 +0530 Subject: [PATCH 07/10] SK-1737 Fix spelling in error message --- .github/workflows/shared-build-and-deploy.yml | 4 +--- src/main/java/com/skyflow/errors/ErrorMessage.java | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/.github/workflows/shared-build-and-deploy.yml b/.github/workflows/shared-build-and-deploy.yml index b8a3186f..faadc389 100644 --- a/.github/workflows/shared-build-and-deploy.yml +++ b/.github/workflows/shared-build-and-deploy.yml @@ -34,9 +34,6 @@ on: gpg-passphrase: required: true -env: - TEST_REUSABLE_TOKEN: ${{ secrets.TEST_REUSABLE_TOKEN }} - jobs: publish: runs-on: ubuntu-latest @@ -92,6 +89,7 @@ jobs: touch .env echo SKYFLOW_CREDENTIALS=${{ secrets.SKYFLOW_CREDENTIALS }} >> .env echo TEST_EXPIRED_TOKEN=${{ secrets.TEST_EXPIRED_TOKEN }} >> .env + echo TEST_REUSABLE_TOKEN=${{ secrets.TEST_REUSABLE_TOKEN }} >> .env - name: Create credentials json id: create-json diff --git a/src/main/java/com/skyflow/errors/ErrorMessage.java b/src/main/java/com/skyflow/errors/ErrorMessage.java index e1034976..d6af06aa 100644 --- a/src/main/java/com/skyflow/errors/ErrorMessage.java +++ b/src/main/java/com/skyflow/errors/ErrorMessage.java @@ -28,7 +28,7 @@ public enum ErrorMessage { NoTokenGenerationMeansPassed("%s0 Initialization failed. Invalid credentials. Specify any one from 'path', 'credentialsString', 'token' or 'apiKey'."), EmptyCredentialFilePath("%s0 Initialization failed. Invalid credentials. Credentials file path must not be empty."), EmptyCredentialsString("%s0 Initialization failed. Invalid credentials. Credentials string must not be empty."), - EmptyToken("%s0 Initialization failed. Invalid credentials. Token mut not be empty."), + EmptyToken("%s0 Initialization failed. Invalid credentials. Token must not be empty."), EmptyApikey("%s0 Initialization failed. Invalid credentials. Api key must not be empty."), InvalidApikey("%s0 Initialization failed. Invalid credentials. Specify valid api key."), EmptyRoles("%s0 Initialization failed. Invalid roles. Specify at least one role."), From a3f5f23ff9573f3be609c39be067705baa2c7b36 Mon Sep 17 00:00:00 2001 From: skyflow-vivek Date: Wed, 11 Dec 2024 07:32:49 +0000 Subject: [PATCH 08/10] [AUTOMATED] Private Release 1.15.0-dev-d2e7155 --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index 10c37658..81c59c51 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.skyflow skyflow-java - 1.15.0-dev.ad9b710 + 1.15.0-dev.d2e7155 jar ${project.groupId}:${project.artifactId} From a0537e8df29d0e115527c373e81f494379880992 Mon Sep 17 00:00:00 2001 From: skyflow-vivek Date: Wed, 11 Dec 2024 13:22:33 +0530 Subject: [PATCH 09/10] SK-1737 Added secrets for shared workflow --- .github/workflows/internal-release.yml | 5 +++-- .github/workflows/release.yml | 4 +++- .github/workflows/shared-build-and-deploy.yml | 15 ++++++++++++--- .../java/com/skyflow/ConnectionClientTests.java | 1 - src/test/java/com/skyflow/VaultClientTests.java | 1 - 5 files changed, 18 insertions(+), 8 deletions(-) diff --git a/.github/workflows/internal-release.yml b/.github/workflows/internal-release.yml index a354c5cc..6aea0e52 100644 --- a/.github/workflows/internal-release.yml +++ b/.github/workflows/internal-release.yml @@ -21,5 +21,6 @@ jobs: server-password: ${{ secrets.ARTIFACTORY_PASSWORD }} gpg-key: ${{ secrets.JFROG_GPG_KEY }} gpg-passphrase: ${{ secrets.JFROG_GPG_PASSPHRASE }} - - + skyflow-credentials: ${{ secrets.SKYFLOW_CREDENTIALS }} >> .env + test-expired-token: ${{ secrets.TEST_EXPIRED_TOKEN }} >> .env + test-reusable-token: ${{ secrets.TEST_REUSABLE_TOKEN }} >> .env diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 7b6c57de..a4c7bfd9 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -15,4 +15,6 @@ jobs: server-password: ${{ secrets.OSSRH_PASSWORD }} gpg-key: ${{ secrets.MAVEN_GPG_PRIVATE_KEY }} gpg-passphrase: ${{ secrets.MAVEN_GPG_PASSPHRASE }} - \ No newline at end of file + skyflow-credentials: ${{ secrets.SKYFLOW_CREDENTIALS }} >> .env + test-expired-token: ${{ secrets.TEST_EXPIRED_TOKEN }} >> .env + test-reusable-token: ${{ secrets.TEST_REUSABLE_TOKEN }} >> .env diff --git a/.github/workflows/shared-build-and-deploy.yml b/.github/workflows/shared-build-and-deploy.yml index faadc389..4bd80367 100644 --- a/.github/workflows/shared-build-and-deploy.yml +++ b/.github/workflows/shared-build-and-deploy.yml @@ -34,6 +34,15 @@ on: gpg-passphrase: required: true + skyflow-credentials: + required: true + + test-expired-token: + required: true + + test-reusable-token: + required: true + jobs: publish: runs-on: ubuntu-latest @@ -87,9 +96,9 @@ jobs: id: create-env run: | touch .env - echo SKYFLOW_CREDENTIALS=${{ secrets.SKYFLOW_CREDENTIALS }} >> .env - echo TEST_EXPIRED_TOKEN=${{ secrets.TEST_EXPIRED_TOKEN }} >> .env - echo TEST_REUSABLE_TOKEN=${{ secrets.TEST_REUSABLE_TOKEN }} >> .env + echo SKYFLOW_CREDENTIALS=${{ secrets.skyflow-credentials }} >> .env + echo TEST_EXPIRED_TOKEN=${{ secrets.test-expired-token }} >> .env + echo TEST_REUSABLE_TOKEN=${{ secrets.test-reusable-token }} >> .env - name: Create credentials json id: create-json diff --git a/src/test/java/com/skyflow/ConnectionClientTests.java b/src/test/java/com/skyflow/ConnectionClientTests.java index ab514844..4a69120c 100644 --- a/src/test/java/com/skyflow/ConnectionClientTests.java +++ b/src/test/java/com/skyflow/ConnectionClientTests.java @@ -58,7 +58,6 @@ public void testSetBearerToken() { // re-use scenario connectionClient.setBearerToken(); } catch (Exception e) { - e.printStackTrace(); Assert.fail(INVALID_EXCEPTION_THROWN); } } diff --git a/src/test/java/com/skyflow/VaultClientTests.java b/src/test/java/com/skyflow/VaultClientTests.java index 66495140..8a02caaf 100644 --- a/src/test/java/com/skyflow/VaultClientTests.java +++ b/src/test/java/com/skyflow/VaultClientTests.java @@ -264,7 +264,6 @@ public void testSetBearerToken() { HttpBearerAuth auth = (HttpBearerAuth) vaultClient.getApiClient().getAuthentication("Bearer"); Assert.assertEquals(bearerToken, auth.getBearerToken()); } catch (Exception e) { - e.printStackTrace(); Assert.fail(INVALID_EXCEPTION_THROWN); } } From 3231ca87aef4153608eaf1f643c47f8cd2101825 Mon Sep 17 00:00:00 2001 From: skyflow-vivek Date: Wed, 11 Dec 2024 07:53:00 +0000 Subject: [PATCH 10/10] [AUTOMATED] Private Release 1.15.0-dev-a0537e8 --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index 81c59c51..63dfe263 100644 --- a/pom.xml +++ b/pom.xml @@ -6,7 +6,7 @@ com.skyflow skyflow-java - 1.15.0-dev.d2e7155 + 1.15.0-dev.a0537e8 jar ${project.groupId}:${project.artifactId}