percy-common 3.1.1 → 3.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4a36d99fd5b06fe4959e00e8f13b9bbf1f7d356832c2a56f42e1f177f285b11d
-  data.tar.gz: a9ce522b6a66d1d96527cbdb774b1fd5dc92738d0c9953cb29ea4371de0bbc04
+  metadata.gz: ce7918aeff2ead877459e0bfd675c86caa2378a0b428dca1c8aaf93d427f276a
+  data.tar.gz: ed60d47e414a8a9ce3662407ae500bb56bdba6087de7cad21b36473ca8f67487
 SHA512:
-  metadata.gz: 3e18a43f607a28a9b6f450470bf7e0bb9c2acaf266cd134ecd499f8da3a9d37fc4f52b6fd8f550d08c8fea2cef83497948935d8025b23756c1cfba2b2dcac08d
-  data.tar.gz: '0062099d255d3cc75ddf646a939c72b7a82dd77c2da97289134e6d9240e9628603c5d0dd14dfde8d461f76d5dcf389b67295e91bb1d6588325de0f3de9d4331a'
+  metadata.gz: 4e8461a0a30b248ecf18d009d935a23a5c8a6076e6b2bf2136282b7235428787f8d62e98c5c00bcf1046525dc075995efd5fe47ac5ec076530aa82dc8915a66b
+  data.tar.gz: 731a137bdc7f022685d42562b2b394689874aeb0087e5e1d9b334d14d64cf1aa8bdf01f55af0dddca33d79625a12c9b2dff271ad04304998a95333c05b481f90

data/.rubocop.yml

@@ -1,3 +1,5 @@
+inherit_from: .rubocop_todo.yml
+
 inherit_gem:
   percy-style:
     - default.yml

data/.rubocop_todo.yml

@@ -0,0 +1,12 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2020-04-30 16:00:36 -0700 using RuboCop version 0.77.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 1
+RSpec/LeakyConstantDeclaration:
+  Exclude:
+    - 'spec/percy/keyword_struct_spec.rb'

data/RELEASING.md

@@ -1,16 +1,13 @@
 # Releasing
 
