Merge pull request BetterCloud#163 from goodrum/fix/lease-revoke-url

FIX: Vault::Api::Leases url to match Hashicorp Vault API
rimitchell · Jun 4, 2019 · aa54dc8 · aa54dc8
2 parents 0732a5f + d42ef45
commit aa54dc8
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/main/java/com/bettercloud/vault/api/Leases.java b/src/main/java/com/bettercloud/vault/api/Leases.java
@@ -54,8 +54,14 @@ public VaultResponse revoke(final String leaseId) throws VaultException {
         int retryCount = 0;
         while (true) {
             try {
+                /**
+                * 2019-03-21
+                * Changed the Lease revoke url due to invalid path.  Vault deprecated the original
+                * path (/v1/sys/revoke) in favor of a new leases mount point (/v1/sys/leases/revoke)
+                * https://github.com/hashicorp/vault/blob/master/CHANGELOG.md#080-august-9th-2017
+                */
                 final RestResponse restResponse = new Rest()//NOPMD
-                        .url(config.getAddress() + "/v1/sys/revoke/" + leaseId)
+                        .url(config.getAddress() + "/v1/sys/leases/revoke/" + leaseId)
                         .header("X-Vault-Token", config.getToken())
                         .optionalHeader("X-Vault-Namespace", this.nameSpace)
                         .connectTimeoutSeconds(config.getOpenTimeout())