mailjet 1.7.11 → 1.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7750315aa012f4278b2ec45974bd121b1046508d8053fee6badd0e5e1ca70b8a
-  data.tar.gz: 19c8881562660013847bab88693e4c16da7e4ed4d570a447b8cb8dcf74a3d202
+  metadata.gz: c9c74309f3d5e557d27fe01548e0998a987c90a9bd1e5e52c187c23d33fb5475
+  data.tar.gz: 3c44b00a89c535c6143d0011fa6e11ca1d6bcdbc82ed984eea4b5f23b0483b89
 SHA512:
-  metadata.gz: 211eec2f72044b08503ac7e987db39d3414d20f3dd42fd2761f4d05e49928c960bd8ff60ab16a1885fedc265b6cd711b7d6e627a13261ff957d527fc3ea1828c
-  data.tar.gz: 9f9f68d47df7a8a4435694a733bd8ef76f6e743987dafaeeddaf476c04283fbc5dd287d8e333a4220099a5d4ee3d9b23a487dd0a84e29a81410b58d8e77400fb
+  metadata.gz: a35a75e7a661f72c28fda90a8403f363252f36cf9573786e512f1721b807b6f450d51dafdc6d8ae8c816b4bd0f32b4abe008f67b6189647a0caa70f0ebbc929d
+  data.tar.gz: 513f1178e1f7e625b177faa7220d60fbb4208a5d38f91642d173ee7a63d39fa3758d189f04e9a4a3e928d84e8d4921a17c22d281bb9fca2b8cf47c7f9f7348ca

data/lib/mailjet/connection.rb

@@ -1,40 +1,32 @@
-require 'rest_client'
+require 'faraday'
 require 'yajl'
 
 module Mailjet
   class Connection
 
-    attr_accessor :adapter, :public_operations, :read_only, :perform_api_call, :read_timeout, :open_timeout
+    attr_accessor :adapter, :public_operations, :read_only, :perform_api_call, :api_key, :secret_key, :options
     alias :read_only? :read_only
 
     def [](suburl, &new_block)
-      broken_url = url.split("/")
+      broken_url = uri.path.split("/")
       if broken_url.include?("contactslist") && broken_url.include?("managemanycontacts") && broken_url.last.to_i > 0
-        self.class.new(url, options[:user], options[:password], options)
+        self.class.new(uri, api_key, secret_key, options)
       else
-        self.class.new(concat_urls(url, suburl), options[:user], options[:password], options)
+        self.class.new(concat_urls(suburl), api_key, secret_key, options)
       end
     end
 
     def initialize(end_point, api_key, secret_key, options = {})
-      # #charles proxy
-      # RestClient.proxy = "http://127.0.0.1:8888"
-      # #
-      # #Output for debugging
-      # RestClient.log =
-      # Object.new.tap do |proxy|
-      #   def proxy.<<(message)
-      #     Rails.logger.info message
-      #   end
-      # end
-      # #
-      adapter_class = options[:adapter_class] || RestClient::Resource
+      self.options = options
+      self.api_key = api_key
+      self.secret_key = secret_key
       self.public_operations = options[:public_operations] || []
       self.read_only = options[:read_only]
-      self.read_timeout = options[:read_timeout]
-      self.open_timeout = options[:open_timeout]
-      # self.adapter = adapter_class.new(end_point, options.merge(user: api_key, password: secret_key, :verify_ssl => false, content_type: 'application/json'))
-      self.adapter = adapter_class.new(end_point, options.merge(user: api_key, password: secret_key, content_type: 'application/json', read_timeout: self.read_timeout, open_timeout: self.open_timeout))
+      self.adapter = Faraday.new(end_point, ssl: { verify: false }) do |conn|
+        conn.response :raise_error, include_request: true
+        conn.request :authorization, :basic, api_key, secret_key
+        conn.headers['Content-Type'] = 'application/json'
+      end
       self.perform_api_call = options.key?(:perform_api_call) ? options[:perform_api_call] : true
     end
 
@@ -54,34 +46,31 @@ module Mailjet
       handle_api_call(:delete, additional_headers, &block)
     end
 