-1. `git pull origin master`
-1. Bump the version number in `lib/percy/common/version.rb`
-1. `git add lib/percy/common/version.rb`
-1. `git commit -m "version bump to X.X.X"`
-1. `git push origin master`
-1. `git tag vX.X.X`
-1. `git push --tags`
-1. `bundle exec rake build`
-1. `gem push pkg/percy-common-X.X.X.gem`
+1. Ensure you are on the expected branch (presumably `master`) and the current branch is up-to-date.
+1. Run `./release.sh`
 1. Visit [RubyGems.org](https://rubygems.org/gems/percy-common) and see the gem has been published
-1. Document the release on Github by first [creating a new release](https://github.com/percy/percy-common/releases/new)
+
+# Creating a Release on GitHub
+
+Finish the pre-started release opened by the script, or [create a new release](https://github.com/percy/percy-common/releases/new):
+
 1. Enter "vX.X.X" as the tag version. It should auto complete to say "Existing Tag"
 1. Enter "vX.X.X" as the release title
 1. Write a brief description as to what is included in this release. Linking to specific PRs is great.

data/lib/percy/common/version.rb

@@ -1,5 +1,5 @@
 module Percy
   module Common
-    VERSION = '3.1.1'.freeze
+    VERSION = '3.1.3'.freeze
   end
 end

data/lib/percy/redis_client.rb

@@ -4,11 +4,14 @@ require 'redis'
 
 module Percy
   class RedisClient
-    attr_reader :options
+    class InvalidConfiguration < ArgumentError
+    end
     attr_reader :client
+    attr_reader :options
 
-    def initialize(options = {})
-      @options = ssl_options.merge(options)
+    def initialize(given_options = {})
+      @provided_options = given_options
+      @options = ssl_options.merge(given_options)
       @client = ::Redis.new(options)
     end
 
@@ -17,7 +20,7 @@ module Percy
     end
 
     private def provided_url
-      options&.dig(:url)
+      @provided_options&.dig(:url)
     end
 
     private def ssl_options
@@ -31,52 +34,63 @@ module Percy
       return {} unless ssl_enabled?
 
       {
-        ca_file: certificate_authority_file,
+        ca_file: certificate_authority,
         cert: OpenSSL::X509::Certificate.new(client_certificate),
         key: OpenSSL::PKey::RSA.new(private_key),
       }
     end
 
     private def client_certificate
-      ENV.fetch(
-        'REDIS_SSL_CLIENT_CERTIFICATE',
-        File.read(client_certificate_path),
-      )
+      @provided_options&.dig(:ssl_params, :cert) ||
+        client_certificate_from_env ||
+        client_certificate_from_path
+    end
+
+    private def client_certificate_from_env
+      ENV['REDIS_SSL_CLIENT_CERTIFICATE']
     end
 
-    private def client_certificate_path
-      ENV.fetch(
-        'REDIS_SSL_CLIENT_CERTIFICATE_PATH',
-        File.join(cert_path, 'user.crt'),
-      )
+    private def client_certificate_from_path
+      File.read(fetch_key('REDIS_SSL_CLIENT_CERTIFICATE_PATH'))
     end
 
     private def private_key
-      ENV.fetch(
-        'REDIS_SSL_PRIVATE_KEY',
-        File.read(private_key_path),
-      )
+      provided_private_key ||
+        private_key_from_env ||
+        private_key_from_path
+    end
+
+    private def provided_private_key
+      @provided_options&.dig(:ssl_params, :key)
+    end
+
+    private def private_key_from_env
+      ENV['REDIS_SSL_PRIVATE_KEY']
+    end
+
+    private def private_key_from_path
+      File.read(fetch_key('REDIS_SSL_PRIVATE_KEY_PATH'))
+    end
+
+    private def certificate_authority
+      provided_certificate_authority ||
+        certificate_authority_from_path
+    end
+
+    private def provided_certificate_authority
+      @provided_options&.dig(:ssl_params, :ca_file)
     end
 
-    private def private_key_path
-      ENV.fetch(
-        'REDIS_SSL_PRIVATE_KEY_PATH',
-        File.join(cert_path, 'user_private.key'),
-      )
+    private def certificate_authority_from_path
+      fetch_key('REDIS_SSL_CERTIFICATE_AUTHORITY_PATH')
     end
 
-    private def certificate_authority_file
-      ENV.fetch(
-        'REDIS_SSL_CERTIFICATE_AUTHORITY_PATH',
-        File.join(cert_path, 'server_ca.pem'),
-      )
+    private def fetch_key(key)
+      ENV.fetch(key) { missing_key(key) }
     end
 
-    private def cert_path
-      ENV.fetch(
-        'REDIS_SSL_CERTIFICATE_PATH',
-        File.expand_path('../../redis', __dir__),
-      )
+    private def missing_key(key)
+      raise InvalidConfiguration, "#{key} is not defined"
     end
   end
 end

data/percy-common.gemspec

@@ -17,13 +17,13 @@ Gem::Specification.new do |spec|
   end
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'dogstatsd-ruby', '~> 4.4.0'
+  spec.add_dependency 'dogstatsd-ruby', '>= 4.4', '< 4.9'
   spec.add_dependency 'excon', '~> 0.57'
-  spec.add_dependency 'redis', '~> 4.1.3'
+  spec.add_dependency 'redis', '>= 4.1.3', '< 5.0.0'
 
   spec.add_development_dependency 'bundler', '~> 2.1.4'
   spec.add_development_dependency 'guard-rspec', '~> 4.7'
-  spec.add_development_dependency 'percy-style', '~> 0.6.0'
-  spec.add_development_dependency 'rake', '~> 12.3'
+  spec.add_development_dependency 'percy-style', '~> 0.7.0'
+  spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rspec', '~> 3.2'
 end

data/release.sh

@@ -0,0 +1,76 @@
+#!/usr/bin/env bash
+
+set -e
+
+cd "$(dirname "$0")" || exit 1
+CURDIR="$(pwd)"
+
+PERCY_COMMON_VERSION="$(grep "VERSION" lib/percy/common/version.rb | awk -F "'" '{print $2}')"
+
+echo "Current percy-common version: $PERCY_COMMON_VERSION"
+
+if [[ $# -lt 1 ]]; then
+  echo "Usage $0 <version>"
+  exit 1
+fi
+
+rm "$CURDIR/"percy-common*.gem >/dev/null 2>&1 || true
+
+delete_existing_version() {
+  git tag -d "v$1" || true
+  git push origin ":v$1" || true
+  # Replace - with .pre. in version string for rubygems compatibility
+  GEM_VERSION=${1/-/.pre.}
+  gem yank percy-common -v "$GEM_VERSION" || true
+}
+
+if [[ $1 =~ ^.*delete$ ]]; then
+  shift
+  echo "Preparing to delete $1"
+  sleep 3
+  delete_existing_version "$PERCY_COMMON_VERSION"
+else
+  CLEAN=$(
+    git diff-index --quiet HEAD --
+    echo $?
+  )
+  if [[ "$CLEAN" == "0" ]]; then
+    if [[ $1 == '--force' ]]; then
+      shift
+      VERSION=$1
+      if [[ -n $VERSION ]]; then
+        echo "Deleting version $VERSION"
+        sleep 1
+        delete_existing_version "$VERSION"
+      else
+        echo "Missing release version"
+        exit 1
+      fi
+    fi
+    VERSION=$1
+    # Replace - with .pre. in version string for rubygems compatibility
+    GEM_VERSION="${VERSION/-/.pre.}"
+    if [[ "$GEM_VERSION" == "$VERSION" ]]; then
+      echo "Releasing $VERSION"
+    else
+      echo "Releasing $VERSION as $GEM_VERSION"
+    fi
+    sleep 1
+
+    sed -i "" -e "s/$PERCY_COMMON_VERSION/$VERSION/g" "lib/percy/common/version.rb"
+    git add "lib/percy/common/version.rb"
+    git commit -a -m "Release $VERSION" || true
+
+    git tag -a "v$VERSION" -m "$1" || true
+    git push origin "v$VERSION" || true
+
+    bundle exec rake build
+    gem push "$CURDIR/pkg/percy-common-$GEM_VERSION.gem"
+    open "https://github.com/percy/percy-common/releases/new?tag=v$VERSION&title=$VERSION"
+    rm "$CURDIR/pkg/percy-common-$GEM_VERSION.gem"
+  else
+    echo "Please commit your changes and try again"
+    exit 1
+  fi
+fi
+echo "Done"

metadata

@@ -1,29 +1,35 @@
 --- !ruby/object:Gem::Specification
 name: percy-common
 version: !ruby/object:Gem::Version
-  version: 3.1.1
+  version: 3.1.3
 platform: ruby
 authors:
 - Perceptual Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-07 00:00:00.000000000 Z
+date: 2020-06-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dogstatsd-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '4.4'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 4.4.0
+        version: '4.9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '4.4'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 4.4.0
+        version: '4.9'
 - !ruby/object:Gem::Dependency
   name: excon
   requirement: !ruby/object:Gem::Requirement
@@ -42,16 +48,22 @@ dependencies:
   name: redis
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 4.1.3
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 5.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 4.1.3
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 5.0.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -86,28 +98,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.0
+        version: 0.7.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.0
+        version: 0.7.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.3'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.3'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -133,6 +145,7 @@ files:
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
+- ".rubocop_todo.yml"
 - Gemfile
 - Guardfile
 - LICENSE
@@ -149,6 +162,7 @@ files:
 - lib/percy/redis_client.rb
 - lib/percy/stats.rb
 - percy-common.gemspec
+- release.sh
 homepage: ''
 licenses: []
 metadata: {}