redirector 1.1.4 → 1.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ccf7066f0ca506084c083b3d4894af1b402feaef3ea4a9b4da17a3798f54e571
-  data.tar.gz: b8c84d4dc6d118797c29b396d14f8e0aec3f75faac49efd13e5cb3ee007269bf
+  metadata.gz: 79bd4542eac104277c220b0b1a1c008c57358b8e1f107b1056d8299094f415d8
+  data.tar.gz: 69763dfd60e894007e85e4da50ea2b362af399bc2842a972d2b3d5a967ff31b0
 SHA512:
-  metadata.gz: f04ec1dd818575060b493db61aaad7c977072def42e1ec1ba2e3ebfba899cb0badaebdfb5f8057d19a10d9a8992c1bda24fc6e36abababb1ce50dba5940c94c8
-  data.tar.gz: 79385fc940de5d5df569b5ab9467715866cfdd6d9511a5b5e8364b2dd848f479c988ba396cf637d9bafe8e53b4093b1db935ba4e10431523065270ca071ab51b
+  metadata.gz: ac144f306013c32555b6c3981d359a9881b69429f6fe844e8470b2edaff6ea6e212e1db3ff6e396166ce7655d9945eb31544f6a452189ae4efa9e923c324607a
+  data.tar.gz: 2498d91d3421de504600032ef4690bf127157ae70cfd411cceb56c925651a2d98e3d8903378a845a000ca810cf54a97a5b2e7e406d366575a2f8f9c5b4df9c46

data/app/models/redirect_rule.rb

@@ -20,7 +20,7 @@ class RedirectRule < ActiveRecord::Base
     validates :active, :inclusion => { :in => ['0', '1', true, false] }
   end
 
-  before_save :strip_source_whitespace
+  before_save :strip_source_whitespace, :strip_destination_whitespace
 
   def self.regex_expression
     if connection_mysql?
@@ -89,4 +89,8 @@ class RedirectRule < ActiveRecord::Base
     self.source = self.source.strip
   end
 
+  def strip_destination_whitespace
+    self.destination = self.destination.strip
+  end
+
 end

data/lib/redirector/version.rb

@@ -1,3 +1,3 @@
 module Redirector
-  VERSION = "1.1.4"
+  VERSION = "1.1.5"
 end

data/spec/models/redirect_rule_spec.rb

@@ -46,6 +46,15 @@ describe RedirectRule do
     end
   end
 
+  describe 'strip_destination_whitespace before_save callback' do
+    it 'strips leading and trailing whitespace when saved' do
+      subject = build(:redirect_rule, :destination => ' /needs-stripping ')
+
+      subject.save
+      expect(subject.reload.destination).to eq('/needs-stripping')
+    end
+  end
+
   describe '.match_for' do
     it 'returns nil if there is no matching rule' do
       expect(RedirectRule.match_for('/someplace', {})).to be_nil

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: redirector
 version: !ruby/object:Gem::Version
-  version: 1.1.4
+  version: 1.1.5
 platform: ruby
 authors:
 - Brian Landau
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-04-30 00:00:00.000000000 Z
+date: 2019-12-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -258,8 +258,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.4
 signing_key: 
 specification_version: 4
 summary: A Rails engine that adds a piece of middleware to the top of your middleware