micky 0.6.9 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f9be790e9b69f56620b27af765fcdd4230b57d5164ac5f24374fc168986db291
-  data.tar.gz: eb604f6cefd295336b46903d050d617108a943ef75d0a7fe1c57acb021f773a5
+  metadata.gz: 898140acbc602cbf145536a9370e9a1bda8806d5bced808576e58d55a583ac4e
+  data.tar.gz: 274bb288eeb2789f70f7104a272aeb8f5f052f9239f4e41fbf50c59553c391a2
 SHA512:
-  metadata.gz: ebb52a2e4a80055e47f31e40a7acbe00c1f15d025fe2c38c986dea4d2d456fe78771a9b3d0f483443521051d7f6f6059a1385e295be2f105f777d41dd6d36692
-  data.tar.gz: 65b29b7898f38eaa2ecff1538b80070dfe03bc108e712f8b9662591ec3f812dcd0866fa57a443124c51c8698f1b7050a5aac005349d8e0ffd7d9eaa3a006bf40
+  metadata.gz: 5bb4814243602605eedd5a2b17386e8f8db6845bfda32ff44f2bc9aa9173ebb00c62a656180e11b09ab4835ac7fe5b421666711a84d0df74ab28c2ea2ff0098b
+  data.tar.gz: c7dfb2acabcc9d1c5fc2f86d33cc387f05c7f32987f384607431cc3fef70047cc6c6a1474a04ddbb13b92435948ef223a8d3d5495e15d74010c5ccb105bd83da

data/lib/micky/request.rb

@@ -5,7 +5,7 @@ module Micky
   class Request
     def initialize(opts = {})
       # Options can be set per request and fallback to module-level defaults
-      [:raise_errors, :max_redirects, :timeout, :skip_resolve, :resolve_timeout, :oauth, :query, :headers, :parsers].each do |name|
+      DEFAULTS.each_key do |name|
         value = opts.has_key?(name) ? opts[name] : Micky.public_send(name)
         instance_variable_set "@#{name}", value
       end
@@ -32,6 +32,7 @@ module Micky
 
       case response = request(uri)
       when Net::HTTPSuccess
+        debug "#{response.code} success"
         Response.new(response, @uri)
       when Net::HTTPRedirection
         previous_uri = uri
@@ -39,7 +40,7 @@ module Micky
 
         if uri.nil?
           raise Micky::NoRedirectLocation, response: response if @raise_errors
-          warn "Micky.#{@request_class_name.downcase}('#{previous_uri}'): No “Location” for #{response.code} response"
+          log "No “Location” for #{response.code} response", previous_uri
           return nil
         end
 
@@ -59,7 +60,7 @@ module Micky
           end
         end
 
-        log "Redirect to #{uri}"
+        debug "#{response.code} redirect to #{uri}"
         request_with_redirect_handling(uri, redirect_count + 1)
       else
         if @raise_errors
@@ -70,7 +71,7 @@ module Micky
             raise Micky::HTTPServerError, response: response
           end
         else
-          log response
+          log "#{response.code} error" if response
           nil
         end
       end
@@ -79,7 +80,7 @@ module Micky
     def request(uri)
       @uri = Micky::URI(uri) or begin
         raise Micky::InvalidURIError, uri if @raise_errors
-        warn "Micky.#{@request_class_name.downcase}('#{uri}'): Invalid URI"
+        log 'Invalid URI', uri
         return nil
       end
 
@@ -159,9 +160,13 @@ module Micky
       end
     end
 
-    def log(message)
+    def log(message, uri = @uri, severity = Logger::WARN)
       message = "#{message.class}: #{message.message}" if message.is_a? Exception
-      warn "Micky.#{@request_class_name.downcase}('#{@uri}'): #{message}"
+      @logger.add(severity, "[Micky] #{@request_class_name.upcase} #{uri} - #{message}")
+    end
+
+    def debug(message, uri = @uri)
+      log(message, uri, Logger::DEBUG)
     end
   end
 end

data/lib/micky/uri.rb

@@ -16,7 +16,7 @@ module Micky
       ::URI.parse(uri)
     rescue ::URI::InvalidURIError
       begin
-        ::URI.parse(::URI.encode(uri))
+        ::URI.parse(::URI::Parser.new.escape(uri))
       rescue ::URI::InvalidURIError
       end
     end

data/lib/micky/version.rb

@@ -1,3 +1,3 @@
 module Micky
-  VERSION = '0.6.9'
+  VERSION = '0.7.0'
 end

data/lib/micky.rb

@@ -5,34 +5,36 @@ require_relative 'micky/request'
 require_relative 'micky/response'
 require_relative 'micky/errors'
 
+require 'logger'
+
 module Micky
+  DEFAULTS = {
+    raise_errors: false,
+    max_redirects: 20,
+    timeout: 10,
+    skip_resolve: false,
+    resolve_timeout: 5,
+    oauth: {},
+    query: {},
+    headers: {},
+    parsers: {
+      'application/json' => -> (body) {
+        require 'json' unless defined? JSON
+        JSON.parse(body) rescue nil
+      }
+    },
+    logger: Logger.new(STDOUT),
+  }
+
   class << self
-    attr_accessor :raise_errors
-    attr_accessor :max_redirects
-    attr_accessor :timeout
-    attr_accessor :skip_resolve
-    attr_accessor :resolve_timeout
-    attr_accessor :oauth
-    attr_accessor :query
-    attr_accessor :headers
-    attr_accessor :parsers
+    DEFAULTS.each_key do |key|
+      attr_accessor key
+    end
   end
 
-  # Reasonable defaults
-  @raise_errors = false
-  @max_redirects = 20
-  @timeout = 10
-  @skip_resolve = false
-  @resolve_timeout = 5
-  @oauth = {}
-  @query = {}
-  @headers = {}
-  @parsers = {
-    'application/json' => -> (body) {
-      require 'json' unless defined? JSON
-      JSON.parse(body) rescue nil
-    }
-  }
+  DEFAULTS.each do |key, value|
+    instance_variable_set :"@#{key}", value
+  end
 
   def self.get(uri, opts = {})
     Request.new(opts).get(uri)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: micky
 version: !ruby/object:Gem::Version
-  version: 0.6.9
+  version: 0.7.0
 platform: ruby
 authors:
 - Rafaël Blais Masson
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-05-13 00:00:00.000000000 Z
+date: 2022-12-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -78,7 +78,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.1
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Lightweight and worry-free HTTP client