Redirect automatically on trailing slash

- Only when route is not found
- Redirects with 302 Moved Permanently status

Author: noteflakes

lib/syntropy/app.rb

@@ -64,8 +64,14 @@ def initialize(**env)
     # @param req [Qeweney::Request] HTTP request
     # @return [void]
     def call(req)
-      route = @router_proc.(req.path, req.route_params)
-      raise Syntropy::Error.not_found('Not found') if !route
+      path = req.path
+      route = @router_proc.(path, req.route_params)
+      if !route
+        if (m = path.match(/^(.+)\/$/))
+          return req.redirect(m[1], Qeweney::Status::MOVED_PERMANENTLY)
+        end
+        raise Syntropy::Error.not_found('Not found')
+      end
 
       req.route = route
       proc = route[:proc] ||= compute_route_proc(route)
@@ -74,7 +80,7 @@ def call(req)
       @logger&.error(
         message: "Error while serving request: #{e.message}",
         method: req.method,
-        path: req.path,
+        path: path,
         error: e
       ) if Error.log_error?(e)
       error_handler = get_error_handler(route)

test/test_app.rb

@@ -122,6 +122,12 @@ def test_app_rendering
     assert_equal Status::INTERNAL_SERVER_ERROR, req.response_status
   end
 
+  def test_automatic_redirect_on_trailing_slash    
+    req = make_request(':method' => 'GET', ':path' => '/test/rss/')
+    assert_equal Status::MOVED_PERMANENTLY, req.response_status
+    assert_equal '/test/rss', req.response_headers['Location']
+  end
+
   def test_app_file_watching
     @machine.sleep 0.3