Replaced CommonHTTPStatus with direct values

Author: michalpokusa

adafruit_httpserver/response.py

@@ -25,7 +25,7 @@
 )
 from .mime_type import MIMEType
 from .request import Request
-from .status import Status, CommonHTTPStatus
+from .status import Status, OK_200
 from .headers import Headers
 
 
@@ -103,7 +103,7 @@ def route_func(request):
     def __init__(  # pylint: disable=too-many-arguments
         self,
         request: Request,
-        status: Union[Status, Tuple[int, str]] = CommonHTTPStatus.OK_200,
+        status: Union[Status, Tuple[int, str]] = OK_200,
         headers: Union[Headers, Dict[str, str]] = None,
         content_type: str = None,
         http_version: str = "HTTP/1.1",

adafruit_httpserver/server.py

@@ -21,7 +21,7 @@
 from .request import Request
 from .response import Response
 from .route import _Routes, _Route
-from .status import CommonHTTPStatus
+from .status import BAD_REQUEST_400, UNAUTHORIZED_401, FORBIDDEN_403, NOT_FOUND_404
 
 
 class Server:
@@ -177,26 +177,20 @@ def poll(self):
                             head_only=(request.method == HEAD),
                         )
                     else:
-                        HTTPResponse(
-                            request, status=CommonHTTPStatus.BAD_REQUEST_400
-                        ).send()
+                        Response(request, status=BAD_REQUEST_400).send()
 
                 except AuthenticationError:
                     Response(
                         request,
-                        status=CommonHTTPStatus.UNAUTHORIZED_401,
+                        status=UNAUTHORIZED_401,
                         headers={"WWW-Authenticate": 'Basic charset="UTF-8"'},
                     ).send()
 
                 except InvalidPathError as error:
-                    Response(request, status=CommonHTTPStatus.FORBIDDEN_403).send(
-                        str(error)
-                    )
+                    Response(request, status=FORBIDDEN_403).send(str(error))
 
                 except FileNotExistsError as error:
-                    Response(request, status=CommonHTTPStatus.NOT_FOUND_404).send(
-                        str(error)
-                    )
+                    Response(request, status=NOT_FOUND_404).send(str(error))
 
         except OSError as error:
             # Handle EAGAIN and ECONNRESET

adafruit_httpserver/status.py

@@ -31,23 +31,38 @@ def __eq__(self, other: "Status"):
         return self.code == other.code and self.text == other.text
 
 
-class CommonHTTPStatus(HTTPStatus):  # pylint: disable=too-few-public-methods
-    """Common HTTP status codes."""
+OK_200 = Status(200, "OK")
+"""200 OK"""
 
-    OK_200 = HTTPStatus(200, "OK")
-    """200 OK"""
+NO_CONTENT_204 = Status(204, "No Content")
+"""204 No Content"""
 
-    BAD_REQUEST_400 = HTTPStatus(400, "Bad Request")
-    """400 Bad Request"""
+TEMPORARY_REDIRECT_307 = Status(307, "Temporary Redirect")
+"""307 Temporary Redirect"""
 
-    UNAUTHORIZED_401 = HTTPStatus(401, "Unauthorized")
-    """401 Unauthorized"""
+PERMANENT_REDIRECT_308 = Status(308, "Permanent Redirect")
+"""308 Permanent Redirect"""
 
-    FORBIDDEN_403 = HTTPStatus(403, "Forbidden")
-    """403 Forbidden"""
+BAD_REQUEST_400 = Status(400, "Bad Request")
+"""400 Bad Request"""
 
-    NOT_FOUND_404 = HTTPStatus(404, "Not Found")
-    """404 Not Found"""
+UNAUTHORIZED_401 = Status(401, "Unauthorized")
+"""401 Unauthorized"""
 
-    INTERNAL_SERVER_ERROR_500 = HTTPStatus(500, "Internal Server Error")
-    """500 Internal Server Error"""
+FORBIDDEN_403 = Status(403, "Forbidden")
+"""403 Forbidden"""
+
+NOT_FOUND_404 = Status(404, "Not Found")
+"""404 Not Found"""
+
+METHOD_NOT_ALLOWED_405 = Status(405, "Method Not Allowed")
+"""405 Method Not Allowed"""
+
+INTERNAL_SERVER_ERROR_500 = Status(500, "Internal Server Error")
+"""500 Internal Server Error"""
+
+NOT_IMPLEMENTED_501 = Status(501, "Not Implemented")
+"""501 Not Implemented"""
+
+SERVICE_UNAVAILABLE_503 = Status(503, "Service Unavailable")
+"""503 Service Unavailable"""