mauth-client 6.3.0 → 6.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3d26cf42f3bc98d5ce98981ee710c52d2b87a9360b6ae54391288182c95fa09a
-  data.tar.gz: 03c4a9d0e96187d57232d5a1923935ad5db9c190a891fda410595010bbf1d946
+  metadata.gz: fc8cbe23e9d1c4aed81b5c3879945892303f21ce8617060f97d59bf4504bf266
+  data.tar.gz: eef092f242b51ca385b1aec1e10c57aed8195483da3a29f94e693cd2488473c3
 SHA512:
-  metadata.gz: 58751eb9040426589a0274390307abfc518c1465372c6b26329debb25a12c06fb63e96fccd4093fd45e90bd24fefd6c64966160b6520513515cd5dc9ac96f804
-  data.tar.gz: 0febdd455262d240d5f456acc00785d8a0941627ca352ffa559526ddadbaab726213b87e24eaa8f576c430b0a856725224ed48909cdc64cf9ce6b3a77b8c4acf
+  metadata.gz: 00d23785893fece78afcebbb52ca56f955e0f1018e050882fe7991b02cd7f118a27cde158c126018006fe93559806ac6268e8469b56cabfbb227c7210da60761
+  data.tar.gz: 7fc815dec75cdcf5663e03dec6896a155b1818504aa7f8716654ffbacfe27c2ba46be57906c3acefc093fb3a6c5f2449e9f2693c7bcb36b8e25a7ae4bfe69eae

data/.rubocop.yml

@@ -0,0 +1,53 @@
+inherit_gem:
+  rubocop-mdsol: rubocop.yml
+
+require:
+  - rubocop-performance
+
+AllCops:
+  Exclude:
+    - gemfiles/**/*
+
+Gemspec/RequireMFA:
+  Enabled: false
+
+Layout/ArgumentAlignment:
+  EnforcedStyle: with_fixed_indentation
+
+Layout/FirstHashElementIndentation:
+  EnforcedStyle: consistent
+
+Layout/LineLength:
+  Exclude:
+    - spec/client/local_authenticator_spec.rb
+
+Lint/MissingSuper:
+  Exclude:
+    - exe/mauth-client
+    - lib/mauth/faraday.rb
+    - lib/mauth/rack.rb
+
+Metrics/AbcSize:
+  Exclude:
+    - lib/mauth/client.rb
+    - lib/mauth/client/local_authenticator.rb
+    - lib/mauth/proxy.rb
+
+Metrics/MethodLength:
+  Exclude:
+    - lib/mauth/client.rb
+
+Naming/FileName:
+  Exclude:
+    - lib/mauth-client.rb
+
+Style/FrozenStringLiteralComment:
+  Enabled: true
+
+Style/GlobalVars:
+  Exclude:
+    - exe/mauth-client
+
+Style/StringLiterals:
+  Enabled: true
+  EnforcedStyle: single_quotes

data/.ruby-version

@@ -0,0 +1 @@
+2.6.9

data/.travis.yml

@@ -3,26 +3,32 @@ language: ruby
 cache: bundler
 
 rvm:
-  - 2.5
   - 2.6
   - 2.7
   - 3.0
+  - 3.1
+
+env:
+  global:
+    - BUNDLE_JOBS=4
 
 jobs:
   exclude:
     - rvm: 3.0
       gemfile: gemfiles/faraday_0.x.gemfile # Faraday v0.x does not officially support Ruby 3.0 (see: https://github.com/lostisland/faraday/releases/tag/v1.3.0)
+    - rvm: 3.1
+      gemfile: gemfiles/faraday_0.x.gemfile # Faraday v0.x does not officially support Ruby 3.0 (see: https://github.com/lostisland/faraday/releases/tag/v1.3.0)
 
 gemfile:
   - gemfiles/faraday_0.x.gemfile
   - gemfiles/faraday_1.x.gemfile
+  - gemfiles/faraday_2.x.gemfile
 
 before_install:
