nacx commented on this pull request.


> +   @Fallback(NullOnNotFoundOr404.class)
+   @OAuthResource("https://vault.azure.net";)
+   DeletedKeyBundle deleteKey(@EndpointParam URI vaultBaseUrl, 
@PathParam("keyName") String keyName);
+
+   @Named("key:get_versions")
+   @GET
+   @SelectJson("value")
+   @Path("/keys/{keyName}/versions")
+   @Fallback(EmptyListOnNotFoundOr404.class)
+   @OAuthResource("https://vault.azure.net";)
+   List<Key> getKeyVersions(@EndpointParam URI vaultBaseUrl, 
@PathParam("keyName") String keyName);
+
+   @Named("key:update")
+   @PATCH
+   @MapBinder(BindToJsonPayload.class)
+   @Path("/keys/{keyName}{keyVersion}")

I've seen the same thing in the certificates methods, and then the tests 
building the version parameter as `"/" + version`. Is that intentional?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/416#discussion_r156001516

Reply via email to