quiet_safari 1.0.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 051cc441a2b0111537de1f6a805be1e7d1a98a2e
+  data.tar.gz: 5292bc813a3d4d36531d38cf6417ddbd6754e15f
+SHA512:
+  metadata.gz: fd1ee12ff5b3581b8fcc5cfc4518a066a762e84bfd08f4eee527a183b096eb4d8486d4e6d453be24a97d3028e439ca5fba68341cc6716631e5916c7793afead2
+  data.tar.gz: e96182f3e3f37b0862960174bd26084f8cc084e016a871950d469de638d21460de565293b67a7b5e943d07d997ffa52e120ede60689606ff920222cb6cbaa099

data/lib/quiet_safari.rb

@@ -0,0 +1,33 @@
+require 'quiet_safari/version'
+require 'logger'
+
+module QuietSafari
+  class Engine < ::Rails::Engine
+    APPL = /apple-touch-icon(-precomposed)?\.png/
+    KEY = 'quiet_safari.old_rails_log_level'
+
+    initializer 'quiet_safari' do |app|
+      app.config.quiet_safari = true
+
+      next if Rails::Rack::Logger.instance_methods.include?(:call_with_quiet_safari)
+
+      Rails::Rack::Logger.class_eval do
+        def call_with_quiet_safari(env)
+          begin
+            if Rails.application.config.quiet_safari && env['PATH_INFO'] =~ APPL
+              env[KEY]           = Rails.logger.level
+              Rails.logger.level = Logger::UNKNOWN
+            end
+
+            call_without_quiet_safari(env)
+          ensure
+            Rails.logger.level = env[KEY] if env[KEY]
+          end
+        end
+
+        alias :call_without_quiet_safari :call
+        alias :call :call_with_quiet_safari
+      end
+    end
+  end
+end

data/lib/quiet_safari/version.rb

@@ -0,0 +1,13 @@
+module QuietSafari
+  class Version
+    MAJOR = 1
+    MINOR = 0
+    PATCH = 0
+
+    def self.to_s
+      [MAJOR, MINOR, PATCH].join('.')
+    end
+  end
+
+  VERSION = Version.to_s
+end

data/spec/quiet_safari_spec.rb

