treat empty path as a directory that needs a slash

Signed-off-by: Sarah McAlear <[email protected]>

Author: Gregg Van Hove

spec/template_app/lib/server_spec.rb

@@ -5,4 +5,9 @@
     response = Bookbinder::Server.new.call({'PATH_INFO' => '/foo/path'})
     expect(response).to eq [301, {"Location"=>"://::0/foo/path/", "Content-Type"=>""}, []]
   end
+
+  it 'adds a / to an empty path' do
+    response = Bookbinder::Server.new.call({'PATH_INFO' => ''})
+    expect(response).to eq [301, {"Location"=>"://::0/", "Content-Type"=>""}, []]
+  end
 end

template_app/lib/server.rb

@@ -24,7 +24,7 @@ def initialize
     end
 
     def call(env)
-      if env['PATH_INFO'] != '/' && env['PATH_INFO'] =~ MATCH
+      if env['PATH_INFO'] == '' || env['PATH_INFO'] =~ MATCH
         env['PATH_INFO'] += '/'
         [301, {'Location' => Rack::Request.new(env).url, 'Content-Type' => ''}, []]
       else
@@ -33,8 +33,8 @@ def call(env)
     end
 
     private
-    # regexp to match strings without periods that start and end with a slash
-    MATCH = %r{^/([^.]*)[^/]$}
+    # regexp to match strings without periods that start but don't end with a slash
+    MATCH = %r{^/([^.]+)[^/]$}
   end
 
   class NotFound