prx_auth-rails 4.1.0 → 4.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d46435a82e0473d353a1f1849bfcedfb4db925e5a1bce443a8043ca948bfda69
-  data.tar.gz: b867f26410a93aee077e2bad3515b2fb9fb0ee4a9499cbb6bdaef9287639c158
+  metadata.gz: 297ff153a8884223afeca8d74d809f965e297e1766acf522ef442070d35682ab
+  data.tar.gz: 97af5447b5a9d5449a289f055680c42b7c213503f3ba3fa4af4825f9083da479
 SHA512:
-  metadata.gz: 308dd3bc5e3eacf014613bac983b097d677f823d60185eb76303345d698f1096e2fa7e24ad74b2f7bf5a2eef4a3222a9bf9ec51a28c1d82698bad48de8d500ad
-  data.tar.gz: b2bf8e7fe515a27e970a4612b7075564366f0a0270c62dca1602e6d51d1dceb7ffe4d5e2143927126bb02b4335bcd09dac501f950c5511cf456c4ffaa309cd42
+  metadata.gz: 32017811c9336ecdfc7e93196918ebbed6fa04c6616c0f36453319c3bdc2c4caa309fe47b3fc40573ab206efee375193867ca36a0466b0366a6980d0d2761e1f
+  data.tar.gz: 578ed2609ec2213d5bbb8397cdf2fabc82b71e43e59b26128df26bd6075a219712ecd15ec23143f4ef54903cfb02f5bd7afb8140cb8ff35323213880c5b3d95b

data/.git-blame-ignore-revs

@@ -0,0 +1,2 @@
+# 2023-04-12 Auto-fix all Ruby files with standardrb
+3da043d8a33e29223834fa3d7a05b013e490ce62

data/.github/workflows/check-project-std.yml

@@ -0,0 +1,23 @@
+name: Check project standards
+
+on:
+  push
+
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    env:
+      RAILS_ENV: test
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+        with:
+          fetch-depth: 0
+      - name: Install Ruby and gems
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: '3.0'
+          bundler-cache: true
+      - name: Lint Ruby files
+        run: bundle exec standardrb

data/Gemfile

@@ -1,4 +1,4 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in prx_auth-rails.gemspec
 gemspec

data/Guardfile

@@ -1,8 +1,8 @@
 guard :minitest, all_after_pass: true do
-  watch(%r{^test/(.*)\/?test_(.*)\.rb})
-  watch(%r{^lib/(.*/)?([^/]+)\.rb})                      { |m| "test/#{m[1]}test_#{m[2]}.rb" }
-  watch(%r{^lib/(.+)\.rb})                               { |m| "test/#{m[1]}_test.rb" }
-  watch(%r{^lib/(.+)\.rb})                                    { |m| "test/#{m[1]}_test.rb" }
+  watch(%r{^test/(.*)/?test_(.*)\.rb})
+  watch(%r{^lib/(.*/)?([^/]+)\.rb}) { |m| "test/#{m[1]}test_#{m[2]}.rb" }
+  watch(%r{^lib/(.+)\.rb}) { |m| "test/#{m[1]}_test.rb" }
+  watch(%r{^lib/(.+)\.rb}) { |m| "test/#{m[1]}_test.rb" }
   watch(%r{^test/.+_test\.rb})
-  watch(%r{^test/test_helper\.rb})                            { 'test' }
+  watch(%r{^test/test_helper\.rb}) { "test" }
 end

data/Rakefile

@@ -6,12 +6,12 @@ load "rails/tasks/engine.rake"
 load "rails/tasks/statistics.rake"
 
 require "bundler/gem_tasks"
-require 'rake'
+require "rake"
 require "rake/testtask"
 
 Rake::TestTask.new(:test) do |t|
-  t.libs << 'test'
-  t.pattern = 'test/**/*_test.rb'
+  t.libs << "test"
+  t.pattern = "test/**/*_test.rb"
   t.verbose = false
 end
 

data/app/controllers/prx_auth/rails/sessions_controller.rb

@@ -9,8 +9,8 @@ module PrxAuth::Rails
     before_action :set_nonce!, only: [:new, :show]
     before_action :set_after_sign_in_path
 
