vonage 7.20.0 → 8.0.0.beta

data/lib/vonage/client.rb

@@ -61,13 +61,6 @@ module Vonage
       @files ||= T.let(Files.new(config), T.nilable(Vonage::Files))
     end
 
-    # @return [Meetings]
-    #
-    sig { returns(T.nilable(Vonage::Meetings)) }
-    def meetings
-      @meetings ||= T.let(Meetings.new(config), T.nilable(Vonage::Meetings))
-    end
-
     # @return [Messages]
     #
     sig { returns(T.nilable(Vonage::Messages)) }
@@ -89,13 +82,6 @@ module Vonage
       @number_insight ||= T.let(NumberInsight.new(config), T.nilable(Vonage::NumberInsight))
     end
 
-    # @return [NumberInsight2]
-    #
-    sig { returns(T.nilable(Vonage::NumberInsight2)) }
-    def number_insight_2
-      @number_insight_2 ||= T.let(NumberInsight2.new(config), T.nilable(Vonage::NumberInsight2))
-    end
-
     # @return [Numbers]
     #
     sig { returns(T.nilable(Vonage::Numbers)) }
@@ -110,13 +96,6 @@ module Vonage
       @pricing ||= T.let(PricingTypes.new(config), T.nilable(Vonage::PricingTypes))
     end
 
-    # @return [ProactiveConnect]
-    #
-    sig { returns(T.nilable(Vonage::ProactiveConnect)) }
-    def proactive_connect
-      @proactive_connect ||= T.let(ProactiveConnect.new(config), T.nilable(Vonage::ProactiveConnect))
-    end
-
     # @return [Redact]
     #
     sig { returns(T.nilable(Vonage::Redact)) }
@@ -138,13 +117,6 @@ module Vonage
       @sms ||= T.let(SMS.new(config), T.nilable(Vonage::SMS))
     end
 
-    # @return [Subaccounts]
-    #
-    sig { returns(T.nilable(Vonage::Subaccounts)) }
-    def subaccounts
-      @subaccounts ||= T.let(Subaccounts.new(config), T.nilable(Vonage::Subaccounts))
-    end
-
     # @return [TFA]
     #
     sig { returns(T.nilable(Vonage::TFA)) }
@@ -152,13 +124,6 @@ module Vonage
       @tfa ||= T.let(TFA.new(config), T.nilable(Vonage::TFA))
     end
 
-    # @return [Users]
-    #
-    sig { returns(T.nilable(Vonage::Users)) }
-    def users
-      @users ||= T.let(Users.new(config), T.nilable(Vonage::Users))
-    end
-
     # @return [Verify]
     #
     sig { returns(T.nilable(Vonage::Verify)) }
@@ -166,13 +131,6 @@ module Vonage
       @verify ||= T.let(Verify.new(config), T.nilable(Vonage::Verify))
     end
 
-    # @return [Verify2]
-    #
-    sig { returns(T.nilable(Vonage::Verify2)) }
-    def verify2
-      @verify2 ||= T.let(Verify2.new(config), T.nilable(Vonage::Verify2))
-    end
-
     # @return [Video]
     #
     sig { returns(T.nilable(Vonage::Video)) }

data/lib/vonage/client_error.rb

@@ -1,6 +1,6 @@
 # typed: strong
 
 module Vonage
-  class ClientError < APIError
+  class ClientError < Error
   end
 end

data/lib/vonage/config.rb

@@ -19,7 +19,6 @@ module Vonage
       self.signature_method = ENV['VONAGE_SIGNATURE_METHOD'] || 'md5hash'
       self.token = T.let(nil, T.nilable(String))
       self.video_host = 'video.api.vonage.com'
-      self.vonage_host = 'api-eu.vonage.com'
     end
 
     # Merges the config with the given options hash.
@@ -131,11 +130,7 @@ module Vonage
 
     # @return [Vonage::Logger]
     #
-    sig { params(logger: T.nilable(
-      defined?(ActiveSupport::BroadcastLogger) ?
-        T.any(::Logger, Vonage::Logger, ActiveSupport::BroadcastLogger)
-      : T.any(::Logger, Vonage::Logger)
-    )).returns(T.nilable(Vonage::Logger)) }
+    sig { params(logger: T.nilable(T.any(::Logger, Vonage::Logger))).returns(T.nilable(Vonage::Logger)) }
     def logger=(logger)
       @logger = T.let(Logger.new(logger), T.nilable(Vonage::Logger))
     end
