removed support for patch API versions and added hard checks on API versions

Author: Puneet Khushwani

lib/ost-sdk-ruby/saas/base.rb

@@ -63,16 +63,6 @@ def sanitize_api_base_url(api_base_url)
 
         api_base_url = api_base_url.gsub(/\/$/, '') # remove trailing slash
 
-        version_str = api_base_url.split('/')[-1]
-
-        return api_base_url unless version_str.include?('v')
-
-        buffer = version_str.split('.')
-        if buffer.length == 3 # dot count in version == 2 ie. version had a patch version ex. v1.0.0
-          regex_match_rsp = /^(.+)(\.\d{1,3})(\/{0,1})$/.match(api_base_url)
-          api_base_url = regex_match_rsp[1]
-        end
-
         return api_base_url
 
       end

lib/ost-sdk-ruby/saas/services.rb

@@ -31,59 +31,29 @@ def initialize(params)
 
       def set_manifest(params)
 
-        int_api_version = extract_integer_api_version(params[:api_base_url])
+        api_version = extract_api_version(params[:api_base_url])
 
         # Provide access to version specific API endpoints
-        if int_api_version >= v1dot2_int_api_version
-          fail 'Unsupported API Version. Please check for new versions of SDK.'
-        elsif int_api_version >= v1dot1_int_api_version
-          @services = OSTSdk::Saas::V1Dot1::Services.new(params)
-        elsif int_api_version >= v1_int_api_version
-          @services = OSTSdk::Saas::V1::Services.new(params)
-        else
+        if api_version == ''
           # puts("You are using an deprecated version of OST API. Please update to the latest version.")
           @services = OSTSdk::Saas::V0::Services.new(params)
+        elsif api_version == 'v1'
+          @services = OSTSdk::Saas::V1::Services.new(params)
+        elsif api_version == 'v1.1'
+          @services = OSTSdk::Saas::V1Dot1::Services.new(params)
+        else
+          fail 'Api endpoint is invalid'
         end
 
       end
 
-      def extract_integer_api_version(api_base_url)
+      def extract_api_version(api_base_url)
 
         api_version = ((api_base_url || '').split("/")[3] || '').downcase
-        return v0_int_api_version if api_version == ''
-
-        regex_match_rsp = /^v(\d{1,3})\.?(\d{0,3})\.?(\*|\d{0,3})$/.match(api_version)
-        fail "invalid version string #{api_version}" if regex_match_rsp.nil?
-
-        int_api_version = 0
-
-        # add API major version
-        int_api_version += regex_match_rsp[1].to_i * 1000000
-
-        # add API minor version
-        int_api_version += (regex_match_rsp[2] == '' ? 0 : regex_match_rsp[2].to_i * 1000)
+        api_major_version = (api_version.split('.')[0] || '')
 
-        # add API Patch Version
-        int_api_version += (regex_match_rsp[3] == '' ? 0 : regex_match_rsp[3].to_i)
-
-        return int_api_version
-
-      end
-
-      def v0_int_api_version
-        0
-      end
-
-      def v1_int_api_version
-        1000000
-      end
-
-      def v1dot1_int_api_version
-        1001000
-      end
+        return api_major_version
 
-      def v1dot2_int_api_version
-        1002000
       end
 
     end