task: metadata: install: pip install -e .[dev]
patch: diff --git a/docs/changelog.md b/docs/changelog.md
index a88f171..856e2c0 100644
--- a/docs/changelog.md
+++ b/docs/changelog.md
@@ -10,6 +10,12 @@ A changelog:
 
 ## In progress
 
+* **Breaking changes**:
+    * `Request.route` is now always set, but `Request.route.payload` is `None`
+      when path is not found. This allows to catch a not found request in the
+      `request` hook. Note: if the `request` hook does not return a response,
+      a 404 is still automatically raised.
+      ([#45](https://github.com/pyrates/roll/pull/45))
 * Added `request.host` shortcut ([#43](https://github.com/pyrates/roll/pull/43))
 
 
diff --git a/docs/reference.md b/docs/reference.md
index cf743ce..b253f06 100644
--- a/docs/reference.md
+++ b/docs/reference.md
@@ -191,7 +191,8 @@ please refers to that documentation for available patterns.
 A namedtuple to collect matched route data with attributes:
 
 * **payload** (`dict`): the data received by the `@app.route` decorator,
-  contains all handlers plus optionnal custom data.
+  contains all handlers plus optionnal custom data. Value is `None` when request
+  path is not found.
 * **vars** (`dict`): URL placeholders resolved for the current route.
 
 
diff --git a/roll/__init__.py b/roll/__init__.py
index 3ec2b20..1ff018f 100644
--- a/roll/__init__.py
+++ b/roll/__init__.py
@@ -15,7 +15,7 @@ from io import BytesIO
 from typing import TypeVar
 from urllib.parse import parse_qs, unquote
 
-from autoroutes import Routes as BaseRoutes
+from autoroutes import Routes
 from biscuits import Cookie, parse
 from httptools import HttpParserError, HttpRequestParser, parse_url
 from multifruits import Parser, extract_filename, parse_content_disposition
@@ -381,16 +381,6 @@ class Protocol(asyncio.Protocol):
             self.writer.close()
 
 
-class Routes(BaseRoutes):
-    """Customized to raise our own `HttpError` in case of 404."""
-
-    def match(self, url: str):
-        payload, params = super().match(url)
-        if not payload:
-            raise HttpError(HTTPStatus.NOT_FOUND, url)
-        return payload, params
-
-
 Route = namedtuple('Route', ['payload', 'vars'])
 
 
@@ -423,6 +413,8 @@ class Roll:
         try:
             request.route = Route(*self.routes.match(request.path))
             if not await self.hook('request', request, response):
+                if not request.route.payload:
+                    raise HttpError(HTTPStatus.NOT_FOUND, request.path)
                 # Uppercased in order to only consider HTTP verbs.
                 if request.method.upper() not in request.route.payload:
                     raise HttpError(HTTPStatus.METHOD_NOT_ALLOWED)

problem_statement: 404 missing from logger
I think it's from https://github.com/pyrates/roll/commit/7655d0ec3fc4009d6f150d8dda4715388abe6c11

Not sure why we moved the request hook after the dispatching.
Was it to have `request.route` set before?
  
repo: pyrates/roll
test_cmd: pytest --no-header -rA --tb=line --color=no -p no:cacheprovider -W ignore::DeprecationWarning
test_patch: diff --git a/tests/test_hooks.py b/tests/test_hooks.py
index a602f20..81b5172 100644
--- a/tests/test_hooks.py
+++ b/tests/test_hooks.py
@@ -67,3 +67,17 @@ async def test_third_parties_can_call_hook_their_way(client, app):
 
     assert await app.hook('custom', myarg='kwarg') == 'kwarg'
     assert await app.hook('custom', 'arg') == 'arg'
+
+
+async def test_request_hook_is_called_even_if_path_is_not_found(client, app):
+
+    @app.listen('request')
+    async def listener(request, response):
+        if not request.route.payload:
+            response.status = 400
+            response.body = b'Really this is a bad request'
+            return True  # Shortcuts the response process.
+
+    resp = await client.get('/not-found')
+    assert resp.status == HTTPStatus.BAD_REQUEST
+    assert resp.body == b'Really this is a bad request'