-    ID_NONCE_SESSION_KEY = 'id_prx_openid_nonce'
-    DEFAULT_SCOPES = 'openid apps'
+    ID_NONCE_SESSION_KEY = "id_prx_openid_nonce"
+    DEFAULT_SCOPES = "openid"
 
     def new
       config = PrxAuth::Rails.configuration
@@ -25,12 +25,12 @@ module PrxAuth::Rails
       id_auth_params = {
         client_id: config.prx_client_id,
         nonce: fetch_nonce,
-        response_type: 'id_token token',
+        response_type: "id_token token",
         scope: scope,
-        prompt: 'necessary'
+        prompt: "necessary"
       }
 
-      url = '//' + config.id_host + '/authorize?' + id_auth_params.to_query
+      url = "//" + config.id_host + "/authorize?" + id_auth_params.to_query
 
       redirect_to url, allow_other_host: true
     end
@@ -48,13 +48,14 @@ module PrxAuth::Rails
     end
 
     def create
-      if valid_nonce? && users_match?
-        clear_nonce!
+      valid_and_matching = valid_nonce? && users_match?
+      clear_nonce!
+
+      if valid_and_matching
         sign_in_user(access_token)
         redirect_to after_sign_in_path_for(current_user)
       else
-        clear_nonce!
-        redirect_to auth_error_sessions_path(error: params[:error] || 'unknown_error')
+        redirect_to auth_error_sessions_path(error: params[:error] || "unknown_error")
       end
     end
 
@@ -67,7 +68,7 @@ module PrxAuth::Rails
       elsif defined?(super)
         super
       else
-        '/'
+        "/"
       end
     end
 
@@ -78,11 +79,11 @@ module PrxAuth::Rails
     end
 
     def id_token
-      params.require('id_token')
+      params.require("id_token")
     end
 
     def access_token
-      params.require('access_token')
+      params.require("access_token")
     end
 
     def id_claims
@@ -106,11 +107,11 @@ module PrxAuth::Rails
     end
 
     def valid_nonce?
-      id_claims['nonce'].present? && id_claims['nonce'] == fetch_nonce
+      id_claims["nonce"].present? && id_claims["nonce"] == fetch_nonce
     end
 
     def users_match?
-      id_claims['sub'].present? && id_claims['sub'] == access_claims['sub']
+      id_claims["sub"].present? && id_claims["sub"] == access_claims["sub"]
     end
 
     def validate_token(token)

data/app/views/prx_auth/rails/sessions/auth_error.html.erb

@@ -1,13 +1,6 @@
 <div class='main'>
   <section>
     <h3>Not authorized for this application.</h3>
-
-    <p>Message was: <pre><%= @auth_error_message %></pre>
-    <% if  @auth_error_message == 'invalid_scope' %>
-      Did you add a row in the account_applications table on id.prx?
-    <% end %>
-    </p>
-
     <p>
       <a href="<%= new_sessions_path %>">Try logging in again</a>
     </p>

data/config/initializers/assets.rb

@@ -1 +1 @@
-Rails.application.config.assets.precompile << %w(prx_auth-rails_manifest.js)
+Rails.application.config.assets.precompile << %w[prx_auth-rails_manifest.js]

data/config/routes.rb

@@ -1,7 +1,7 @@
 PrxAuth::Rails::Engine.routes.draw do
-  scope module: 'prx_auth/rails' do
-    resource 'sessions', except: :index, :defaults => { :format => 'html' } do
-      get 'auth_error', to: 'sessions#auth_error'
+  scope module: "prx_auth/rails" do
+    resource "sessions", except: :index, defaults: {format: "html"} do
+      get "auth_error", to: "sessions#auth_error"
     end
   end
 end

data/lib/prx_auth/rails/configuration.rb

@@ -1,13 +1,13 @@
 class PrxAuth::Rails::Configuration
   attr_accessor :install_middleware,
-                :namespace,
-                :prx_client_id,
-                :prx_scope,
-                :id_host,
-                :cert_path
+    :namespace,
+    :prx_client_id,
+    :prx_scope,
+    :id_host,
+    :cert_path
 
-  DEFAULT_ID_HOST = 'id.prx.org'
-  DEFAULT_CERT_PATH = 'api/v1/certs'
+  DEFAULT_ID_HOST = "id.prx.org"
+  DEFAULT_CERT_PATH = "api/v1/certs"
 
   def initialize
     @install_middleware = true