-    def options
-      self.adapter.options
-    end
-
-    def concat_urls(*options)
-      self.adapter.concat_urls(*options)
+    def concat_urls(suburl)
+      self.adapter.build_url(suburl.to_s)
     end
 
-    def url
-      self.adapter.url
+    def uri
+      self.adapter.build_url
     end
 
     private
 
     def handle_api_call(method, additional_headers = {}, payload = {}, &block)
-      formatted_payload = (additional_headers[:content_type] == :json) ? Yajl::Encoder.encode(payload) : payload
+      formatted_payload = (additional_headers["Content-Type"] == 'application/json') ? Yajl::Encoder.encode(payload) : payload
       raise Mailjet::MethodNotAllowed unless method_allowed(method)
 
       if self.perform_api_call
         if [:get, :delete].include?(method)
-          @adapter.send(method, additional_headers, &block)
+          @adapter.send(method, nil, additional_headers[:params], &block)
         else
-          @adapter.send(method, formatted_payload, additional_headers, &block)
+          @adapter.send(method, nil, formatted_payload, additional_headers, &block)
         end
       else
         return Yajl::Encoder.encode({'Count' => 0, 'Data' => [mock_api_call: true], 'Total' => 0})
       end
-    rescue RestClient::Exception => e
+
+    rescue Faraday::Error => e
       handle_exception(e, additional_headers, formatted_payload)
     end
 
@@ -91,20 +80,20 @@ module Mailjet
     end
 
     def handle_exception(e, additional_headers, payload = {})
-      return e.http_body if e.http_headers[:content_type].include?("text/plain")
+      return e.response_body if e.response_headers[:content_type].include?("text/plain")
 
       params = additional_headers[:params] || {}
       formatted_payload = (additional_headers[:content_type] == :json) ? Yajl::Parser.parse(payload) : payload
       params = params.merge!(formatted_payload) if formatted_payload.is_a?(Hash)
 
-      http_body = if e.http_headers[:content_type].include?("application/json")
-        e.http_body
+      response_body = if e.response_headers[:content_type].include?("application/json")
+        e.response_body
       else
         "{}"
       end
 
-      if sent_invalid_email?(e.http_body, @adapter.url)
-        return e.http_body
+      if sent_invalid_email?(e.response_body, @adapter.build_url)
+        return e.response_body
       else
         raise communication_error(e)
       end
@@ -112,20 +101,20 @@ module Mailjet
 
     def communication_error(e)
       if e.respond_to?(:response) && e.response
-        return case e.response.code
+        return case e.response_status
         when Unauthorized::CODE
-          Unauthorized.new(e.message, e.response)
+          Unauthorized.new(e.message, e)
         when BadRequest::CODE
-          BadRequest.new(e.message, e.response)
+          BadRequest.new(e.message, e)
         else
-          CommunicationError.new(e.message, e.response)
+          CommunicationError.new(e.message, e)
         end
       end
       CommunicationError.new(e.message)
     end
 
-    def sent_invalid_email?(error_http_body, url)
-      return false unless url.include?('v3.1/send')
+    def sent_invalid_email?(error_http_body, uri)
+      return false unless uri.path.include?('v3.1/send')
       return unless error_http_body
 
       parsed_body = Yajl::Parser.parse(error_http_body)
@@ -138,6 +127,5 @@ module Mailjet
   end
 
   class MethodNotAllowed < StandardError
-
   end
 end

data/lib/mailjet/exception/errors.rb

@@ -51,13 +51,13 @@ module Mailjet
       @code = if response.nil?
                 NOCODE
               else
-                response.code
+                response.response_status
               end
 
       api_message = begin
-        Yajl::Parser.parse(response.body)['ErrorMessage']
+        Yajl::Parser.parse(response.response_body)['ErrorMessage']
       rescue Yajl::ParseError
-        response.body
+        response.response_body
       rescue NoMethodError
         "Unknown API error"
       rescue

data/lib/mailjet/resource.rb

@@ -15,7 +15,7 @@ require 'active_support/core_ext/hash/indifferent_access'
 module Mailjet
   module Resource
     # define here available options for filtering
