vonage 7.27.0 → 8.0.0.beta

data/lib/vonage/keys.rb

@@ -27,17 +27,7 @@ module Vonage
         'has_video',
         'remove_stream',
         'screenshare_type',
-        'session_id',
-        'stream_mode',
-        'archive_mode',
-        'language_code',
-        'max_duration',
-        'partial_captions',
-        'status_callback_url',
-        'audio_rate',
-        'phone_number',
-        'hashed_phone_number',
-        'max_age'
+        'archive_mode'
       ]
       hash.transform_keys do |k|
         if exceptions.include?(k.to_s)

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/message.rb

@@ -5,7 +5,6 @@ module Vonage
     CHANNELS = {
       sms: Vonage::Messaging::Channels::SMS,
       mms: Vonage::Messaging::Channels::MMS,
-      rcs: Vonage::Messaging::Channels::RCS,
       whatsapp: Vonage::Messaging::Channels::WhatsApp,
       messenger: Vonage::Messaging::Channels::Messenger,
       viber: Vonage::Messaging::Channels::Viber

data/lib/vonage/messaging.rb

@@ -1,21 +1,16 @@
 # typed: true
 # frozen_string_literal: true
-require 'forwardable'
 
 module Vonage
   class Messaging < Namespace
-    extend Forwardable
-
     self.authentication = BearerToken
 
     self.request_body = JSON
 
-    def_delegators Message, *Message::CHANNELS.keys
-
     # Send a Message.
     #
     # @example
-    #   message = client.messaging.sms(message: "Hello world!")
+    #   message = Vonage::Messaging::Message.sms(message: "Hello world!")
     #   response = client.messaging.send(to: "447700900000", from: "447700900001", **message)
     #
     # @option params [required, String] :to
@@ -25,35 +20,10 @@ module Vonage
     # @option params [required, Hash] **message
     #   The Vonage Message object to use for this message.
     #
-    # @see https://developer.vonage.com/api/messages#SendMessage
-    #
-    def send(to:, from:, **message)
-      request('/v1/messages', params: {to: to, from: from, **message}, type: Post)
-    end
-
-    # Update a Message Object.
-    #
-    # @example
-    #   message = client.messaging.update(message_uuid: "aaaaaaaa-bbbb-4ccc-8ddd-0123456789ab", status: "read")
-    #
-    # @option params [required, String] :message_uuid. the UUID of the message to update.
-    #
-    # `:message_uuid` is always required. Other parameters will depend on the message channel and the specific action being performed.
-    # @see https://developer.vonage.com/api/messages#UpdateMessage
-    #
-    def update(message_uuid:, **params)
-      request("/v1/messages/#{message_uuid}", params: params, type: Patch)
-    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`
+    # @see https://developer.vonage.com/api/messages-olympus#SendMessage
     #
-    # @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)
+    def send(params)
+      request('/v1/messages', params: params, type: Post)
     end
   end
 end

data/lib/vonage/namespace.rb

@@ -1,9 +1,6 @@
 # typed: true
 # frozen_string_literal: true
-
 require 'net/http'
-require 'net/http/persistent'
-require 'net/http/post/multipart'
 require 'json'
 
 module Vonage
@@ -13,7 +10,8 @@ module Vonage
 
       @host = set_host
 
-      @http = Net::HTTP::Persistent.new
+      @http = Net::HTTP.new(@host, Net::HTTP.https_default_port, p_addr = nil)
+      @http.use_ssl = true
 
       @config.http.set(@http) unless @config.http.nil?
     end
@@ -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
@@ -57,33 +55,33 @@ module Vonage
     Post = Net::HTTP::Post
     Delete = Net::HTTP::Delete
 
-    def build_request(path:, type: Get, params: {}, auth_data: nil)
+    def build_request(path:, type: Get, params: {})
       authentication = self.class.authentication.new(@config)
-      authentication.update(params, auth_data)
+      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
 
+      # Set BasicAuth if neeeded
+      authentication.update(uri)
+
       # instantiate request
       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 Authorization header if needed
-      authentication.update(request, auth_data)
+      # 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
@@ -91,8 +89,7 @@ module Vonage
     def make_request!(request, &block)
       logger.log_request_info(request)
 
-      uri = URI("https://" + @host + request.path)
-      response = @http.request(uri, request, &block)
+      response = @http.request(request, &block)
 
       logger.log_response_info(response, @host)
 
@@ -103,32 +100,17 @@ module Vonage
       response
     end
 
-    def request(path, params: nil, type: Get, response_class: Response, auth_data: nil, &block)
-      auto_advance =
-        (
-          if !params.nil? && params.key?(:auto_advance)
-            params[:auto_advance]
-          else
-            false
-          end
-        )
+    def request(path, params: nil, type: Get, response_class: Response, &block)
+      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)
+      params = params.tap { |params| params.delete(:auto_advance) } if !params.nil? && params.key?(:auto_advance)
 
-      request = build_request(path: path, params: params || {}, type: type, auth_data: auth_data)
+      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
 
@@ -136,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, auth_data: nil, &block)
-      authentication = self.class.authentication.new(@config) unless no_auth
-
-      uri = override_uri ? URI(override_uri) : URI('https://' + @host + path)
-
-      http = Net::HTTP.new(uri.host, Net::HTTP.https_default_port, p_addr = nil)
-      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 Authorization header if needed
-        authentication.update(request, auth_data) 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)
@@ -199,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
 
@@ -215,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
@@ -270,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)
@@ -294,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

data/lib/vonage/numbers.rb

@@ -5,8 +5,6 @@ module Vonage
   class Numbers < Namespace
     include Keys
 
-    self.authentication = Basic
-
     self.host = :rest_host
 
     # Retrieve all the inbound numbers associated with your Vonage account.
@@ -17,8 +15,6 @@ module Vonage
     #     puts "#{item.msisdn} #{item.country} #{item.type}"
     #   end
     #
-    # @param [Hash] params
-    #
     # @option params [String] :application_id
     #   The application that you want to return the numbers for.
     #
@@ -49,12 +45,14 @@ module Vonage
     #   Set this to `true` to auto-advance through all the pages in the record
     #   and collect all the data. The default is `false`.
     #
+    # @param [Hash] params
+    #
     # @return [ListResponse]
     #
     # @see https://developer.nexmo.com/api/developer/numbers#getOwnedNumbers
     #
     def list(params = nil)
-      request("/account/numbers", params: params, response_class: ListResponse)
+      request('/account/numbers', params: params, response_class: ListResponse)
     end
 
     # Retrieve inbound numbers that are available for the specified country.
@@ -65,8 +63,6 @@ module Vonage
     #     puts "#{item.msisdn} #{item.type} #{item.cost}"
     #   end
     #
-    # @param [Hash] params
-    #
     # @option params [required, String] :country
     #   The two character country code in ISO 3166-1 alpha-2 format.
     #
@@ -84,7 +80,7 @@ module Vonage
     #   - `2` - Search for numbers that end with **:pattern**
     #
     # @option params [String] :features
-    #   Available features are `SMS`, `MMS`, and `VOICE`.
+    #   Available features are `SMS` and `VOICE`.
     #   To look for numbers that support both, use a comma-separated value: `SMS,VOICE`.
     #
     # @option params [Integer] :size
@@ -97,12 +93,14 @@ module Vonage
     #   Set this to `true` to auto-advance through all the pages in the record
     #   and collect all the data. The default is `false`.
     #
+    # @param [Hash] params
+    #
     # @return [ListResponse]
     #
     # @see https://developer.nexmo.com/api/developer/numbers#getAvailableNumbers
     #
     def search(params)
-      request("/number/search", params: params, response_class: ListResponse)
+      request('/number/search', params: params, response_class: ListResponse)
     end
 
     # Request to purchase a specific inbound number.
@@ -110,8 +108,6 @@ module Vonage
     # @example
     #   response = client.numbers.buy(country: 'GB', msisdn: '447700900000')
     #
-    # @param [Hash] params
-    #
     # @option params [required, String] :country
     #   The two character country code in ISO 3166-1 alpha-2 format.
     #
@@ -122,17 +118,14 @@ module Vonage
     #   If you'd like to perform an action on a subaccount, provide the `api_key` of that account here.
     #   If you'd like to perform an action on your own account, you do not need to provide this field.
     #
+    # @param [Hash] params
+    #
     # @return [Response]
     #
     # @see https://developer.nexmo.com/api/developer/numbers#buyANumber
     #
     def buy(params)
-      request(
-        "/number/buy",
-        params: params,
-        type: Post,
-        response_class: Response
-      )
+      request('/number/buy', params: params, type: Post, response_class: Response)
     end
 
     # Cancel your subscription for a specific inbound number.
@@ -140,8 +133,6 @@ module Vonage
     # @example
     #   response = client.numbers.cancel(country: 'GB', msisdn: '447700900000')
     #
-    # @param [Hash] params
-    #
     # @option params [required, String] :country
     #   The two character country code in ISO 3166-1 alpha-2 format.
     #
@@ -152,17 +143,14 @@ module Vonage
     #   If you'd like to perform an action on a subaccount, provide the `api_key` of that account here.
     #   If you'd like to perform an action on your own account, you do not need to provide this field.
     #
+    # @param [Hash] params
+    #
     # @return [Response]
     #
     # @see https://developer.nexmo.com/api/developer/numbers#cancelANumber
     #
     def cancel(params)
-      request(
-        "/number/cancel",
-        params: params,
-        type: Post,
-        response_class: Response
-      )
+      request('/number/cancel', params: params, type: Post, response_class: Response)
     end
 
     # Change the behaviour of a number that you own.
@@ -177,8 +165,6 @@ module Vonage
     #
     #   response = client.numbers.update(params)
     #
-    # @param [Hash] params
-    #
     # @option params [required, String] :country
     #   The two character country code in ISO 3166-1 alpha-2 format.
     #
@@ -205,30 +191,21 @@ module Vonage
     # @option params [String] :voice_status_callback
     #   A webhook URI for Vonage to send a request to when a call ends.
     #
+    # @param [Hash] params
+    #
     # @return [Response]
     #
     # @see https://developer.nexmo.com/api/developer/numbers#updateANumber
     #
     def update(params)
-      request(
-        "/number/update",
-        params: camelcase(params),
-        type: Post,
-        response_class: Response
-      )
+      request('/number/update', params: camelcase(params), type: Post, response_class: Response)
     end
 
     private
 
     # A specific implementation of iterable_request for Numbers, because the Numbers API
     # handles pagination differently to other Vonage APIs
-    def iterable_request(
-      path,
-      response: nil,
-      response_class: nil,
-      params: {},
-      &block
-    )
+    def iterable_request(path, response: nil, response_class: nil, params: {}, &block)
       response = parse(response, response_class)
       params[:index] = 1 unless params.has_key?(:index)
       size = params.fetch(:size, 10)

data/lib/vonage/server_error.rb

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