@@ -21,15 +21,15 @@ class PrxAuth::Rails::Configuration
       if defined?(::Rails)
         klass = ::Rails.application.class
         parent_name = if ::Rails::VERSION::MAJOR >= 6
-                        klass.module_parent_name
-                      else
-                        klass.parent_name
-                      end
+          klass.module_parent_name
+        else
+          klass.parent_name
+        end
         klass_name = if parent_name.present?
-                       parent_name
-                     else
-                       klass.name
-                     end
+          parent_name
+        else
+          klass.name
+        end
 
         klass_name.underscore.intern
       end

data/lib/prx_auth/rails/engine.rb

@@ -5,7 +5,7 @@ module PrxAuth
         ::ApplicationController.helper_method [
           :current_user, :prx_jwt,
           :current_user_info, :current_user_name, :current_user_apps,
-          :account_name_for, :account_for, :accounts_for,
+          :account_name_for, :account_for, :accounts_for
         ]
       end
     end

data/lib/prx_auth/rails/ext/controller.rb

@@ -1,18 +1,18 @@
-require 'prx_auth/rails/token'
-require 'open-uri'
+require "prx_auth/rails/token"
+require "open-uri"
 
 module PrxAuth
   module Rails
     module Controller
       class SessionTokenExpiredError < RuntimeError; end
 
-      PRX_AUTH_ENV_KEY = 'prx.auth'.freeze
-      PRX_JWT_SESSION_KEY = 'prx.auth.jwt'.freeze
+      PRX_AUTH_ENV_KEY = "prx.auth".freeze
+      PRX_JWT_SESSION_KEY = "prx.auth.jwt".freeze
       # subtracted from the JWT ttl
-      PRX_JWT_REFRESH_TTL = 300.freeze
-      PRX_ACCOUNT_MAPPING_SESSION_KEY = 'prx.auth.account.mapping'.freeze
-      PRX_USER_INFO_SESSION_KEY = 'prx.auth.info'.freeze
-      PRX_REFRESH_BACK_KEY = 'prx.auth.back'.freeze
+      PRX_JWT_REFRESH_TTL = 300
+      PRX_ACCOUNT_MAPPING_SESSION_KEY = "prx.auth.account.mapping".freeze
+      PRX_USER_INFO_SESSION_KEY = "prx.auth.info".freeze
+      PRX_REFRESH_BACK_KEY = "prx.auth.back".freeze
 
       def prx_auth_token
         env_token || session_token
@@ -24,7 +24,7 @@ module PrxAuth
       end
 
       def set_after_sign_in_path
-        return if self.class == PrxAuth::Rails::SessionsController
+        return if instance_of?(PrxAuth::Rails::SessionsController)
 
         session[PRX_REFRESH_BACK_KEY] = request.fullpath
       end
@@ -54,17 +54,17 @@ module PrxAuth
       def current_user_info
         session[PRX_USER_INFO_SESSION_KEY] ||= begin
           info = fetch_userinfo
-          info.slice('name', 'preferred_username', 'email', 'image_href', 'apps')
+          info.slice("name", "preferred_username", "email", "image_href", "apps")
         end
       end
 
       def current_user_name
-        current_user_info['name'] || current_user_info['preferred_username'] || current_user_info['email']
+        current_user_info["name"] || current_user_info["preferred_username"] || current_user_info["email"]
       end
 
       def current_user_apps
