tree 29ec78e960b9a8af5015b72f67941b4402dbfe88
parent ad22fefecaf7988fd7072dc71042efbf86fc5162
author Ed Tanous <edtanous@google.com> 1631826800 -0700
committer Ed Tanous <ed@tanous.net> 1631914665 +0000

Make fewer copies

There are apparently places where we make complete copies of the request
structure, including the body data within it.  Recently,
59b98b2222fddbea3d6f678d9e94006521f0c381 came along and made the size of
the Request structure include the body payload.  This is good for object
ownership, but bad for code that wants to make a copy.

This commit tries to do something less insane, and construct the Payload
object (which only really includes the http headers, uri and method)
before jumping into callback hell, thus meaning that the Payload object
is copied 4 times instead of the Request object.

Tested:
Code compiles.  Deleted the boost::beast::http::message copy
constructor, and observed that new code now compiles without needing a
copy.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ie81bade61fb1f6f392b163f98c84540e09a26690