Tectu / malloy

A cross-platform C++20 library providing embeddable server & client components for HTTP and WebSocket.
BSD 3-Clause "New" or "Revised" License
66 stars 8 forks source link

fix: endpoint tests build #98

Closed 0x00002a closed 2 years ago

0x00002a commented 2 years ago

fixes: #93

Turns out beast::http::request<...> does actually have a constructor taking just the header. I am unsure as to why this wasn't being used but I'm guessing that's part of the reason as to why the code just stopped compiling. In any case, this fix just explicitly calls that constructor before passing it on to the malloy wrapper.

Also I removed the return since its kind of misleading (as the function returns void)

Tectu commented 2 years ago

Awesome, thanks!