azure-armrest 0.9.2 → 0.9.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2b445a2a57cca396bfb03a3f62393f4c0669f708
-  data.tar.gz: 9a4984a64e5db8057e7c018b8f9730134735dd6a
+  metadata.gz: a842138ac4ccb97028715b1975498aa6f9707aab
+  data.tar.gz: fdcd24eb8de27e193d8871e5899f65c845b8a833
 SHA512:
-  metadata.gz: aefd6f6a4936e5f93e6144f4dec4d841f6335dc65542c0ad00fdb881a3153ca9e2abd163b527633cc4a1eba09495b93917d97f6ecbbf23bd2f01a0c0a6373c0a
-  data.tar.gz: e7338c46f28895160763f8bc1df1b9c0118fa16830435c1fdd90f5d62bdb3bce01595a78e0aa292ea55f8a5887de8a0dfab7dd395ed3e2ba38d63b0f29372167
+  metadata.gz: 6b3f2e25995360728b5963e9a1b3678fd23605450afe2de905646c346cbb45957437f883b85603c50cacd39a05b9b9c43cb689591837cda9b687191f75d691d0
+  data.tar.gz: d247131df9eb986b1d1c8108bd3a538373c8d3662026f9995631329f8270d14092fe56af004767f78bb11c7ea143906c557a21c3a7da3a1393e49f409df41215

data/CHANGES

@@ -1,3 +1,8 @@
+= 0.9.3 - 2-Nov-2017
+* Fixed an issue in the get_by_id and delete_by_id methods where an invalid
+  request string could be generated. This primarily affected the
+  TemplateDeploymentService#delete_associated_resources method.
+
 = 0.9.2 - 31-Oct-2017
 * Fixed a bug in the StorageAccountService#list_private_images method. It
   now accepts an option to skip accessor definitions.
@@ -5,7 +10,7 @@
 * Fixed a bug where an argument to a sleep call could be a string by mistake.
 * Fixed issues with continuation results for list methods in the
   StorageAccount model class.
-* The json dependency restriction was loosened to 1.anything.
+* The json dependency restriction was loosened to 2.anything.
 * The nokogiri dependency was removed. We now rely on Hash#from_xml courtesy
   of activesupport, which we were already using.
 

data/lib/azure/armrest/resource_group_based_service.rb

@@ -132,10 +132,9 @@ module Azure
       #
       def get_by_id(id_string)
         info = parse_id_string(id_string)
-        api_version = api_version_lookup(info['provider'], info['service_name'], info['subservice_name'])
-        service_name = info['subservice_name'] || info['service_name'] || 'resourceGroups'
+        url = convert_id_string_to_url(id_string, info)
 
-        url = File.join(configuration.environment.resource_url, id_string) + "?api-version=#{api_version}"
+        service_name = info['subservice_name'] || info['service_name'] || 'resourceGroups'
 
         model_class = SERVICE_NAME_MAP.fetch(service_name.downcase) do
           raise ArgumentError, "unable to map service name #{service_name} to model"
@@ -147,10 +146,7 @@ module Azure
       alias get_associated_resource get_by_id
 
       def delete_by_id(id_string)
-        info = parse_id_string(id_string)
-        api_version = api_version_lookup(info['provider'], info['service_name'], info['subservice_name'])
-        url = File.join(configuration.environment.resource_url, id_string) + "?api-version=#{api_version}"
-
+        url = convert_id_string_to_url(id_string)
         delete_by_url(url, id_string)
       end
 
@@ -192,6 +188,16 @@ module Azure
 
       private
 
+      def convert_id_string_to_url(id_string, info = nil)
+        if id_string.include?('api-version')
+          File.join(configuration.environment.resource_url, id_string)
+        else
+          info ||= parse_id_string(id_string)
+          api_version = api_version_lookup(info['provider'], info['service_name'], info['subservice_name'])
+          File.join(configuration.environment.resource_url, id_string) + "?api-version=#{api_version}"
+        end
+      end
+
       # Parse the provider and service name out of an ID string.
       def parse_id_string(id_string)
         regex = %r{

data/lib/azure/armrest/version.rb

@@ -1,6 +1,6 @@
 module Azure
   module Armrest
     # The version of the azure-armrest library.
-    VERSION = '0.9.2'.freeze
+    VERSION = '0.9.3'.freeze
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: azure-armrest
 version: !ruby/object:Gem::Version
-  version: 0.9.2
+  version: 0.9.3
 platform: ruby
 authors:
 - Daniel J. Berger
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-31 00:00:00.000000000 Z
+date: 2017-11-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json