fix reader not skipping over all CRLF

Author: thatstoasty

lightbug_http/http/request.mojo

@@ -56,10 +56,13 @@ struct HTTPRequest(Writable, Stringable):
         var request = HTTPRequest(
             URI.parse(addr + uri), headers=headers, method=method, protocol=protocol, cookies=cookies
         )
-        try:
-            request.read_body(reader, content_length, max_body_size)
-        except e:
-            raise Error("HTTPRequest.from_bytes: Failed to read request body: " + str(e))
+
+        if content_length > 0:
+            try:
+                reader.skip_carriage_return()
+                request.read_body(reader, content_length, max_body_size)
+            except e:
+                raise Error("HTTPRequest.from_bytes: Failed to read request body: " + str(e))
 
         return request
 

lightbug_http/io/bytes.mojo

@@ -205,6 +205,7 @@ struct ByteReader[origin: Origin]:
             raise OutOfBoundsError
 
         self.read_pos += count
+        print(start, start + count)
         return self._inner[start : start + count]
 
     fn read_until(mut self, char: Byte) -> ByteView[origin]:

lightbug_http/service.mojo

@@ -2,6 +2,7 @@ from lightbug_http.http import HTTPRequest, HTTPResponse, OK, NotFound
 from lightbug_http.io.bytes import Bytes, bytes
 from lightbug_http.strings import to_string
 from lightbug_http.header import HeaderKey
+from utils import StringSlice
 
 
 trait HTTPService:

tests/lightbug_http/http/test_request.mojo

@@ -18,7 +18,15 @@ def test_request_from_bytes():
 
 
 def test_read_body():
-    ...
+    alias data = "GET /redirect HTTP/1.1\r\nHost: 127.0.0.1:8080\r\nUser-Agent: python-requests/2.32.3\r\nAccept-Encoding: gzip, deflate, br, zstd\r\nAccept: */\r\nContent-Length: 17\r\nconnection: keep-alive\r\n\r\nThis is the body!"
+    var request = HTTPRequest.from_bytes("127.0.0.1", 4096, data.as_bytes())
+    testing.assert_equal(request.protocol, "HTTP/1.1")
+    testing.assert_equal(request.method, "GET")
+    testing.assert_equal(request.uri.request_uri, "/redirect")
+    testing.assert_equal(request.headers["Host"], "127.0.0.1:8080")
+    testing.assert_equal(request.headers["User-Agent"], "python-requests/2.32.3")
+
+    testing.assert_equal(request.get_body(), "This is the body!")
 
 
 def test_encode():