@@ -0,0 +1,75 @@
+require 'spec_helper'
+
+require 'rails'
+require 'rails/all'
+
+require 'quiet_safari'
+
+RSpec.describe QuietSafari do
+  def app
+    @app ||= Class.new(Rails::Application) do
+      config.active_support.deprecation = :notify
+      config.secret_token = '685e1a60792fa0d036a82a52c0f97e42'
+      config.eager_load = false
+
+      initialize!
+    end
+  end
+
+  let(:request)  { Rack::MockRequest.env_for(path) }
+  let(:response) { StringIO.new }
+
+  before do
+    Rails.logger = Logger.new(response)
+    Rails.logger.formatter = lambda { |s, d, p, m| "#{m}\n" }
+  end
+
+  shared_examples_for 'a quiet request' do
+    before { app.config.quiet_safari = true }
+
+    it 'does not output errors' do
+      app.call request
+
+      expect(response.string).to be_empty
+    end
+  end
+
+  context 'configured to be quiet' do
+    context 'requesting /apple-touch-icon.png' do
+      let(:path) { '/apple-touch-icon.png' }
+
+      it_behaves_like 'a quiet request'
+    end
+
+    context 'requesting /apple-touch-icon-precomposed.png' do
+      let(:path) { '/apple-touch-icon-precomposed.png' }
+
+      it_behaves_like 'a quiet request'
+    end
+  end
+
+  shared_examples_for 'a noisy request' do
+    before { app.config.quiet_safari = false }
+
+    it 'outputs errors' do
+      app.call request
+
+      expect(response.string).to include("Started GET \"#{path}\"")
+      expect(response.string).to include('ActionController::RoutingError')
+    end
+  end
+
+  context 'not configured to be quiet' do
+    context 'requesting /apple-touch-icon.png' do
+      let(:path) { '/apple-touch-icon.png' }
+
+      it_behaves_like 'a noisy request'
+    end
+
+    context 'requesting /apple-touch-icon-precomposed.png' do
+      let(:path) { '/apple-touch-icon-precomposed.png' }
+
+      it_behaves_like 'a noisy request'
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,88 @@
+# This file was generated by the `rspec --init` command. Conventionally, all
+# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
+# The generated `.rspec` file contains `--require spec_helper` which will cause
+# this file to always be loaded, without a need to explicitly require it in any
+# files.
+#
+# Given that it is always loaded, you are encouraged to keep this file as
+# light-weight as possible. Requiring heavyweight dependencies from this file
+# will add to the boot time of your test suite on EVERY test run, even for an
+# individual file that may not need all of that loaded. Instead, consider making
+# a separate helper file that requires the additional dependencies and performs
+# the additional setup, and require it from the spec files that actually need
+# it.
+#
+# The `.rspec` file also contains a few flags that are not defaults but that
+# users commonly want.
+#
+# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
+
+RSpec.configure do |config|
+  # rspec-expectations config goes here. You can use an alternate
+  # assertion/expectation library such as wrong or the stdlib/minitest
+  # assertions if you prefer.
+  config.expect_with :rspec do |expectations|
+    # This option will default to `true` in RSpec 4. It makes the `description`
+    # and `failure_message` of custom matchers include text for helper methods
+    # defined using `chain`, e.g.:
+    #     be_bigger_than(2).and_smaller_than(4).description
+    #     # => "be bigger than 2 and smaller than 4"
+    # ...rather than:
+    #     # => "be bigger than 2"
+    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
+  end
+
+  # rspec-mocks config goes here. You can use an alternate test double
+  # library (such as bogus or mocha) by changing the `mock_with` option here.
+  config.mock_with :rspec do |mocks|
+    # Prevents you from mocking or stubbing a method that does not exist on
+    # a real object. This is generally recommended, and will default to
+    # `true` in RSpec 4.
+    mocks.verify_partial_doubles = true
+  end
+
+  # These two settings work together to allow you to limit a spec run
+  # to individual examples or groups you care about by tagging them with
+  # `:focus` metadata. When nothing is tagged with `:focus`, all examples
+  # get run.
+  config.filter_run :focus
+  config.run_all_when_everything_filtered = true
+
+  # Limits the available syntax to the non-monkey patched syntax that is
+  # recommended. For more details, see:
+  #   - http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax
+  #   - http://teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/
+  #   - http://myronmars.to/n/dev-blog/2014/05/notable-changes-in-rspec-3#new__config_option_to_disable_rspeccore_monkey_patching
+  config.disable_monkey_patching!
+
+  # This setting enables warnings. It's recommended, but in some cases may
+  # be too noisy due to issues in dependencies.
+  config.warnings = true
+
+  # Many RSpec users commonly either run the entire suite or an individual
+  # file, and it's useful to allow more verbose output when running an
+  # individual spec file.
+  if config.files_to_run.one?
+    # Use the documentation formatter for detailed output,
+    # unless a formatter has already been configured
+    # (e.g. via a command-line flag).
+    config.default_formatter = 'doc'
+  end
+
+  # Print the 10 slowest examples and example groups at the
+  # end of the spec run, to help surface which specs are running
+  # particularly slow.
+  config.profile_examples = 10
+
+  # Run specs in random order to surface order dependencies. If you find an
+  # order dependency and want to debug it, you can fix the order by providing
+  # the seed, which is printed after each run.
+  #     --seed 1234
+  config.order = :random
+
+  # Seed global randomization in this process using the `--seed` CLI option.
+  # Setting this allows you to use `--seed` to deterministically reproduce
+  # test failures related to randomization by passing the same `--seed` value
+  # as the one that triggered the failure.
+  Kernel.srand config.seed
+end

metadata

@@ -0,0 +1,82 @@
+--- !ruby/object:Gem::Specification
+name: quiet_safari
+version: !ruby/object:Gem::Version
+  version: 1.0.0
+platform: ruby
+authors:
+- David Celis
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2015-02-20 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: railties
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.1.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.1.0
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.2'
+description: |
+  Get rid of ActionController::RoutingErrors that happen when Safari requests PNGs
+  that don't exist, such as `apple-touch-icon.png` or, of course, the illustrious
+  `apple-touch-icon-precomposed.png`. Because who cares?
+email:
+- me@davidcel.is
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- lib/quiet_safari.rb
+- lib/quiet_safari/version.rb
+- spec/quiet_safari_spec.rb
+- spec/spec_helper.rb
+homepage: https://github.com/davidcelis/quiet_safari
+licenses:
+- MIT
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.4.5
+signing_key: 
+specification_version: 4
+summary: Because who cares about the apple-touch-icon?
+test_files:
+- spec/quiet_safari_spec.rb
+- spec/spec_helper.rb
+has_rdoc: