gh 0.15.0 → 0.18.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 16d87ecc5b3c30bf97a8a4bb031f92f5f0f6158f
+  data.tar.gz: 1faf61eced641abc7cc16f078713406ddac90a8f
+SHA512:
+  metadata.gz: 4d950b13a2578b2d3224501075e7a4700464e207919d7d2f208cc54ce4eb16bb5216daae061d3834d6e79cc1295a7753b16d1ab3766b231b8eb779de014a1a33
+  data.tar.gz: bd2f9066b1f1be9060d10a9a246d60959d4e6f7e2597a3652594c6155db8fb9b72aa8ef553fd4287c33634bed4b08ad51da1f725ea71fe8c45ad75531cb3bfef

data/lib/gh.rb

@@ -7,7 +7,6 @@ module GH
   autoload :Case,             'gh/case'
   autoload :CustomLimit,      'gh/custom_limit'
   autoload :Error,            'gh/error'
-  autoload :FaradayAdapter,   'gh/faraday_adapter'
   autoload :Instrumentation,  'gh/instrumentation'
   autoload :LazyLoader,       'gh/lazy_loader'
   autoload :LinkFollower,     'gh/link_follower'
@@ -17,6 +16,7 @@ module GH
   autoload :Parallel,         'gh/parallel'
   autoload :Remote,           'gh/remote'
   autoload :Response,         'gh/response'
+  autoload :ResponseXHeaderFormatter, 'gh/response_x_header_formatter'
   autoload :ResponseWrapper,  'gh/response_wrapper'
   autoload :Stack,            'gh/stack'
   autoload :TokenCheck,       'gh/token_check'

data/lib/gh/merge_commit.rb

@@ -58,9 +58,9 @@ module GH
 
     def github_done_checking?(hash)
       case hash['mergeable_state']
-      when 'checking'                               then false
-      when 'unknown'                                then hash['merged']
-      when 'clean', 'dirty', 'unstable', 'stable'   then true
+      when 'checking'                                                           then false
+      when 'unknown'                                                            then hash['merged']
+      when 'clean', 'dirty', 'unstable', 'stable', 'blocked', 'behind', 'draft' then true
       else fail "unknown mergeable_state #{hash['mergeable_state'].inspect} for #{url(hash)}"
       end
     end

data/lib/gh/normalizer.rb

@@ -99,7 +99,7 @@ module GH
       when /^repos?_(.*)$/              then "repository_#{$1}"
       when /^(.*)_repo$/                then "#{$1}_repository"
       when /^(.*)_repos$/               then "#{$1}_repositories"
-      when 'commit', 'commit_id', 'id'  then value =~ /^\w{40}$/ ? 'sha' : key
+      when 'commit', 'commit_id', 'id'  then value.to_s =~ /^\w{40}$/ ? 'sha' : key
       when 'comments'                   then Numeric === value ? 'comment_count'    : key
       when 'forks'                      then Numeric === value ? 'fork_count'       : key
       when 'repositories'               then Numeric === value ? 'repository_count' : key

data/lib/gh/remote.rb

@@ -1,5 +1,6 @@
 require 'gh'
 require 'faraday'
+require 'active_support/core_ext/string'
 
 module GH
   # Public: This class deals with HTTP requests to Github. It is the base Wrapper you always want to use.
@@ -7,7 +8,7 @@ module GH
   class Remote < Wrapper
     attr_reader :api_host, :connection, :headers, :prefix
 
-    # Public: Generates a new Rempte instance.
+    # Public: Generates a new Remote instance.
     #
     # api_host - HTTP host to send requests to, has to include schema (https or http)
     # options  - Hash with configuration options:
@@ -16,7 +17,6 @@ module GH
     #            :password - Github password used for login (optional).
     #            :origin   - Value of the origin request header (optional).
     #            :headers  - HTTP headers to be send on every request (optional).
-    #            :adapter  - HTTP library to use for making requests (optional, default: :net_http)
     #
     # It is highly recommended to set origin, but not to set headers.
     # If you set the username, you should also set the password.
@@ -48,7 +48,10 @@ module GH
         builder.request(:basic_auth, username, password)  if username and password
         builder.request(:retry)
         builder.response(:raise_error)
-        #builder.use(options[:adapter] || GH::FaradayAdapter)
+        if defined? FaradayMiddleware::Instrumentation
+          builder.use :instrumentation
+        end
+        builder.response(:logger, nil, formatter: GH.const_get(options[:formatter].camelize)) if options[:formatter]
         builder.adapter(:net_http)
       end
     end
@@ -74,7 +77,8 @@ module GH
 
     # Internal: ...
     def http(verb, url, headers = {}, &block)
