we-call 0.7.0 → 0.9.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 2f6b6c396092e080b179bf2811abd96992fd7e1a
-  data.tar.gz: f3d35f54e93e60b405e28a732766155d2110d209
+SHA256:
+  metadata.gz: 83cdb2541d4d694624f06d63e03dce612db88ce09e0ff7532c4d701424aedb46
+  data.tar.gz: a3862e1f9f6f4f1685ab2c7c8077e4a166ab3ebfa8a017fb32aa701d7dc68705
 SHA512:
-  metadata.gz: aa4b02af461ee342404ba0bd2ef959cbd1728cc89208fa1574fe6510b8daa85e80aa9111fe04729c57124ad2bae0bdaa736bddb800755affa0b484bfa457efe1
-  data.tar.gz: b13a9ecdc979fdddc1d1cd3fd3be82eac6288d9be470e54414c46b3e2f7b39c261a0725a91bab51ab74578581fcade181295cb953e1ce6311057e3b6d7aa82a9
+  metadata.gz: 1e8f5628d633564aaf0183a89ebebb9e9c4ee82d298d2deee8513b26dd5f8a52326ce57278c3f9782986def685212995f034a3e18d65321a34081689d248ca2a
+  data.tar.gz: 3792fa6710c0e4fcca018ad96464772d5776864bea0fc19e2c493bd871a9008eb09f6a675892008df4b9fe8454cdda66eca8a0e09d38dee5018428c8fa985b49

data/.circleci/config.yml

@@ -0,0 +1,42 @@
+version: 2.1
+
+x-shared: &x-shared
+  docker:
+    - image: circleci/ruby:2.6.3-stretch-node
+
+jobs:
+  build:
+    <<: *x-shared
+    steps:
+      - checkout
+      - run:
+          name: Build
+          command: ./script/cibuild
+      - persist_to_workspace:
+          root: .
+          paths: .
+  publish-gem:
+    <<: *x-shared
+    steps:
+      - attach_workspace:
+          at: ~/project # circleci default working directory
+      - run:
+          name: Publish Gem
+          command: ./script/publish_gem
+workflows:
+  version: 2
+  build:
+    jobs:
+      - build:
+          filters:
+            tags:
+              only: /v[0-9]+(\.[0-9]+)+/
+      - publish-gem:
+          requires:
+            - build
+          filters:
+            branches:
+              ignore:
+                - /.*/
+            tags:
+              only: /v[0-9]+(\.[0-9]+)+/

data/.coveralls.yml

@@ -1 +1 @@
-service_name: travis-ci
+service_name: circleci

data/.github/workflows/build.yml