@@ -206,9 +201,6 @@ module Vonage
 
     sig { returns(String) }
     attr_accessor :video_host
-    
-    sig { returns(String) }
-    attr_accessor :vonage_host
 
     protected
 

data/lib/vonage/errors.rb

@@ -1,25 +1,14 @@
 # typed: strict
 # frozen_string_literal: true
-require "json"
+require 'json'
 
 module Vonage
   module Errors
     extend T::Sig
 
-    sig do
-      params(
-        response:
-          T.any(
-            Net::HTTPUnauthorized,
-            Net::HTTPClientError,
-            Net::HTTPServerError,
-            T.untyped
-          )
-      ).returns(Vonage::Error)
-    end
+    sig {params(response: T.any(Net::HTTPUnauthorized, Net::HTTPClientError, Net::HTTPServerError, T.untyped)).returns(Vonage::Error)}
     def self.parse(response)
-      exception_class =
-        case response
+      exception_class = case response
         when Net::HTTPUnauthorized
           AuthenticationError
         when Net::HTTPClientError
@@ -27,40 +16,34 @@ module Vonage
         when Net::HTTPServerError
           ServerError
         else
-          APIError
+          Error
         end
 
-      message = response.content_type.to_s.include?("json") ? set_message(response) : ""
-
-      exception_class.new(message, http_response: response)
-    end
-
-    def self.set_message(response)
-      hash = ::JSON.parse(response.body)
-
-      if hash.key?("error_title")
-        hash["error_title"]
-      elsif hash.key?("error-code-label")
-        hash["error-code-label"]
-      elsif hash.key?("description")
-        hash["description"]
-      elsif hash.key?("message")
-        hash["message"]
-      elsif problem_details?(hash)
-        problem_details_message(hash)
-      else
-        ""
+      message = if response.content_type == 'application/json'
+        hash = ::JSON.parse(response.body)
+
+        if hash.key?('error_title')
+          hash['error_title']
+        elsif hash.key?('error-code-label')
+          hash['error-code-label']
+        elsif hash.key?('description')
+          hash['description']
+        elsif problem_details?(hash)
+          problem_details_message(hash)
+        end
       end
+
+      exception_class.new(message)
     end
 
     sig { params(hash: T::Hash[String, T.untyped]).returns(T::Boolean) }
     def self.problem_details?(hash)
-      hash.key?("title") && hash.key?("detail") && hash.key?("type")
+      hash.key?('title') && hash.key?('detail') && hash.key?('type')
     end
 
     sig { params(hash: T::Hash[String, T.untyped]).returns(String) }
     def self.problem_details_message(hash)
-      "#{hash["title"]}. #{hash["detail"]} See #{hash["type"]} for more info, or email support@vonage.com if you have any questions."
+      "#{hash['title']}. #{hash['detail']} See #{hash['type']} for more info, or email support@nexmo.com if you have any questions."
     end
   end
 

data/lib/vonage/gsm7.rb

@@ -4,7 +4,7 @@ module Vonage
   module GSM7
     extend T::Sig
     
-    CHARACTERS = "\n\f\r !\"\#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\\]^_abcdefghijklmnopqrstuvwxyz{|}~ ¡£¤¥§¿ÄÅÆÇÉÑÖØÜßàäåæèéìñòöøùüΓΔΘΛΞΠΣΦΨΩ€"
+    CHARACTERS = "\n\f\r !\"\#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\\]^_abcdefghijklmnopqrstuvwxyz{|}~ ¡£¤¥§¿ÄÅÆÉÑÖØÜßàäåæçèéìñòöøùüΓΔΘΛΞΠΣΦΨΩ€"
 
     REGEXP = /\A[#{Regexp.escape(CHARACTERS)}]*\z/
 

data/lib/vonage/jwt.rb

@@ -39,22 +39,5 @@ module Vonage
       payload[:private_key] = private_key if private_key && !payload[:private_key]
       @token = Vonage::JWTBuilder.new(payload).jwt.generate
     end
-
-    # Validate a JSON Web Token from a Vonage Webhook.
-    #
-    # Certain Vonage APIs include a JWT signed with a user's account signature secret in
-    # the Authorization header of Webhook requests. This method can be used to verify that those requests originate
-    # from the Vonage API.
-    #
-    # @param [String, required] :token The JWT from the Webhook's Authorization header
-    # @param [String, required] :signature_secret The account signature secret
-    #
-    # @return [Boolean] true, if the JWT is verified, false otherwise
-    #
-    # @see https://developer.vonage.com/en/getting-started/concepts/webhooks#decoding-signed-webhooks
-    #
-    def self.verify_hs256_signature(token:, signature_secret:)
-      verify_signature(token, signature_secret, 'HS256')
-    end
   end
 end

data/lib/vonage/keys.rb

@@ -27,8 +27,6 @@ module Vonage
         'has_video',
         'remove_stream',
         'screenshare_type',
-        'session_id',
-        'stream_mode',
         'archive_mode'
       ]
       hash.transform_keys do |k|