-  - gem update --system --force -N > /dev/null && echo "Rubygems version $(gem --version)"
-  - gem install bundler --force -N && bundle --version
+  - gem update --system --force -N > /dev/null && echo "Rubygems version $(gem --version)" && bundle --version
 
 install:
-  - bundle install --jobs=3 --retry=3
+  - bundle install
   - >-
     curl -H 'Cache-Control: no-cache'
     https://raw.githubusercontent.com/mdsol/fossa_ci_scripts/main/travis_ci/fossa_install.sh |
@@ -30,6 +36,7 @@ install:
 
 script:
   - bundle exec rspec
+  - bundle exec rubocop
   - bundle exec rake benchmark
   - >-
     curl -H 'Cache-Control: no-cache'
@@ -44,4 +51,4 @@ deploy:
   on:
     tags: true
     repo: mdsol/mauth-client-ruby
-    condition: $TRAVIS_RUBY_VERSION == 3.0 && $BUNDLE_GEMFILE == $TRAVIS_BUILD_DIR/gemfiles/faraday_1.x.gemfile
+    condition: $TRAVIS_RUBY_VERSION == 3.1 && $BUNDLE_GEMFILE == $TRAVIS_BUILD_DIR/gemfiles/faraday_1.x.gemfile

data/Appraisals

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 appraise 'faraday_0.x' do
   gem 'faraday', '0.9.0'
 end
@@ -5,3 +7,7 @@ end
 appraise 'faraday_1.x' do
   gem 'faraday', '~> 1.0'
 end
+
+appraise 'faraday_2.x' do
+  gem 'faraday', '~> 2.0'
+end

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## v6.4.0
+- Support Ruby 3.1.
+- Drop support for Ruby < 2.6.0.
+- Allow Faraday 2.x.
+
 ## v6.3.0
 - Support Ruby 3.0.
 - Drop support for Ruby < 2.5.0.

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in mauth-client.gemspec

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bundler/gem_tasks'
 require 'rspec/core/rake_task'
 require 'mauth/request_and_response'
@@ -16,7 +18,7 @@ class TestSignableRequest < MAuth::Request
   attr_accessor :headers
 
   def merge_headers(headers)
-    self.class.new(@attributes_for_signing).tap{|r| r.headers = (@headers || {}).merge(headers) }
+    self.class.new(@attributes_for_signing).tap { |r| r.headers = (@headers || {}).merge(headers) }
   end
 
   def x_mws_time
@@ -37,7 +39,7 @@ class TestSignableRequest < MAuth::Request
 end
 
 desc 'Runs benchmarks for the library.'