@@ -0,0 +1,23 @@
+name: Ruby
+
+on:
+  push:
+    branches: [ $default-branch ]
+  pull_request:
+    branches: [ $default-branch ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: 2.6
+    - name: Install dependencies
+      run: bundle install
+    - name: Generate gemfiles
+      run: bundle exec appraisal install
+    - name: Run tests
+      run: bundle exec appraisal rspec

data/.gitignore

@@ -10,3 +10,4 @@
 /spec/reports/
 /tmp/
 .byebug_history
+we-call-*.gem

data/.travis.yml

@@ -6,18 +6,21 @@ before_script:
 script:
   - bundle exec appraisal rspec
 gemfiles:
-  - faraday_0.9.gemfile
   - faraday_0.10.gemfile
   - faraday_0.11.gemfile
   - faraday_0.12.gemfile
   - faraday_0.13.gemfile
+  - faraday_0.15.gemfile
+  - faraday_0.14.gemfile
   - rails_4.2.gemfile
   - rails_5.0.gemfile
   - rails_5.1.gemfile
+  - rails_5.2.gemfile
 rvm:
-  - 2.2.7
-  - 2.3.4
-  - 2.4.1
+  - 2.2
+  - 2.3
+  - 2.4
+  - 2.5
   - ruby-head
 matrix:
   allow_failures:

data/Appraisals

@@ -1,3 +1,11 @@
+appraise "faraday-0.15" do
+  gem "faraday", "~> 0.15.0"
+end
+
+appraise "faraday-0.14" do
+  gem "faraday", "~> 0.14.0"
+end
+
 appraise "faraday-0.13" do
   gem "faraday", "~> 0.13.0"
 end
@@ -14,8 +22,8 @@ appraise "faraday-0.10" do
   gem "faraday", "~> 0.10.0"
 end
 
-appraise "faraday-0.9" do
-  gem "faraday", "~> 0.9.0"
+appraise "rails-5.2" do
+  gem "rails", "~> 5.2.0"
 end
 
 appraise "rails-5.1" do

data/CHANGELOG.md

@@ -5,6 +5,38 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
+## [v0.9.1] - 2020-11-20
+### Added
+- Automatically retry on network errors([#31])
+
+[activesupport]: https://github.com/rails/rails/tree/master/activesupport
+[#31]: https://github.com/wework/we-call-gem/pull/31
+
+## [v0.9.0] - 2018-09-18
+### Added
+- Automatically reports deprecations to ActiveSupport if [activesupport] gem exists ([#21])
+- Defaults detect_deprecations to nil instead of :active_support
+
+[activesupport]: https://github.com/rails/rails/tree/master/activesupport
+[#21]: https://github.com/wework/we-call-gem/pull/21
+
+## [v0.8.0] - 2018-08-27
+### Added
+- Support for Ruby v2.5
+- Support for Rails v5.2
+- Support for Faraday v0.14 and v0.15
+- Automatically reports deprecations to Rollbar if [rollbar] gem exists ([#17])
+
+[rollbar]: https://github.com/rollbar/rollbar-gem
+[#17]: https://github.com/wework/we-call-gem/pull/17
+
+### Breaking
+- Dropped support for Faraday v0.9 (may still work but its no longer tested or recommended)
+
+## [v0.7.1] - 2018-02-20
+### Fixed
+- Call the gzip middleware to handle gzipped responses, which have been broken since v0.7 for typhoeus users
+
 ## [v0.7.0] - 2017-10-07
 ### Breaking Changes
 - Removed `We::Call::Deprecated` and `We::Call::Annotations`. Deprecation logic is now handled by [rails-sunset] instead. I fully understand the irony of removing deprecation logic without deprecation

data/Gemfile

@@ -1,4 +1,6 @@
 source 'https://rubygems.org'
 
+ruby '~> 2.2'
+
 # Specify your gem's dependencies in we-call.gemspec
 gemspec

data/README.md

@@ -1,6 +1,6 @@
 # We::Call
 
-[![Build Status][travis-image]][travis-url]
+[![Build Status][ci-image]][ci-url]
 [![Coverage Status][coveralls-image]][coveralls-url]
 [![MIT License][license-image]][license-url]
 
@@ -102,6 +102,36 @@ As well as `timeout: num_seconds` which can set the entire open/read (essentiall
 
 ### Client
 
+**Retry**
+
+Automatically enabled, the retry middleware will retry the request in case of network errors. By default, the middleware will retry up to 3 times, waiting 1 second between the retries.
+
+Disable the middleware:
+
+```ruby
+We::Call.configure do |config|
+  config.retry = false
+end
+```
+
+Adjust the middleware:
+
+```ruby
+We::Call.configure do |config|
+  config.retry_options = { interval: 0.5 }
+end
+```
+
+The gem smartly merges the options passed, so you can specify your own list of exceptions without being afraid to override the default ones:
+
+```ruby
+We::Call.configure do |config|
+  config.retry_options = { exceptions: [Faraday::ResourceNotFound] }
+end
+```
+
+Check [Faraday's Retry Docs](https://github.com/lostisland/faraday/blob/master/docs/middleware/request/retry.md) for a list of available options.
+
 **DetectDeprecations**
 
 Automatically enabled, the faraday-sunset middleware will watch for the [Sunset header](https://tools.ietf.org/html/draft-wilde-sunset-header-03) and send warning to `ActiveSupport::Deprecation` if enabled, or to whatever is in `ENV['rake.logger']`.
@@ -122,9 +152,10 @@ Easy! Check your logs for `user_agent=service-name; app_name=service-name;` The
 
 ## Requirements
 
-- **Ruby:** v2.2 - v2.4
-- **Faraday:** v0.9 - v0.13
+- **Ruby:** v2.2 - v2.5
+- **Faraday:** v0.10 - v0.15
 
+_**Note:** Other versions of Faraday may work, but we can't test against all of them forever._
 
 ## TODO
 
@@ -154,10 +185,10 @@ Bug reports and pull requests are welcome on GitHub at [wework/we-call](https://
 
 
 [coveralls-image]:https://coveralls.io/repos/github/wework/we-call-gem/badge.svg?branch=master
-[coveralls-url]:https://coveralls.io/github/wework/we-call-gem?branch=master
+[coveralls-url]:https://coveralls.io/github/wework/we-call-gem?branch=main
 
-[travis-url]:https://travis-ci.org/wework/we-call-gem
-[travis-image]: https://travis-ci.org/wework/we-call-gem.svg?branch=master
+[ci-url]:https://app.circleci.com/pipelines/github/wework/we-call-gem
+[ci-image]:https://circleci.com/gh/wework/we-call-gem.svg?style=shield
 
 [license-url]: LICENSE
 [license-image]: http://img.shields.io/badge/license-MIT-000000.svg?style=flat-square

data/lib/we/call.rb

@@ -1,13 +1,12 @@
 require 'faraday'
+require 'faraday_middleware'
 require 'faraday-sunset'
 require 'typhoeus'
 
 module We
   module Call
-    autoload :Annotations,   "we/call/annotations"
     autoload :Connection,    "we/call/connection"
     autoload :Configuration, "we/call/configuration"
-    autoload :Deprecated,    "we/call/deprecated"
     autoload :Middleware,    "we/call/middleware"
     autoload :VERSION,       "we/call/version"
 

data/lib/we/call/configuration.rb

@@ -4,12 +4,14 @@ require 'faraday_middleware'
 module We
   module Call
     class Configuration
-      attr_accessor :app_env, :app_env_header, :app_name, :app_name_header, :detect_deprecations
+      attr_accessor :app_env, :app_env_header, :app_name, :app_name_header, :detect_deprecations,
+                    :retry, :retry_options
 
       def initialize
         @app_env_header = 'X-App-Env'
         @app_name_header = 'X-App-Name'
-        @detect_deprecations = :active_support
+        @retry = true
+        @retry_options = {}
       end
     end
   end

data/lib/we/call/connection.rb

@@ -14,6 +14,12 @@ module We
       DEFAULT_ADAPTER_CLASS = Faraday::Adapter::Typhoeus
       DEFAULT_ADAPTER = :typhoeus
 
+      DEFAULT_RETRY_OPTIONS = {
+        max: 3,
+        interval: 1,
+        exceptions: [Faraday::ConnectionFailed, Faraday::TimeoutError]
+      }
+
       class MissingApp < ArgumentError; end
       class MissingEnv < ArgumentError; end
       class MissingTimeout < ArgumentError; end
@@ -21,12 +27,16 @@ module We
 
       QueryableBuilder = Class.new(Faraday::RackBuilder) do
         def adapter?
-          @has_adapter || false
+          @adapter || false
         end
 
         def adapter(key, *args, &block)
           super
-          @has_adapter = true
+          @adapter = key
+        end
+
+        def get_adapter
+          @adapter || DEFAULT_ADAPTER
         end
       end
 
@@ -68,9 +78,16 @@ module We
           if config.detect_deprecations
             faraday.response :sunset, setup_sunset_middleware(faraday)
           end
+          if config.retry
+            faraday.request :retry, fetch_retry_options
+          end
 
           yield faraday if block_given?
 
+          unless adapter_handles_gzip?(faraday.builder.get_adapter)
+            faraday.use :gzip
+          end
+
           faraday.adapter DEFAULT_ADAPTER unless faraday.builder.adapter?
         end
       end
@@ -95,14 +112,28 @@ module We
         raise MissingOpenTimeout, 'open_timeout must be set, and defaults to 1 second. This is the time until a connection is established with another server, and after 1 sec it\'s probably not there.'
       end
 
-      def setup_sunset_middleware(faraday)
-        # In v0.5.0 this was a bool switch, now it takes :active_support or an instance of a Logger
-        if config.detect_deprecations == true || config.detect_deprecations == :active_support
-          return { active_support: true }
+      # @return [Boolean] Does the adapter handle gzip automatically or not
+      # https://github.com/lostisland/faraday_middleware/blob/master/lib/faraday_middleware/gzip.rb#L9
+      def adapter_handles_gzip?(adapter)
+        [:em_http, :net_http, :net_http_persistent].include?(adapter)
+      end
 
+      def setup_sunset_middleware(faraday)
+        options = { rollbar: :auto, active_support: :auto }
         # Pass something that might be a logger or anything with a warn method
-        elsif config.detect_deprecations.respond_to?(:warn)
-          return { logger: config.detect_deprecations }
+        if config.detect_deprecations.respond_to?(:warn)
+          options = options.merge({ logger: config.detect_deprecations })
+        end
+        options
+      end
+
+      def fetch_retry_options
+        DEFAULT_RETRY_OPTIONS.merge(config.retry_options) do |key, default_val, new_val|
+          if key == :exceptions
+            default_val + Array(new_val)
+          else
+            new_val
+          end
         end
       end
 

data/lib/we/call/version.rb

@@ -1,5 +1,5 @@
 module We
   module Call
-    VERSION = "0.7.0"
+    VERSION = '0.9.1'
   end
 end

data/script/cibuild

@@ -0,0 +1,7 @@
+#!/bin/bash
+set -e
+cd "$(dirname "$0")/.."
+
+bundle install
+bundle exec appraisal install
+bundle exec appraisal rspec

data/script/publish_gem

@@ -0,0 +1,18 @@
+#!/bin/bash
+set -eox pipefail
+
+mkdir -p ~/.gem
+echo :rubygems_api_key: $RUBYGEMS_APIKEY > ~/.gem/credentials
+chmod 0600 ~/.gem/credentials
+
+VERSION_FILE="lib/we/call/version.rb"
+if [[ -n $CIRCLE_TAG ]]; then
+  # Determine the version from the tag; remove the leading v "v3.3.0" -> "3.3.0"
+  VERSION=$(echo "${CIRCLE_TAG}" | sed 's/^v//')
+  # Replace the version for the gemspec with the tag version
+  sed -i.bak -E "s/VERSION = .*/VERSION = '$VERSION'/" $VERSION_FILE
+fi
+
+gem build *.gemspec
+
+gem push *.gem

data/we-call.gemspec

@@ -21,17 +21,17 @@ Gem::Specification.new do |spec|
   spec.require_paths        = ["lib"]
 
   spec.add_dependency "typhoeus", "~> 1.3"
-  spec.add_dependency "faraday", ">= 0.9.0", "< 1.0"
-  spec.add_dependency "faraday_middleware", '~> 0'
-  spec.add_dependency "faraday-sunset", "~> 0.1.0"
+  spec.add_dependency "faraday", ">= 0.9.0", "< 1"
+  spec.add_dependency "faraday_middleware", '~> 0.10'
+  spec.add_dependency "faraday-sunset", '~> 0.2'
 
-  spec.add_development_dependency "appraisal", "~> 2"
-  spec.add_development_dependency "coveralls", '~> 0.7'
+  spec.add_development_dependency "appraisal", "~> 2.0"
+  spec.add_development_dependency "coveralls_reborn", '~> 0.13'
   spec.add_development_dependency "bundler", "~> 1.14"
   spec.add_development_dependency "rake", "~> 12.0"
   spec.add_development_dependency "rails", ">= 4.2"
   spec.add_development_dependency "rspec", "~> 3.5"
   spec.add_development_dependency "simplecov", '~> 0.15'
   spec.add_development_dependency "hashie", "~> 3.5"
-  spec.add_development_dependency "vcr", '~> 3'
+  spec.add_development_dependency "vcr", '~> 4.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: we-call
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.9.1
 platform: ruby
 authors:
 - WeWork Engineering
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-07 00:00:00.000000000 Z
+date: 2020-11-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: typhoeus
@@ -33,7 +33,7 @@ dependencies:
         version: 0.9.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -43,63 +43,63 @@ dependencies:
         version: 0.9.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1'
 - !ruby/object:Gem::Dependency
   name: faraday_middleware
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.10'
 - !ruby/object:Gem::Dependency
   name: faraday-sunset
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.0
+        version: '0.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.0
+        version: '0.2'
 - !ruby/object:Gem::Dependency
   name: appraisal
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: coveralls
+  name: coveralls_reborn
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '0.13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '0.13'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -190,14 +190,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '4.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '4.0'
 description: Handles conventions of making calls to other services, with required
   metadata for tracking calls between services, deprecations of endpoints, trace IDs,
   throttling, etc.
@@ -207,7 +207,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".circleci/config.yml"
 - ".coveralls.yml"
+- ".github/workflows/build.yml"
 - ".gitignore"
 - ".rspec"
 - ".travis.yml"
@@ -227,6 +229,8 @@ files:
 - lib/we/call/middleware/server.rb
 - lib/we/call/middleware/server/log_user_agent.rb
 - lib/we/call/version.rb
+- script/cibuild
+- script/publish_gem
 - we-call.gemspec
 homepage: https://github.com/wework/we-call-gem
 licenses:
@@ -247,8 +251,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Making healthy, happy HTTP calls