tree b3c1ee58ef584d47e00c78ab6440979937ce6d5c
parent afb9ee06281deea4944016396736a9bc35865094
author Ed Tanous <edtanous@google.com> 1671495129 -0800
committer Ed Tanous <ed@tanous.net> 1671658157 +0000

Use owning strings for aggregator URI

Clang correctly notes that, because the segments() object is an rvalue,
and is destroyed after the std::string_view is used on the next line,
this is technically undefined behavior.  Make these use owning
std::strings instead of std::string_view to avoid the lifetime problem.

Tested:
Enabled aggregation without including a satellite config.  Sending a
GET request to /redfish/v1/Chassis/5B247A_Test returned a 404 due to
hitting one of the sections of changed code.  The other section is not
currently reachable, but uses the same type of change.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I1b2b2ad444e2718c0cb167f4506ea1d0915b3a5b
Signed-off-by: Carson Labrado <clabrado@google.com>