Skip to content

Commit

Permalink
Address nits.
Browse files Browse the repository at this point in the history
Signed-off-by: blake-snyder <blakesnyder@google.com>
  • Loading branch information
blake-snyder committed Aug 7, 2024
1 parent 6e2817c commit 6c06e24
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions source/common/orca/orca_parser.cc
Original file line number Diff line number Diff line change
Expand Up @@ -10,18 +10,24 @@
#include "absl/strings/string_view.h"

using ::Envoy::Http::HeaderMap;
using ::Envoy::Http::LowerCaseString;
using xds::data::orca::v3::OrcaLoadReport;

namespace Envoy {
namespace Orca {

namespace {

const Http::LowerCaseString& endpointLoadMetricsHeaderBin() {
CONSTRUCT_ON_FIRST_USE(Http::LowerCaseString, kEndpointLoadMetricsHeaderBin);
}

} // namespace

absl::StatusOr<OrcaLoadReport> parseOrcaLoadReportHeaders(const HeaderMap& headers) {
OrcaLoadReport load_report;

// Binary protobuf format.
if (const auto header_bin = headers.get(LowerCaseString(kEndpointLoadMetricsHeaderBin));
!header_bin.empty()) {
if (const auto header_bin = headers.get(endpointLoadMetricsHeaderBin()); !header_bin.empty()) {
const auto header_value = header_bin[0]->value().getStringView();
const std::string decoded_value = Envoy::Base64::decode(header_value);
if (!load_report.ParseFromString(decoded_value)) {
Expand Down

0 comments on commit 6c06e24

Please sign in to comment.