-      connection.run_request(verb, url, nil, headers, &block)
+      body = headers.delete :body
+      connection.run_request(verb, url, body, headers, &block)
     rescue Exception => error
       raise Error.new(error, nil, :verb => verb, :url => url, :headers => headers)
     end

data/lib/gh/response_x_header_formatter.rb

@@ -0,0 +1,12 @@
+require 'faraday/logging/formatter'
+
+module GH
+  class ResponseXHeaderFormatter < Faraday::Logging::Formatter
+    def request(env)
+    end
+
+    def response(env)
+      info('Response') { env.response_headers.select {|k,v| k =~ /^x-/}.sort.to_h }
+    end
+  end
+end

data/lib/gh/token_check.rb

@@ -18,7 +18,7 @@ module GH
       @check_token = false
 
       auth_header = "Basic %s" % Base64.encode64("#{client_id}:#{client_secret}").gsub("\n", "")
-      http :head, path_for("/applications/#{client_id}/tokens/#{token}?client_id=#{client_id}&client_secret=#{client_secret}"), "Authorization" => auth_header
+      http :post, path_for("/applications/#{client_id}/token"), :body => "{\"access_token\": \"#{token}\"}", "Authorization" => auth_header
     rescue GH::Error(:response_status => 404) => error
       raise GH::TokenInvalid, error
     end

data/lib/gh/version.rb

@@ -1,4 +1,4 @@
 module GH
   # Public: Library version.
-  VERSION = "0.15.0"
+  VERSION = '0.18.0'
 end

metadata

@@ -1,142 +1,139 @@
 --- !ruby/object:Gem::Specification
 name: gh
 version: !ruby/object:Gem::Version
-  version: 0.15.0
-  prerelease: 
+  version: 0.18.0
 platform: ruby
 authors:
 - Konstantin Haase
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-11-18 00:00:00.000000000 Z
+date: 2020-06-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: faraday
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: backports
+  name: faraday_middleware
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: multi_json
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
   name: addressable
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.4.0
+        version: '2.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.4.0
+        version: '2.4'
 - !ruby/object:Gem::Dependency
   name: net-http-persistent
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.9'
 - !ruby/object:Gem::Dependency
   name: net-http-pipeline
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: multi-layer client for the github api v3
@@ -152,7 +149,6 @@ files:
 - lib/gh/case.rb
 - lib/gh/custom_limit.rb
 - lib/gh/error.rb
-- lib/gh/faraday_adapter.rb
 - lib/gh/instrumentation.rb
 - lib/gh/lazy_loader.rb
 - lib/gh/link_follower.rb
@@ -164,6 +160,7 @@ files:
 - lib/gh/remote.rb
 - lib/gh/response.rb
 - lib/gh/response_wrapper.rb
+- lib/gh/response_x_header_formatter.rb
 - lib/gh/stack.rb
 - lib/gh/token_check.rb
 - lib/gh/version.rb
@@ -171,26 +168,25 @@ files:
 homepage: https://github.com/travis-ci/gh
 licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.6.14
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: layered github client
 test_files: []

data/lib/gh/faraday_adapter.rb

@@ -1,60 +0,0 @@
-require 'gh'
-require 'faraday'
-require 'thread'
-require 'net/https'
-require 'net/http/pipeline'
-require 'net/http/persistent'
-
-module GH
-  # Faraday adapter based on Net::HTTP, with persistent connection and pipeline support.
-  class FaradayAdapter < Faraday::Adapter::NetHttpPersistent
-    class Manager
-      def initialize(*)
-        @mutex    = Mutex.new
-        @requests = {}
-      end
-
-      def add_request(http, env, adapter)
-        url = env[:url] + '/'
-        env[:adapter] = adapter
-
-        @mutex.synchronize do
-          @requests[url] ||= []
-          @requests[url] << env
-        end
-      end
-
-      def run
-        requests = nil
-        @mutex.synchronize { requests, @requests = @requests, {} }
-        http = Net::HTTP::Persistent.new 'GH'
-        requests.each do |url, envs|
-          _requests = envs.map { |env| env[:adapter].create_request(env) }
-          responses = http.pipeline(url, _requests)
-          envs.zip(responses) do |env, http_response|
-            env[:adapter].save_response(env, http_response.code.to_i, http_response.body) do |headers|
-              http_response.each_header { |key, value| headers[key] = value }
-            end
-            env[:response].finish(env)
-          end
-        end
-      end
-    end
-
-    self.supports_parallel = true
-
-    def self.setup_parallel_manager(options = {})
-      Manager.new(options)
-    end
-
-    def call(env)
-      catch(:parallel) { super }
-    end
-
-    def perform_request(http, env)
-      return super unless env[:parallel_manager]
-      env[:parallel_manager].add_request(http, env, self)
-      throw :parallel, @app.call(env)
-    end
-  end
-end