-    OPTIONS = [:version, :url, :perform_api_call, :api_key, :secret_key, :read_timeout, :open_timeout]
+    OPTIONS = [:version, :url, :perform_api_call, :api_key, :secret_key]
 
     NON_JSON_URLS = ['v3/send/message'] # urls that don't accept JSON input
     DATA_URLS = ['plain', 'csv'] # url for send binary data , 'CSVError/text:csv'
@@ -39,21 +39,20 @@ module Mailjet
             options[:secret_key] || Mailjet.config.secret_key,
             public_operations: public_operations,
             read_only: read_only,
-            perform_api_call: options[:perform_api_call],
-            open_timeout: options[:open_timeout],
-            read_timeout: options[:read_timeout])
+            perform_api_call: options[:perform_api_call])
         end
 
         def self.default_headers
           if NON_JSON_URLS.include?(self.resource_path) # don't use JSON if Send API
-            default_headers = { accept: :json, accept_encoding: :deflate }
-          elsif DATA_URLS.any? { |data_type| default_headers = { content_type: "text/#{data_type}" } if
-                                                                                self.resource_path.include?(data_type)
-                               }
+            default_headers = { 'Accept' =>'application/json', 'Accept-Encoding' => 'deflate' }
+          elsif DATA_URLS.any? do |data_type|
+            default_headers = { 'Content-Type' => "text/#{data_type}" } if self.resource_path.include?(data_type)
+            end
           else
-            default_headers = { accept: :json, accept_encoding: :deflate, content_type: :json } #use JSON if *not* Send API
+            # use JSON if *not* Send API
+            default_headers = {'Content-Type' =>'application/json', 'Accept' =>'application/json', 'Accept-Encoding' => 'deflate'}
           end
-          return default_headers.merge!(user_agent: "mailjet-api-v3-ruby/#{Gem.loaded_specs["mailjet"].version}")
+          return default_headers.merge!('User-Agent' => "mailjet-api-v3-ruby/#{Gem.loaded_specs["mailjet"].version}")
         end
       end
     end
@@ -69,7 +68,7 @@ module Mailjet
         opts = define_options(options)
         params = format_params(params)
         response = connection(opts).get(default_headers.merge!(params: params))
-        attribute_array = parse_api_json(response)
+        attribute_array = parse_api_json(response.body)
         attribute_array.map{ |attributes| instanciate_from_api(attributes) }
       rescue Mailjet::ApiError => error
         raise error
@@ -78,7 +77,7 @@ module Mailjet
       def count(options = {})
         opts = define_options(options)
         response_json = connection(opts).get(default_headers.merge!(params: {limit: 1, countrecords: 1}))
-        response_hash = Yajl::Parser.parse(response_json)
+        response_hash = Yajl::Parser.parse(response_json.body)
         response_hash['Total']
       rescue Mailjet::ApiError => error
         raise error
@@ -95,7 +94,7 @@ module Mailjet
         opts = define_options(options)
         self.resource_path = create_action_resource_path(normalized_id, job_id) if self.action
 
-        attributes = parse_api_json(connection(opts)[normalized_id].get(default_headers)).first
+        attributes = parse_api_json(connection(opts)[normalized_id].get(default_headers).body).first
         instanciate_from_api(attributes)
 
       rescue Mailjet::CommunicationError => e
@@ -106,6 +105,14 @@ module Mailjet
         end
       end
 
+
+      def find_by_id(id, options = {})
+        # if action method, ammend url to appropriate id
+        opts = define_options(options)
+        self.resource_path = create_action_resource_path(id) if self.action
+        connection(opts).get(default_headers)
+      end
+
       def create(attributes = {}, options = {})
         # if action method, ammend url to appropriate id
         opts = define_options(options)
@@ -139,19 +146,12 @@ module Mailjet
       def send_data(id, binary_data = nil, options = {})
         opts = define_options(options)
         self.resource_path = create_action_resource_path(id) if self.action
+        response = connection(opts).post(binary_data, default_headers.merge({'Content-Length' => "#{binary_data.size}", 'Transfer-Encoding' => 'chunked'}))
 
-        response_hash = Yajl::Parser.parse(connection(opts).post(binary_data, default_headers))
+        response_hash = response.respond_to?(:body) ? Yajl::Parser.parse(response.body) : Yajl::Parser.parse(response)
         response_hash['ID'] ? response_hash['ID'] : response_hash
       end
 