-        apps = (current_user_info.try(:[], 'apps') || []).map do |name, url|
-          label = name.sub(/^https?:\/\//, '').sub(/\..+/, '').capitalize
+        apps = (current_user_info.try(:[], "apps") || []).map do |name, url|
+          label = name.sub(/^https?:\/\//, "").sub(/\..+/, "").capitalize
           ["PRX #{label}", url]
         end
 
@@ -90,7 +90,7 @@ module PrxAuth
       end
 
       def account_name_for(account_id)
-        account_for(account_id).try(:[], 'name')
+        account_for(account_id).try(:[], "name")
       end
 
       def account_for(account_id)
@@ -110,8 +110,8 @@ module PrxAuth
         missing = ids - session[PRX_ACCOUNT_MAPPING_SESSION_KEY].keys
         if missing.present?
           fetch_accounts(missing).each do |account|
-            minimal = account.slice('name', 'type')
-            session[PRX_ACCOUNT_MAPPING_SESSION_KEY][account['id']] = minimal
+            minimal = account.slice("name", "type")
+            session[PRX_ACCOUNT_MAPPING_SESSION_KEY][account["id"]] = minimal
           end
         end
 
@@ -119,9 +119,9 @@ module PrxAuth
       end
 
       def fetch_accounts(ids)
-        ids_param = ids.map(&:to_s).join(',')
+        ids_param = ids.map(&:to_s).join(",")
         resp = fetch("/api/v1/accounts?account_ids=#{ids_param}")
-        resp.try(:[], '_embedded').try(:[], 'prx:items') || []
+        resp.try(:[], "_embedded").try(:[], "prx:items") || []
       end
 
       def fetch_userinfo
@@ -132,8 +132,8 @@ module PrxAuth
         url = "https://#{PrxAuth::Rails.configuration.id_host}#{path}"
         options = {}
         options[:ssl_verify_mode] = OpenSSL::SSL::VERIFY_NONE if ::Rails.env.development?
-        options['Authorization'] = "Bearer #{token}" if token
-        JSON.parse(URI.open(url, options).read)
+        options["Authorization"] = "Bearer #{token}" if token
+        JSON.parse(URI.open(url, options).read) # standard:disable Security/Open
       end
 
       # token from data set by prx_auth rack middleware

data/lib/prx_auth/rails/railtie.rb

@@ -1,6 +1,6 @@
-require 'rails/railtie'
-require 'prx_auth/rails/ext/controller'
-require 'rack/prx_auth'
+require "rails/railtie"
+require "prx_auth/rails/ext/controller"
+require "rack/prx_auth"
 
 module PrxAuth::Rails
   class Railtie < ::Rails::Railtie

data/lib/prx_auth/rails/token.rb

@@ -1,4 +1,4 @@
-require 'rack/prx_auth'
+require "rack/prx_auth"
 
 class PrxAuth::Rails::Token
   def initialize(token_data)
@@ -6,17 +6,17 @@ class PrxAuth::Rails::Token
     @namespace = PrxAuth::Rails.configuration.namespace
   end
 
-  def authorized?(resource, namespace=nil, scope=nil)
+  def authorized?(resource, namespace = nil, scope = nil)
     namespace, scope = @namespace, namespace if scope.nil? && !namespace.nil?
     @token_data.authorized?(resource, namespace, scope)
   end
 
-  def globally_authorized?(namespace, scope=nil)
+  def globally_authorized?(namespace, scope = nil)
     namespace, scope = @namespace, namespace if scope.nil?
     @token_data.globally_authorized?(namespace, scope)
   end
 
-  def resources(namespace=nil, scope=nil)
+  def resources(namespace = nil, scope = nil)
     namespace, scope = @namespace, namespace if scope.nil? && !namespace.nil?
     @token_data.resources(namespace, scope)
   end
@@ -32,4 +32,17 @@ class PrxAuth::Rails::Token
   def authorized_account_ids(scope)
     @token_data.authorized_account_ids(scope)
   end
+
+  def except!(*resources)
+    @token_data = @token_data.except(*resources)
+    self
+  end
+
+  def except(*resources)
+    dup.except!(*resources)
+  end
+
+  def empty_resources?
+    @token_data.empty_resources?
+  end
 end

data/lib/prx_auth/rails/version.rb

@@ -2,6 +2,6 @@
 
 module PrxAuth
   module Rails
-    VERSION = '4.1.0'
+    VERSION = "4.2.0"
   end
 end

data/lib/prx_auth/rails.rb

@@ -27,10 +27,10 @@ module PrxAuth
         host = configuration.id_host
         path = configuration.cert_path
         protocol =
-          if host.include?('localhost') || host.include?('127.0.0.1')
-            'http'
+          if host.include?("localhost") || host.include?("127.0.0.1")
+            "http"
           else
-            'https'
+            "https"
           end
 
         app.middleware.insert_after Rack::Head, Rack::PrxAuth,

data/prx_auth-rails.gemspec

@@ -1,37 +1,36 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'prx_auth/rails/version'
+require "prx_auth/rails/version"
 
 Gem::Specification.new do |spec|
-  spec.name          = "prx_auth-rails"
-  spec.version       = PrxAuth::Rails::VERSION
-  spec.authors       = ["Chris Rhoden"]
-  spec.email         = ["carhoden@gmail.com"]
-  spec.description   = "Rails integration for next generation PRX Authorization system."
-  spec.summary       = "Rails integration for next generation PRX Authorization system."
-  spec.homepage      = "https://github.com/PRX/prx_auth-rails"
-  spec.license       = "MIT"
+  spec.name = "prx_auth-rails"
+  spec.version = PrxAuth::Rails::VERSION
+  spec.authors = ["Chris Rhoden"]
+  spec.email = ["carhoden@gmail.com"]
+  spec.description = "Rails integration for next generation PRX Authorization system."
+  spec.summary = "Rails integration for next generation PRX Authorization system."
+  spec.homepage = "https://github.com/PRX/prx_auth-rails"
+  spec.license = "MIT"
 
-  spec.required_ruby_version = '>= 2.3'
+  spec.required_ruby_version = ">= 2.3"
 
-  spec.files         = `git ls-files`.split($/)
-  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.files = `git ls-files`.split($/)
+  spec.executables = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_runtime_dependency 'actionpack'
+  spec.add_runtime_dependency "actionpack"
 
-  spec.add_development_dependency 'bundler'
-  spec.add_development_dependency 'rake'
-  spec.add_development_dependency 'coveralls', '~> 0'
-  spec.add_development_dependency 'guard'
-  spec.add_development_dependency 'guard-minitest'
-  spec.add_development_dependency 'm', '~> 1.5'  
+  spec.add_development_dependency "bundler"
+  spec.add_development_dependency "rake"
+  spec.add_development_dependency "coveralls", "~> 0"
+  spec.add_development_dependency "guard"
+  spec.add_development_dependency "guard-minitest"
+  spec.add_development_dependency "m", "~> 1.5"
   spec.add_development_dependency "rails", "~> 6.1.0"
-  spec.add_development_dependency 'pry'
-  spec.add_development_dependency 'sqlite3'
-  spec.add_development_dependency 'webmock'
+  spec.add_development_dependency "pry"
+  spec.add_development_dependency "sqlite3"
+  spec.add_development_dependency "webmock"
+  spec.add_development_dependency "standard"
 
-  spec.add_runtime_dependency 'prx_auth', ">= 1.7.0"
+  spec.add_runtime_dependency "prx_auth", ">= 1.8.0"
 end

data/test/dummy/app/controllers/application_controller.rb

@@ -1,10 +1,10 @@
 class ApplicationController < ActionController::Base
-
   before_action :authenticate!
 
-  def index; end
+  def index
+  end
 
   def after_sign_in_path_for(_resource)
-    '/after-sign-in-path'
+    "/after-sign-in-path"
   end
 end

data/test/dummy/app/mailers/application_mailer.rb

@@ -1,4 +1,4 @@
 class ApplicationMailer < ActionMailer::Base
-  default from: 'from@example.com'
-  layout 'mailer'
+  default from: "from@example.com"
+  layout "mailer"
 end

data/test/dummy/bin/rails

@@ -1,5 +1,5 @@
 #!/usr/bin/env ruby
 load File.expand_path("spring", __dir__)
-APP_PATH = File.expand_path('../config/application', __dir__)
+APP_PATH = File.expand_path("../config/application", __dir__)
 require_relative "../config/boot"
 require "rails/commands"

data/test/dummy/bin/setup

@@ -2,7 +2,7 @@
 require "fileutils"
 
 # path to your application root.
-APP_ROOT = File.expand_path('..', __dir__)
+APP_ROOT = File.expand_path("..", __dir__)
 
 def system!(*args)
   system(*args) || abort("\n== Command #{args} failed ==")
@@ -13,9 +13,9 @@ FileUtils.chdir APP_ROOT do
   # This script is idempotent, so that you can run it at any time and get an expectable outcome.
   # Add necessary setup steps to this file.
 
-  puts '== Installing dependencies =='
-  system! 'gem install bundler --conservative'
-  system('bundle check') || system!('bundle install')
+  puts "== Installing dependencies =="
+  system! "gem install bundler --conservative"
+  system("bundle check") || system!("bundle install")
 
   # puts "\n== Copying sample files =="
   # unless File.exist?('config/database.yml')
@@ -23,11 +23,11 @@ FileUtils.chdir APP_ROOT do
   # end
 
   puts "\n== Preparing database =="
-  system! 'bin/rails db:prepare'
+  system! "bin/rails db:prepare"
 
   puts "\n== Removing old logs and tempfiles =="
-  system! 'bin/rails log:clear tmp:clear'
+  system! "bin/rails log:clear tmp:clear"
 
   puts "\n== Restarting application server =="
-  system! 'bin/rails restart'
+  system! "bin/rails restart"
 end

data/test/dummy/config/boot.rb

@@ -1,5 +1,5 @@
 # Set up gems listed in the Gemfile.
-ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../../Gemfile', __dir__)
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../../Gemfile", __dir__)
 
 require "bundler/setup" if File.exist?(ENV["BUNDLE_GEMFILE"])
-$LOAD_PATH.unshift File.expand_path('../../../lib', __dir__)
+$LOAD_PATH.unshift File.expand_path("../../../lib", __dir__)

data/test/dummy/config/environments/development.rb

@@ -16,13 +16,13 @@ Rails.application.configure do
 
   # Enable/disable caching. By default caching is disabled.
   # Run rails dev:cache to toggle caching.
-  if Rails.root.join('tmp', 'caching-dev.txt').exist?
+  if Rails.root.join("tmp", "caching-dev.txt").exist?
     config.action_controller.perform_caching = true
     config.action_controller.enable_fragment_cache_logging = true
 
     config.cache_store = :memory_store
     config.public_file_server.headers = {
-      'Cache-Control' => "public, max-age=#{2.days.to_i}"
+      "Cache-Control" => "public, max-age=#{2.days.to_i}"
     }
   else
     config.action_controller.perform_caching = false

data/test/dummy/config/environments/production.rb

@@ -13,7 +13,7 @@ Rails.application.configure do
   config.eager_load = true
 
   # Full error reports are disabled and caching is turned on.
-  config.consider_all_requests_local       = false
+  config.consider_all_requests_local = false
   config.action_controller.perform_caching = true
 
   # Ensures that a master key has been made available in either ENV["RAILS_MASTER_KEY"]
@@ -22,7 +22,7 @@ Rails.application.configure do
 
   # Disable serving static files from the `/public` folder by default since
   # Apache or NGINX already handles this.
-  config.public_file_server.enabled = ENV['RAILS_SERVE_STATIC_FILES'].present?
+  config.public_file_server.enabled = ENV["RAILS_SERVE_STATIC_FILES"].present?
 
   # Compress CSS using a preprocessor.
   # config.assets.css_compressor = :sass
@@ -53,7 +53,7 @@ Rails.application.configure do
   config.log_level = :info
 
   # Prepend all log lines with the following tags.
-  config.log_tags = [ :request_id ]
+  config.log_tags = [:request_id]
 
   # Use a different cache store in production.
   # config.cache_store = :mem_cache_store
@@ -89,9 +89,9 @@ Rails.application.configure do
   # config.logger = ActiveSupport::TaggedLogging.new(Syslog::Logger.new 'app-name')
 
   if ENV["RAILS_LOG_TO_STDOUT"].present?
-    logger           = ActiveSupport::Logger.new(STDOUT)
+    logger = ActiveSupport::Logger.new($stdout)
     logger.formatter = config.log_formatter
-    config.logger    = ActiveSupport::TaggedLogging.new(logger)
+    config.logger = ActiveSupport::TaggedLogging.new(logger)
   end
 
   # Do not dump schema after migrations.

data/test/dummy/config/environments/test.rb

@@ -19,11 +19,11 @@ Rails.application.configure do
   # Configure public file server for tests with Cache-Control for performance.
   config.public_file_server.enabled = true
   config.public_file_server.headers = {
-    'Cache-Control' => "public, max-age=#{1.hour.to_i}"
+    "Cache-Control" => "public, max-age=#{1.hour.to_i}"
   }
 
   # Show full error reports and disable caching.
-  config.consider_all_requests_local       = true
+  config.consider_all_requests_local = true
   config.action_controller.perform_caching = false
   config.cache_store = :null_store