fix(web): crash on unsupported derivative build

Fatbuildrweb crashed with HTTP/500 when builds were submitted with a
derivative that is not declared in build pipelines, without clear error
message sent to client. This is fixed by this commit. In this case,
HTTP/404 error is sent with a clear error message that is displayed by
fatbuildrctl.

fix #149

Author: rezib

CHANGELOG.md

@@ -93,6 +93,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   when authenticating with JWT token on unexisting remote HTTP instance (#137).
 - Fix crash with when watching streamed tasks output with HTTP REST API (#138).
 - Avoid buffering with HTTP response headers on reverse proxies (#139).
+- Fix crash of fatbuildrweb on builds with unsupported derivative (#149).
 - docs: Add missing path parameter in REST API to retrieve artifact information.
 
 ## [2.0.0] - 2023-05-05

fatbuildr/protocols/http/server/views.py

@@ -216,7 +216,12 @@ def pipelines_formats(instance):
     filter_derivative = request.args.get('derivative')
 
     if filter_derivative:
-        formats = connection.pipelines_derivative_formats(filter_derivative)
+        # May raise FatbuildrServerError if derivative is not declared in build
+        # pipelines. In this case, forward as a 404.
+        try:
+            formats = connection.pipelines_derivative_formats(filter_derivative)
+        except FatbuildrServerError as err:
+            abort(404, str(err))
     else:
         formats = connection.pipelines_formats()
     for format in formats: