Merge pull request laravel#69 from lukasgeiter/fix-trailing-slash

Fix trailing slash

src/Application.php

@@ -1348,7 +1348,7 @@ public function getPathInfo()
     {
         $query = isset($_SERVER['QUERY_STRING']) ? $_SERVER['QUERY_STRING'] : '';
 
-        return '/'.ltrim(str_replace('?'.$query, '', $_SERVER['REQUEST_URI']), '/');
+        return '/'.trim(str_replace('?'.$query, '', $_SERVER['REQUEST_URI']), '/');
     }
 
     /**

tests/FullApplicationTest.php

@@ -27,6 +27,46 @@ public function testBasicRequest()
     }
 
 
+    public function testRequestWithoutSymfonyClass()
+    {
+        $app = new Application;
+
+        $app->get('/', function () {
+            return response('Hello World');
+        });
+
+        $_SERVER['REQUEST_METHOD'] = 'GET';
+        $_SERVER['REQUEST_URI'] = '/';
+
+        $response = $app->dispatch();
+
+        $this->assertEquals(200, $response->getStatusCode());
+        $this->assertEquals('Hello World', $response->getContent());
+
+        unset($_SERVER['REQUEST_METHOD'], $_SERVER['REQUEST_URI']);
+    }
+
+
+    public function testRequestWithoutSymfonyClassTrailingSlash()
+    {
+        $app = new Application;
+
+        $app->get('/foo', function () {
+            return response('Hello World');
+        });
+
+        $_SERVER['REQUEST_METHOD'] = 'GET';
+        $_SERVER['REQUEST_URI'] = '/foo/';
+
+        $response = $app->dispatch();
+
+        $this->assertEquals(200, $response->getStatusCode());
+        $this->assertEquals('Hello World', $response->getContent());
+
+        unset($_SERVER['REQUEST_METHOD'], $_SERVER['REQUEST_URI']);
+    }
+
+
     public function testRequestWithParameters()
     {
         $app = new Application;