data/lib/vonage/logger.rb

@@ -7,11 +7,7 @@ module Vonage
   class Logger
     extend T::Sig
 
-    sig { params(logger: T.nilable(
-      defined?(ActiveSupport::BroadcastLogger) ?
-        T.any(::Logger, Vonage::Logger, ActiveSupport::BroadcastLogger)
-      : T.any(::Logger, Vonage::Logger)
-    )).void }
+    sig { params(logger: T.nilable(T.any(::Logger, Vonage::Logger))).void }
     def initialize(logger)
       @logger = logger || ::Logger.new(nil)
     end
@@ -24,6 +20,8 @@ module Vonage
 
     sig { params(request: T.any(Net::HTTP::Post, Net::HTTP::Get, Net::HTTP::Delete, Net::HTTP::Put, Net::HTTP::Patch)).void }
     def log_request_info(request)
+      @logger = T.let(@logger, T.nilable(T.any(::Logger, Vonage::Logger)))
+
       T.must(@logger).info do
         format('Vonage API request', {
           method: request.method,

data/lib/vonage/messaging/channels/viber.rb

@@ -2,7 +2,7 @@
 
 module Vonage
   class Messaging::Channels::Viber  < Messaging::Message
-    MESSAGE_TYPES = ['text', 'image', 'video', 'file']
+    MESSAGE_TYPES = ['text', 'image']
 
     attr_reader :data
 
@@ -33,13 +33,6 @@ module Vonage
       when 'image'
         raise Vonage::ClientError.new(":message must be a Hash") unless message.is_a? Hash
         raise Vonage::ClientError.new(":url is required in :message") unless message[:url]
-      when 'video'
-        raise Vonage::ClientError.new(":message must be a Hash") unless message.is_a? Hash
-        raise Vonage::ClientError.new(":url is required in :message") unless message[:url]
-        raise Vonage::ClientError.new(":thumb_url is required in :message") unless message[:thumb_url]
-      when 'file'
-        raise Vonage::ClientError.new(":message must be a Hash") unless message.is_a? Hash
-        raise Vonage::ClientError.new(":url is required in :message") unless message[:url]
       end
     end
   end

data/lib/vonage/messaging/channels/whats_app.rb

@@ -2,7 +2,7 @@
 
 module Vonage
   class Messaging::Channels::WhatsApp < Messaging::Message
-    MESSAGE_TYPES = ['text', 'image', 'audio', 'video', 'file', 'template', 'sticker', 'custom']
+    MESSAGE_TYPES = ['text', 'image', 'audio', 'video', 'file', 'template', 'custom']
 
     attr_reader :data
 
@@ -35,8 +35,6 @@ module Vonage
         raise Vonage::ClientError.new(":name is required in :template") unless message[:name]
         raise Vonage::ClientError.new(":whatsapp is required in :opts") unless opts[:whatsapp]
         raise Vonage::ClientError.new(":locale is required in :whatsapp") unless opts[:whatsapp][:locale]
-      when 'sticker'
-        raise Vonage::ClientError.new(":message must contain either :id or :url") unless message.has_key?(:id) ^ message.has_key?(:url)
       when 'custom'
         raise Vonage::ClientError.new(":message must be a Hash") unless message.is_a? Hash
       else

data/lib/vonage/messaging.rb

@@ -25,16 +25,5 @@ module Vonage
     def send(params)
       request('/v1/messages', params: params, type: Post)
     end
-
-    # Validate a JSON Web Token from a Messages API Webhook.
-    #
-    # @param [String, required] :token The JWT from the Webhook's Authorization header
-    # @param [String, optional] :signature_secret The account signature secret. Required, unless `signature_secret`
-    #   is set in `Config`
-    #
-    # @return [Boolean] true, if the JWT is verified, false otherwise
-    def verify_webhook_token(token:, signature_secret: @config.signature_secret)
-      JWT.verify_hs256_signature(token: token, signature_secret: signature_secret)
-    end
   end
 end

data/lib/vonage/namespace.rb

@@ -1,8 +1,6 @@
 # typed: true
 # frozen_string_literal: true
-
 require 'net/http'
-require 'net/http/post/multipart'
 require 'json'
 
 module Vonage
@@ -23,7 +21,8 @@ module Vonage
     end
 
     def self.host=(host)
-      raise ArgumentError unless %i[rest_host video_host vonage_host].include?(host)
+      raise ArgumentError unless host == :rest_host || host == :video_host
+
       @host = host
     end
 
@@ -48,7 +47,6 @@ module Vonage
     end
 
     protected
-
     # :nocov:
 
     Get = Net::HTTP::Get
@@ -61,7 +59,7 @@ module Vonage
       authentication = self.class.authentication.new(@config)
       authentication.update(params)
 
-      uri = URI("https://" + @host + path)
+      uri = URI('https://' + @host + path)
       unless type.const_get(:REQUEST_HAS_BODY) || params.empty?
         uri.query = Params.encode(params)
       end
@@ -73,20 +71,17 @@ module Vonage
       request = type.new(uri)
 
       # set headers
-      request["User-Agent"] = UserAgent.string(
-        @config.app_name,
-        @config.app_version
-      )
-      request["Accept"] = "application/json"
-      self.class.request_headers.each { |key, value| request[key] = value }
+      request['User-Agent'] = UserAgent.string(@config.app_name, @config.app_version)
+      request['Accept'] = 'application/json'
+      self.class.request_headers.each do |key, value|
+        request[key] = value
+      end
 
       # Set BearerToken if needed
       authentication.update(request)
 
       # set body
-      if type.const_get(:REQUEST_HAS_BODY)
-        self.class.request_body.update(request, params)
-      end
+      self.class.request_body.update(request, params) if type.const_get(:REQUEST_HAS_BODY)
 
       request
     end
@@ -106,31 +101,16 @@ module Vonage
     end
 
     def request(path, params: nil, type: Get, response_class: Response, &block)
-      auto_advance =
-        (
-          if !params.nil? && params.key?(:auto_advance)
-            params[:auto_advance]
-          else
-            false
-          end
-        )
-
-      params =
-        params.tap { |params| params.delete(:auto_advance) } if !params.nil? &&
-        params.key?(:auto_advance)
+      auto_advance = !params.nil? && params.key?(:auto_advance) ? params[:auto_advance] : false
+
+      params = params.tap { |params| params.delete(:auto_advance) } if !params.nil? && params.key?(:auto_advance)
 
       request = build_request(path: path, params: params || {}, type: type)
 
       response = make_request!(request, &block)
 
       if auto_advance
-        iterable_request(
-          path,
-          response: response,
-          response_class: response_class,
-          params: params,
-          &block
-        )
+        iterable_request(path, response: response, response_class: response_class, params: params, &block)
       else
         return if block
 
@@ -138,59 +118,22 @@ module Vonage
       end
     end
 
-    def multipart_post_request(path, filepath:, file_name:, mime_type:, params: {}, override_uri: nil, no_auth: false, response_class: Response, &block)
-      authentication = self.class.authentication.new(@config) unless no_auth
-
-      uri = override_uri ? URI(override_uri) : URI('https://' + @host + path)
-
-      http = override_uri ? Net::HTTP.new(uri.host, Net::HTTP.https_default_port, p_addr = nil) : @http
-      http.use_ssl = true
-      http.set_debug_output($stdout)
-
-      response = File.open(filepath) do |file|
-        request = Net::HTTP::Post::Multipart.new(
-          uri,
-          params.merge(file: Multipart::Post::UploadIO.new(file, mime_type, file_name))
-        )
-
-        request['User-Agent'] = UserAgent.string(@config.app_name, @config.app_version)
-
-        # Set BearerToken if needed
-        authentication.update(request) unless no_auth
-
-        logger.log_request_info(request)
-
-        http.request(request, &block)
-      end
-
-      logger.log_response_info(response, @host)
-
-      return if block
-
-      logger.debug(response.body) if response.body
-
-      parse(response, response_class)
-    end
-
     def iterable_request(path, response: nil, response_class: nil, params: {}, &block)
       json_response = ::JSON.parse(response.body)
       response = parse(response, response_class)
       remainder = remaining_count(json_response)
 
       while remainder > 0
-        params = { page_size: json_response["page_size"] }
-
-        if json_response["record_index"] && json_response["record_index"] == 0
-          params[:record_index] = json_response["page_size"]
-        elsif json_response["record_index"] &&
-              json_response["record_index"] != 0
-          params[:record_index] = (
-            json_response["record_index"] + json_response["page_size"]
-          )
+        params = { page_size: json_response['page_size'] }
+
+        if json_response['record_index'] && json_response['record_index'] == 0
+          params[:record_index] = json_response['page_size']
+        elsif json_response['record_index'] && json_response['record_index'] != 0
+          params[:record_index] = (json_response['record_index'] + json_response['page_size'])
         end
 
-        if json_response["total_pages"]
-          params[:page] = json_response["page"] + 1
+        if json_response['total_pages']
+          params[:page] = json_response['page'] + 1
         end
 
         request = build_request(path: path, type: Get, params: params)
@@ -201,15 +144,11 @@ module Vonage
         json_response = ::JSON.parse(paginated_response.body)
         remainder = remaining_count(json_response)
 
-        if response.respond_to?("_embedded")
-          collection_name = collection_name(response["_embedded"])
-          response["_embedded"][collection_name].push(
-            *next_response["_embedded"][collection_name]
-          )
+        if response.respond_to?('_embedded')
+          collection_name = collection_name(response['_embedded'])
+          response['_embedded'][collection_name].push(*next_response['_embedded'][collection_name])
         else
-          response[collection_name(response)].push(
-            *next_response[collection_name(next_response)]
-          )
+          response[collection_name(response)].push(*next_response[collection_name(next_response)])
         end
       end
 
@@ -217,51 +156,43 @@ module Vonage
     end
 
     def remaining_count(params)
-      if params.key?("total_pages")
-        params["total_pages"] - params["page"]
-      elsif params.key?("count")
-        params["count"] -
-          (
-            if params["record_index"] == 0
-              params["page_size"]
-            else
-              (params["record_index"] + params["page_size"])
-            end
-          )
+      if params.key?('total_pages')
+        params['total_pages'] - params['page']
+      elsif params.key?('count')
+        params['count'] - (params['record_index'] == 0 ? params['page_size'] : (params['record_index'] + params['page_size']))
       else
         0
       end
     end
 
     def collection_name(params)
-      @collection_name ||=
-        case
-        when params.respond_to?("calls")
-          "calls"
-        when params.respond_to?("users")
-          "users"
-        when params.respond_to?("legs")
-          "legs"
-        when params.respond_to?("data")
-          "data"
-        when params.respond_to?("conversations")
-          "conversations"
-        when params.respond_to?("applications")
-          "applications"
-        when params.respond_to?("records")
-          "records"
-        when params.respond_to?("reports")
-          "reports"
-        when params.respond_to?("networks")
-          "networks"
-        when params.respond_to?("countries")
-          "countries"
-        when params.respond_to?("media")
-          "media"
-        when params.respond_to?("numbers")
-          "numbers"
-        when params.respond_to?("events")
-          "events"
+      @collection_name ||= case
+        when params.respond_to?('calls')
+          'calls'
+        when params.respond_to?('users')
+          'users'
+        when params.respond_to?('legs')
+          'legs'
+        when params.respond_to?('data')
+          'data'
+        when params.respond_to?('conversations')
+          'conversations'
+        when params.respond_to?('applications')
+          'applications'
+        when params.respond_to?('records')
+          'records'
+        when params.respond_to?('reports')
+          'reports'
+        when params.respond_to?('networks')
+          'networks'
+        when params.respond_to?('countries')
+          'countries'
+        when params.respond_to?('media')
+          'media'
+        when params.respond_to?('numbers')
+          'numbers'
+        when params.respond_to?('events')
+          'events'
         else
           params.entity.attributes.keys[0].to_s
         end
@@ -272,7 +203,7 @@ module Vonage
       when Net::HTTPNoContent
         response_class.new(nil, response)
       when Net::HTTPSuccess
-        if response['Content-Type'] && response['Content-Type'].split(';').first == 'application/json' && !response.body.empty?
+        if response['Content-Type'].split(';').first == 'application/json'
           entity = ::JSON.parse(response.body, object_class: Vonage::Entity)
 
           response_class.new(entity, response)
@@ -296,8 +227,6 @@ module Vonage
         @config.rest_host
       when :video_host
         @config.video_host
-      when :vonage_host
-        @config.vonage_host
       else
         @config.api_host
       end