Implement HTTP protocol #469

swlynch99 · 2022-10-03T19:25:16Z

This PR introduces a rather simple HTTP protocol. It supports 3 commands - GET, PUT (set), and DELETE which is enough for simple cache use-cases. I have included tests for both request parsing and response emission.

In detail, this PR includes:

A HTTP parser that wraps httparse
A very simple HTTP response builder that does no validation
Tests for the above

Due to limitations around what pelikan allows parsers to do (they can't emit responses) the response parser doesn't directly return a request object but instead returns a wrapper around a Result<Request, Response>. The error response should be sent by the client in the place of creating its own response.

This PR is the first part of breaking apart #468 into mergeable chunks.

src/protocol/http/Cargo.toml

src/protocol/http/src/lib.rs

src/protocol/http/src/request.rs

src/protocol/http/tests/request.rs

src/protocol/http/tests/response.rs

brayniac

LGTM

swlynch99 added 12 commits October 3, 2022 11:28

Write HTTP protocol

6611cf4

Fix license headers

07f1f69

Fix bad assert

889934e

Remove debug logging from util.rs

278fb53

Implement Default for RequestParser

505fb48

Add debug implementations for request types

51a6980

Add request parsing tests

d11c67f

formatting

ee170e3

Update Cargo.toml to use workspace dependencies

3e829e6

Add response tests

c54745b

Add support for klog with http requests

cb2f5ab

Refactor klog impls somewhat

cfaa3a1

swlynch99 requested a review from brayniac October 3, 2022 19:25