-task :benchmark do
+task :benchmark do # rubocop:disable Metrics/BlockLength
   mc = MAuth::Client.new(
     private_key: OpenSSL::PKey::RSA.generate(2048),
     app_uuid: SecureRandom.uuid,

data/examples/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 gem 'faraday', '~> 1.0'

data/examples/Gemfile.lock

@@ -1,13 +1,12 @@
 PATH
   remote: ..
   specs:
-    mauth-client (6.3.0)
+    mauth-client (6.4.0)
       addressable (~> 2.0)
       coderay (~> 1.0)
       dice_bag (>= 0.9, < 2.0)
-      faraday (>= 0.9, < 2.0)
+      faraday (>= 0.9, < 3.0)
       faraday-http-cache (>= 2.0, < 3.0)
-      faraday_middleware (>= 0.9, < 2.0)
       rack
       term-ansicolor (~> 1.0)
 
@@ -17,21 +16,22 @@ GEM
     addressable (2.8.0)
       public_suffix (>= 2.0.2, < 5.0)
     coderay (1.1.3)
-    dice_bag (1.6.0)
+    dice_bag (1.6.1)
       diff-lcs (~> 1.0)
       rake
       thor (< 2.0)
-    diff-lcs (1.4.4)
-    faraday (1.8.0)
+    diff-lcs (1.5.0)
+    faraday (1.9.3)
       faraday-em_http (~> 1.0)
       faraday-em_synchrony (~> 1.0)
       faraday-excon (~> 1.1)
-      faraday-httpclient (~> 1.0.1)
+      faraday-httpclient (~> 1.0)
+      faraday-multipart (~> 1.0)
       faraday-net_http (~> 1.0)
-      faraday-net_http_persistent (~> 1.1)
+      faraday-net_http_persistent (~> 1.0)
       faraday-patron (~> 1.0)
       faraday-rack (~> 1.0)
-      multipart-post (>= 1.2, < 3)
+      faraday-retry (~> 1.0)
       ruby2_keywords (>= 0.0.4)
     faraday-em_http (1.0.0)
     faraday-em_synchrony (1.0.0)
@@ -39,12 +39,13 @@ GEM
     faraday-http-cache (2.2.0)
       faraday (>= 0.8)
     faraday-httpclient (1.0.1)
+    faraday-multipart (1.0.3)
+      multipart-post (>= 1.2, < 3)
     faraday-net_http (1.0.1)
     faraday-net_http_persistent (1.2.0)
     faraday-patron (1.0.0)
     faraday-rack (1.0.0)
-    faraday_middleware (1.2.0)
-      faraday (~> 1.0)
+    faraday-retry (1.0.3)
     multipart-post (2.1.1)
     public_suffix (4.0.6)
     rack (2.2.3)
@@ -53,8 +54,8 @@ GEM
     sync (0.5.0)
     term-ansicolor (1.7.1)
       tins (~> 1.0)
-    thor (1.1.0)
-    tins (1.29.1)
+    thor (1.2.1)
+    tins (1.31.0)
       sync
 
 PLATFORMS

data/examples/get_user_info.rb

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 abort "USAGE: ./#{__FILE__} <USER UUID>" unless ARGV.size == 1
 
@@ -6,7 +7,7 @@ require 'bundler/setup'
 Bundler.require(:default)
 
 def config
-  @conf ||= YAML.load(File.open("./config.yml"))
+  @config ||= YAML.safe_load(File.open('./config.yml'))
 end
 
 # get user information
@@ -18,8 +19,8 @@ end
 def get_data_from_imedidata(resource_name)
   puts "fetching #{resource_name}..."
   begin
-    connection = Faraday::Connection.new(url: config["imedidata"]["host"]) do |builder|
-      builder.use MAuth::Faraday::RequestSigner, config["mauth"]
+    connection = Faraday::Connection.new(url: config['imedidata']['host']) do |builder|
+      builder.use MAuth::Faraday::RequestSigner, config['mauth']
       builder.adapter Faraday.default_adapter
     end
 
@@ -44,15 +45,14 @@ end
 
 get_user_info_mauth(ARGV[0])
 
-
 ### OTHER EXAMPLES
 
 #### get study groups for an user
 def get_study_groups_mauth(user_uuid)
- get_data_from_imedidata "users/#{user_uuid}/study_groups.json"
+  get_data_from_imedidata "users/#{user_uuid}/study_groups.json"
 end
 
 #### get roles for a user in an application study
 def get_user_study_roles_mauth(user_uuid, study_uuid)
- get_data_from_imedidata "users/#{user_uuid}/studies/#{study_uuid}/apps/#{config["mauth"]["app_uuid"]}/roles.json"
+  get_data_from_imedidata "users/#{user_uuid}/studies/#{study_uuid}/apps/#{config["mauth"]["app_uuid"]}/roles.json"
 end

data/exe/mauth-client

@@ -1,10 +1,9 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 $LOAD_PATH.unshift File.expand_path('../lib', File.dirname(__FILE__))
 
-require 'rubygems'
 require 'faraday'
-require 'faraday_middleware'
 require 'logger'
 require 'mauth/client'
 require 'mauth/faraday'
@@ -17,41 +16,42 @@ require 'optparse'
 
 # $options default values
 $options = {
-  :authenticate_response => true,
-  :verbose => true,
-  :color => nil,
-  :no_ssl_verify => false,
+  authenticate_response: true,
+  verbose: true,
+  color: nil,
+  no_ssl_verify: false
 }
 
 additional_headers = []
 opt_parser = OptionParser.new do |opts|
-  opts.banner = "Usage: mauth-client [options] <verb> <url> [body]"
+  opts.banner = 'Usage: mauth-client [options] <verb> <url> [body]'
 
-  opts.on("-v", "--[no-]verbose", "Run verbosely - output is like curl -v (this is the default)") do |v|
+  opts.on('-v', '--[no-]verbose', 'Run verbosely - output is like curl -v (this is the default)') do |v|
     $options[:verbose] = v
   end
-  opts.on("-q", "Run quietly - only outputs the response body (same as --no-verbose)") do |v|
+  opts.on('-q', 'Run quietly - only outputs the response body (same as --no-verbose)') do |v|
     $options[:verbose] = !v
   end
-  opts.on("--[no-]authenticate", "Authenticate the response received") do |v|
+  opts.on('--[no-]authenticate', 'Authenticate the response received') do |v|
     $options[:authenticate_response] = v
   end
-  opts.on("--[no-]color", "Color the output (defaults to color if the output device is a TTY)") do |v|
+  opts.on('--[no-]color', 'Color the output (defaults to color if the output device is a TTY)') do |v|
     $options[:color] = v
   end
-  opts.on("-t", "--content-type CONTENT-TYPE", "Sets the Content-Type header of the request") do |v|
+  opts.on('-t', '--content-type CONTENT-TYPE', 'Sets the Content-Type header of the request') do |v|
     $options[:content_type] = v
   end
-  opts.on("-H", "--header LINE", "accepts a json string of additional headers to included. IE 'cache-expirey: 10, other: value") do |v|
+  opts.on('-H', '--header LINE',
+    "accepts a json string of additional headers to included. IE 'cache-expirey: 10, other: value") do |v|
     additional_headers << v
   end
-  opts.on("--no-ssl-verify", "Disables SSL verification - use cautiously!") do
+  opts.on('--no-ssl-verify', 'Disables SSL verification - use cautiously!') do
     $options[:no_ssl_verify] = true
   end
-  $options[:additional_headers] =additional_headers
+  $options[:additional_headers] = additional_headers
 end
 opt_parser.parse!
-abort(opt_parser.help) unless (2..3).include?(ARGV.size)
+abort(opt_parser.help) unless (2..3).cover?(ARGV.size)
 
 # FIND MAUTH CONFIG
 
@@ -61,32 +61,32 @@ possible_mauth_config_files = [
   ENV['MAUTH_CONFIG_YAML'],
   '~/.mauth_config.yml',
   './config/mauth.yml',
-  './mauth.yml',
+  './mauth.yml'
 ].compact
 
 mauth_config_yml = possible_mauth_config_files.detect do |filename|
-  File.exists?(File.expand_path(filename))
+  File.exist?(File.expand_path(filename))
 end
 unless mauth_config_yml
-  message = "could not find mauth config. giving up. please place a mauth config in one of the standard places, " +
-    "or point the MAUTH_CONFIG_YML environment variable at an existing one. standard places are:" +
-    possible_mauth_config_files.map{|f| "\n\t#{f}" }.join
+  message = 'could not find mauth config. giving up. please place a mauth config in one of the standard places, ' \
+            'or point the MAUTH_CONFIG_YML environment variable at an existing one. standard places are:' +
+            possible_mauth_config_files.map { |f| "\n\t#{f}" }.join
   abort message
 end
 
-mauth_config = MAuth::Client.default_config(:mauth_config_yml => File.expand_path(mauth_config_yml))
+mauth_config = MAuth::Client.default_config(mauth_config_yml: File.expand_path(mauth_config_yml))
 
 # INSTANTIATE MAUTH CLIENT
 
-logger = Logger.new(STDERR)
+logger = Logger.new($stderr)
 mauth_client = MAuth::Client.new(mauth_config.merge('logger' => logger))
 
 # OUTPUTTERS FOR FARADAY THAT SHOULD MOVE TO A LIB SOMEWHERE
 
 # outputs the response body to the given output device (defaulting to STDOUT)
 class FaradayOutputter < Faraday::Middleware
-  def initialize(app, outdev=STDOUT)
-    @app=app
+  def initialize(app, outdev = $stdout)
+    @app = app
     @outdev = outdev
   end
 
@@ -100,7 +100,6 @@ end
 # this is to approximate `curl -v`s output. but it's all faked, whereas curl gives you
 # the real text written and read for request and response. whatever, close enough.
 class FaradayCurlVOutputter < FaradayOutputter
-
   # defines a method with the given name, applying coloring defined by any additional arguments.
   # if $options[:color] is set, respects that; otherwise, applies color if the output device is a tty.
   def self.color(name, *color_args)
@@ -129,10 +128,11 @@ class FaradayCurlVOutputter < FaradayOutputter
   color :response_header
   color :response_blankline, :intense_green, :bold
 
-  def call(request_env)
+  def call(request_env) # rubocop:disable Metrics/AbcSize
     @outdev.puts "#{info('*')} #{info_body("connect to #{request_env[:url].host} on port #{request_env[:url].port}")}"
-    @outdev.puts "#{info('*')} #{info_body("getting our SSL on")}" if request_env[:url].scheme=='https'
-    @outdev.puts "#{request('>')} #{request_verb(request_env[:method].to_s.upcase)} #{request_env[:url].path} #{protocol('HTTP/1.1' || 'or something - TODO')}"
+    @outdev.puts "#{info('*')} #{info_body("getting our SSL on")}" if request_env[:url].scheme == 'https'
+    @outdev.puts "#{request('>')} #{request_verb(request_env[:method].to_s.upcase)} #{request_env[:url].path}" \
+                 "#{protocol('HTTP/1.1' || 'or something - TODO')}"
     request_env[:request_headers].each do |k, v|
       @outdev.puts "#{request('>')} #{request_header(k)}#{request(':')} #{v}"
     end
@@ -142,11 +142,12 @@ class FaradayCurlVOutputter < FaradayOutputter
       @outdev.puts "#{request('>')} #{line}"
     end
     @app.call(request_env).on_complete do |response_env|
-      @outdev.puts "#{response('<')} #{protocol('HTTP/1.1' || 'or something - TODO')} #{response_status(response_env[:status].to_s)}"
+      @outdev.puts "#{response('<')} #{protocol('HTTP/1.1' || 'or something - TODO')} " \
+                   "#{response_status(response_env[:status].to_s)}"
       request_env[:response_headers].each do |k, v|
         @outdev.puts "#{response('<')} #{response_header(k)}#{response(':')} #{v}"
       end
-      @outdev.puts "#{response_blankline  ('<')} "
+      @outdev.puts "#{response_blankline('<')} "
       response_body = color_body_by_content_type(response_env[:body], response_env[:response_headers]['Content-Type'])
       (response_body || '').split("\n", -1).each do |line|
         @outdev.puts "#{response('<')} #{line}"
@@ -161,62 +162,61 @@ class FaradayCurlVOutputter < FaradayOutputter
 
   # a mapping for each registered CodeRay scanner to the Media Types which represent
   # that language. extremely incomplete!
-  CodeRayForMediaTypes = {
-    :c => [],
-    :cpp => [],
-    :clojure => [],
-    :css => ['text/css', 'application/css-stylesheet'],
-    :delphi => [],
-    :diff => [],
-    :erb => [],
-    :groovy => [],
-    :haml => [],
-    :html => ['text/html'],
-    :java => [],
-    :java_script => ['application/javascript', 'text/javascript', 'application/x-javascript'],
-    :json => ['application/json', %r(\Aapplication/.*\+json\z)],
-    :php => [],
-    :python => ['text/x-python'],
-    :ruby => [],
-    :sql => [],
-    :xml => ['text/xml', 'application/xml', %r(\Aapplication/.*\+xml\z)],
-    :yaml => [],
-  }
+  CODE_RAY_FOR_MEDIA_TYPES = {
+    c: [],
+    cpp: [],
+    clojure: [],
+    css: ['text/css', 'application/css-stylesheet'],
+    delphi: [],
+    diff: [],
+    erb: [],
+    groovy: [],
+    haml: [],
+    html: ['text/html'],
+    java: [],
+    java_script: ['application/javascript', 'text/javascript', 'application/x-javascript'],
+    json: ['application/json', %r{\Aapplication/.*\+json\z}],
+    php: [],
+    python: ['text/x-python'],
+    ruby: [],
+    sql: [],
+    xml: ['text/xml', 'application/xml', %r{\Aapplication/.*\+xml\z}],
+    yaml: []
+  }.freeze
 
   # takes a body and a content type; returns the body, with coloring (ansi colors for terminals)
   # possibly added, if it's a recognized content type and #color? is true
   def color_body_by_content_type(body, content_type)
-    if body && color?
-      # kinda hacky way to get the media_type. faraday should supply this ...
-      require 'rack'
-      media_type = ::Rack::Request.new({'CONTENT_TYPE' => content_type}).media_type
-      coderay_scanner = CodeRayForMediaTypes.reject{|k,v| !v.any?{|type| type === media_type} }.keys.first
-      if coderay_scanner
-        require 'coderay'
-        if coderay_scanner == :json
-          body = begin
-            JSON.pretty_generate(JSON.parse(body))
-          rescue JSON::ParserError
-            body
-          end
-        end
-        body = CodeRay.scan(body, coderay_scanner).encode(:terminal)
+    return body unless body && color?
+
+    # kinda hacky way to get the media_type. faraday should supply this ...
+    require 'rack'
+    media_type = ::Rack::Request.new({ 'CONTENT_TYPE' => content_type }).media_type
+    coderay_scanner = CODE_RAY_FOR_MEDIA_TYPES.select { |_k, v| v.any?(media_type) }.keys.first
+    return body unless coderay_scanner
+
+    require 'coderay'
+    if coderay_scanner == :json
+      body = begin
+        JSON.pretty_generate(JSON.parse(body))
+      rescue JSON::ParserError
+        body
       end
     end
-    body
+    CodeRay.scan(body, coderay_scanner).encode(:terminal)
   end
 end
 
 # CONFIGURE THE FARADAY CONNECTION
 faraday_options = {}
 if $options[:no_ssl_verify]
-  faraday_options[:ssl] = {:verify => false}
+  faraday_options[:ssl] = { verify: false }
 end
 connection = Faraday.new(faraday_options) do |builder|
-  builder.use MAuth::Faraday::MAuthClientUserAgent, "MAuth-Client CLI"
-  builder.use MAuth::Faraday::RequestSigner, :mauth_client => mauth_client
+  builder.use MAuth::Faraday::MAuthClientUserAgent, 'MAuth-Client CLI'
+  builder.use MAuth::Faraday::RequestSigner, mauth_client: mauth_client
   if $options[:authenticate_response]
-    builder.use MAuth::Faraday::ResponseAuthenticator, :mauth_client => mauth_client
+    builder.use MAuth::Faraday::ResponseAuthenticator, mauth_client: mauth_client
   end
   builder.use $options[:verbose] ? FaradayCurlVOutputter : FaradayOutputter
   builder.adapter Faraday.default_adapter
@@ -224,41 +224,38 @@ end
 
 httpmethod, url, body = *ARGV
 
-unless Faraday::Connection::METHODS.map{|m| m.to_s.downcase }.include?(httpmethod.downcase)
+unless Faraday::Connection::METHODS.map { |m| m.to_s.downcase }.include?(httpmethod.downcase)
   abort "Unrecognized HTTP method given: #{httpmethod}\n\n" + opt_parser.help
 end
 
 headers = {}
 if $options[:content_type]
   headers['Content-Type'] = $options[:content_type]
-else
-  if body
-    # I'd rather not have a default content-type, but if none is set then the HTTP adapter sets this to
-    # application/x-www-form-urlencoded anyway. application/json is a better default for our purposes.
-    headers['Content-Type'] = 'application/json'
-  end
+elsif body
+  headers['Content-Type'] = 'application/json'
+  # I'd rather not have a default content-type, but if none is set then the HTTP adapter sets this to
+  # application/x-www-form-urlencoded anyway. application/json is a better default for our purposes.
 end
 
-if $options[:additional_headers]
-  $options[:additional_headers].each do |cur|
-    raise "Headers must be in the format of [key]:[value]" unless cur.include?(':')
-    key, throw_away ,value = cur.partition(':')
-    headers[key] = value
-  end
+$options[:additional_headers]&.each do |cur|
+  raise 'Headers must be in the format of [key]:[value]' unless cur.include?(':')
+
+  key, _throw_away, value = cur.partition(':')
+  headers[key] = value
 end
 
 # OH LOOK IT'S FINALLY ACTUALLY CONNECTING TO SOMETHING
 
 begin
-  response = connection.run_request(httpmethod.downcase.to_sym, url, body, headers)
+  connection.run_request(httpmethod.downcase.to_sym, url, body, headers)
 rescue MAuth::InauthenticError, MAuth::UnableToAuthenticateError, MAuth::MAuthNotPresent, MAuth::MissingV2Error => e
-  if $options[:color].nil? ? STDERR.tty? : $options[:color]
+  if $options[:color].nil? ? $stderr.tty? : $options[:color]
     class_color = Term::ANSIColor.method(e.is_a?(MAuth::UnableToAuthenticateError) ? :intense_yellow : :intense_red)
     message_color = Term::ANSIColor.method(e.is_a?(MAuth::UnableToAuthenticateError) ? :yellow : :red)
   else
-    class_color = proc{|s| s }
-    message_color = proc{|s| s }
+    class_color = proc { |s| s }
+    message_color = proc { |s| s }
   end
-  STDERR.puts(class_color.call(e.class.to_s))
-  STDERR.puts(message_color.call(e.message))
+  warn(class_color.call(e.class.to_s))
+  warn(message_color.call(e.message))
 end

data/exe/mauth-proxy

@@ -1,14 +1,14 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 $LOAD_PATH.unshift File.expand_path('../lib', File.dirname(__FILE__))
 
-require 'rubygems'
 require 'mauth/proxy'
 require 'rack'
 
 headers = []
 headers_index = ARGV.find_index('--header')
-while(headers_index) do
+while headers_index
   headers << ARGV[headers_index + 1]
   ARGV.delete_at(headers_index + 1)
   ARGV.delete_at(headers_index)
@@ -16,25 +16,25 @@ while(headers_index) do
 end
 
 authenticate_responses = !ARGV.delete('--no-authenticate')
-browser_proxy = !(ARGV.delete('--browser_proxy').nil?)
+browser_proxy = !ARGV.delete('--browser_proxy').nil?
 
 target_uri = browser_proxy ? ARGV : ARGV.pop
 
 if !target_uri || target_uri.empty?
-  abort("Usage: mauth-proxy [rack options] --browser_proxy [--no-authenticate] <target URI> <target URI> ...\n" +
-    "or: mauth-proxy [rack options] <target URI>")
+  abort("Usage: mauth-proxy [rack options] --browser_proxy [--no-authenticate] <target URI> <target URI> ...\n" \
+        'or: mauth-proxy [rack options] <target URI>')
 end
 
 rack_server_options = Rack::Server::Options.new.parse!(ARGV)
 
-# for security, this rack server will only accept local connections, so override Host 
+# for security, this rack server will only accept local connections, so override Host
 # to 127.0.0.1 (from the default of 0.0.0.0)
 #
-# this means that the '-o' / '--host' option to Rack::Server::Options is ignored. 
-rack_server_options[:Host] = "127.0.0.1"
+# this means that the '-o' / '--host' option to Rack::Server::Options is ignored.
+rack_server_options[:Host] = '127.0.0.1'
 
-rack_server_options[:app] = MAuth::Proxy.new(target_uri, :authenticate_responses => authenticate_responses,
-  :browser_proxy => browser_proxy, :headers => headers)
+rack_server_options[:app] = MAuth::Proxy.new(target_uri, authenticate_responses: authenticate_responses,
+  browser_proxy: browser_proxy, headers: headers)
 
 mauth_proxy_server = Rack::Server.new(rack_server_options)
 mauth_proxy_server.start