rewritten 0.8.2 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 209d70c9a3b200f7541f26509e60f3606869fd94
-  data.tar.gz: a1bb41ce3c1da38d98912211b9e894f50f987d95
+  metadata.gz: e1abdc39d5f840f0792d1f903d000b5d6d36e95b
+  data.tar.gz: 2746b966bf4d31bb8254fc1a452951a59cdb451c
 SHA512:
-  metadata.gz: ca36c2bd7e90bb093adabd289195c197477ef3a89feb6782e46f559054889ee2cd06f08f96dd8849e54bd68c078eff307e23f4e504c2c949d52c341e2954dc14
-  data.tar.gz: cd854f0f45cf703d7310881c9cf9bf1a91bddd1d628df6ab62a95ff899462984fd88458dc1fa3bfc9815215cab2290c8b74a7c34628910aee15db7a35e12c671
+  metadata.gz: 56b3daa7c907ffc0c845bb341d8348201eb486d08729882c574cf9469e7f8c3c314d4a7cd3e9d26338df6c58c6e425f1505e96f122c4fd06bf583862232bb57d
+  data.tar.gz: f6613822fafe35ccffa7a85b705395a9646421dabdf6cd1a0d5341b7ab234c9eb035916b1e0a085c8b3b475636a1d1ed6d1a18e565930cb5faa2ff57080aa7f0

data/HISTORY.rdoc

@@ -1,3 +1,8 @@
+== 0.9.0
+
+  * always chomp '/' from request_path
+  * downcase option for Rack::Rewritten::Url 
+
 == 0.8.0
 
   * introduce the [H] flag to stop translating

data/lib/rack/url.rb

@@ -9,6 +9,7 @@ module Rack
       def initialize(app, &block)
         @app = app
         @translate_backwards = false
+        @downcase_before_lookup = false
 
         instance_eval(&block) if block_given?
       end
@@ -19,10 +20,11 @@ module Rack
         subdomain = env["SUBDOMAIN"] ? "#{env["SUBDOMAIN"]}:" : ""
 
         path = "#{subdomain}#{req.path_info}"
+        path.downcase! if downcase_before_lookup?
 
-        if ::Rewritten.includes?(path) or translate_backwards? && ::Rewritten.exist_translation_for?(path) 
+        if ::Rewritten.includes?(path.chomp("/")) or translate_backwards? && ::Rewritten.exist_translation_for?(path) 
 
-          to = ::Rewritten.includes?(path) || path
+          to = ::Rewritten.includes?(path.chomp("/")) || path
 
           current_path = ::Rewritten.get_current_translation(to)
           current_path = current_path.split(":").last
@@ -69,6 +71,13 @@ module Rack
         @translate_backwards = yes_or_no
       end
 
+      def downcase_before_lookup?
+        @downcase_before_lookup
+      end
+      
+      def downcase_before_lookup=(yes_or_no)
+        @downcase_before_lookup = yes_or_no
+      end
 
       
     end

data/lib/rewritten/version.rb

@@ -1,4 +1,4 @@
 module Rewritten
-  VERSION = "0.8.2"
+  VERSION = "0.9.0"
 end
 

data/test/rack/rewritten_url_test.rb

@@ -60,6 +60,41 @@ describe Rack::Rewritten::Url do
       ret[0].must_equal 200
     end
 
+    describe "/ behavior" do
+      
+      it "must 301 redirect paths with / in the end to their chomped version" do
+        ret = @rack.call request_url('/foo/bar/')
+        @app.verify
+        ret[0].must_equal 301
+        ret[1]['Location'].must_equal "http://www.example.org/foo/baz"
+      end
+
+    end
+
+    describe "caps behavior" do
+      
+      it "must diferentiate caps" do
+        @app.expect :call, [200, {'Content-Type' => 'text/plain'},[""]], [Hash]
+        ret = @rack.call request_url('/Foo/Bar')
+        @app.verify
+        ret[0].must_equal 200
+      end
+
+      it "must ignore caps if wished" do
+        @rack = Rack::Rewritten::Url.new(@app) do
+          self.downcase_before_lookup = true
+        end
+
+        ret = @rack.call request_url('/Foo/Bar')
+
+        @app.verify
+        ret[0].must_equal 301
+        ret[1]['Location'].must_equal "http://www.example.org/foo/baz"
+      end
+
+    end
+
+
     describe "enforce nice urls" do
 
       it "must not redirect from resource url to nice url by default" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rewritten
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.9.0
 platform: ruby
 authors:
 - Kai Rubarth