-      def find_by_id(id, options = {})
-        # if action method, ammend url to appropriate id
-        opts = define_options(options)
-        self.resource_path = create_action_resource_path(id) if self.action
-
-        connection(opts).get(default_headers)
-      end
-
       def instanciate_from_api(attributes = {})
         self.new(attributes.merge!(persisted: true))
       end
@@ -284,9 +284,9 @@ module Mailjet
       if opts[:perform_api_call] && !persisted?
         # get attributes only for entity creation
         self.attributes = if self.resource_path == 'send'
-          Yajl::Parser.parse(response)
+          Yajl::Parser.parse(response.body)
         else
-          parse_api_json(response).first
+          parse_api_json(response.body).first
         end
       end
 

data/lib/mailjet/resources/campaigndraft_detailcontent.rb

@@ -11,7 +11,7 @@ module Mailjet
       opts = define_options(options)
       self.resource_path = create_action_resource_path(id, job_id) if self.action
 
-      raw_data = parse_api_json(connection(opts)[id].get(default_headers))
+      raw_data = parse_api_json(connection(opts)[id].get(default_headers).body)
 
       raw_data.map do |entity|
         instanciate_from_api(entity)

data/lib/mailjet/resources/contact_getcontactslists.rb

@@ -11,7 +11,7 @@ module Mailjet
       opts = define_options(options)
       self.resource_path = create_action_resource_path(id, job_id) if self.action
 
-      raw_data = parse_api_json(connection(opts).get(default_headers))
+      raw_data = parse_api_json(connection(opts).get(default_headers).body)
 
       raw_data.map do |entity|
         instanciate_from_api(entity)

data/lib/mailjet/resources/messagehistory.rb

@@ -12,7 +12,7 @@ module Mailjet
       opts = define_options(options)
       self.resource_path = create_action_resource_path(id, job_id) if self.action
 
-      raw_data = parse_api_json(connection(opts)[id].get(default_headers))
+      raw_data = parse_api_json(connection(opts)[id].get(default_headers).body)
 
       raw_data.map do |entity|
         instanciate_from_api(entity)

data/lib/mailjet/resources/messageinformation.rb

@@ -12,7 +12,7 @@ module Mailjet
       opts = define_options(options)
       self.resource_path = create_action_resource_path(id, job_id) if self.action
 
-      raw_data = parse_api_json(connection(opts)[id].get(default_headers))
+      raw_data = parse_api_json(connection(opts)[id].get(default_headers).body)
 
       raw_data.map do |entity|
         instanciate_from_api(entity)

data/lib/mailjet/resources/openinformation.rb

@@ -12,7 +12,7 @@ module Mailjet
       opts = define_options(options)
       self.resource_path = create_action_resource_path(id, job_id) if self.action
 
-      raw_data = parse_api_json(connection(opts)[id].get(default_headers))
+      raw_data = parse_api_json(connection(opts)[id].get(default_headers).body)
 
       raw_data.map do |entity|
         instanciate_from_api(entity)

data/lib/mailjet/resources/template_detailcontent.rb

@@ -12,7 +12,7 @@ module Mailjet
 
       opts = define_options(options)
       response = connection(opts).get(default_headers)
-      attributes = parse_api_json(response).first
+      attributes = parse_api_json(response.body).first
 
       instanciate_from_api(attributes)
     rescue Mailjet::CommunicationError => e

data/lib/mailjet/version.rb

@@ -1,3 +1,3 @@
 module Mailjet
-  VERSION = "1.7.11"
+  VERSION = "1.8.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mailjet
 version: !ruby/object:Gem::Version
-  version: 1.7.11
+  version: 1.8.0
 platform: ruby
 authors:
 - Tyler Nappy
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-05-01 00:00:00.000000000 Z
+date: 2024-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -42,19 +42,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.4.0
 - !ruby/object:Gem::Dependency
-  name: rest-client
+  name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: '2.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: '2.1'
 - !ruby/object:Gem::Dependency
   name: yajl-ruby
   requirement: !ruby/object:Gem::Requirement
@@ -306,7 +306,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.21
+rubygems_version: 3.1.6
 signing_key:
 specification_version: 4
 summary: Mailjet a powerful all-in-one email service provider clients can use to get