Skip to content

Commit

Permalink
non optional
Browse files Browse the repository at this point in the history
Signed-off-by: Shriram Rajagopalan <shriramr@google.com>
  • Loading branch information
Shriram Rajagopalan committed May 23, 2023
1 parent 241c681 commit 9ef31df
Showing 1 changed file with 11 additions and 8 deletions.
19 changes: 11 additions & 8 deletions source/extensions/filters/http/ext_proc/ext_proc.h
Original file line number Diff line number Diff line change
Expand Up @@ -70,18 +70,20 @@ class ExtProcLoggingInfo : public Envoy::StreamInfo::FilterState::Object {
void setBytesSent(uint64_t bytes_sent) { bytes_sent_ = bytes_sent; }
void setBytesReceived(uint64_t bytes_received) { bytes_received_ = bytes_received; }
void setClusterInfo(absl::optional<Upstream::ClusterInfoConstSharedPtr> cluster_info) {
cluster_info_ = cluster_info;
if (cluster_info) {
cluster_info_ = cluster_info.value();
}
}
void setUpstreamHost(absl::optional<Upstream::HostDescriptionConstSharedPtr> upstream_host) {
upstream_host_ = upstream_host;
if (upstream_host) {
upstream_host_ = upstream_host.value();
}
}

uint64_t bytesSent() const { return bytes_sent_; }
uint64_t bytesReceived() const { return bytes_received_; }
absl::optional<Upstream::ClusterInfoConstSharedPtr> clusterInfo() const { return cluster_info_; }
absl::optional<Upstream::HostDescriptionConstSharedPtr> upstreamHost() const {
return upstream_host_;
}
Upstream::ClusterInfoConstSharedPtr clusterInfo() const { return cluster_info_; }
Upstream::HostDescriptionConstSharedPtr upstreamHost() const { return upstream_host_; }
const GrpcCalls& grpcCalls(envoy::config::core::v3::TrafficDirection traffic_direction) const;
const Envoy::ProtobufWkt::Struct& filterMetadata() const { return filter_metadata_; }

Expand All @@ -91,9 +93,10 @@ class ExtProcLoggingInfo : public Envoy::StreamInfo::FilterState::Object {
GrpcCalls encoding_processor_grpc_calls_;
const Envoy::ProtobufWkt::Struct filter_metadata_;
// The following stats are populated for ext_proc filters using Envoy gRPC only.
// The bytes sent and received are for the entire stream.
uint64_t bytes_sent_{0}, bytes_received_{0};
absl::optional<Upstream::ClusterInfoConstSharedPtr> cluster_info_;
absl::optional<Upstream::HostDescriptionConstSharedPtr> upstream_host_;
Upstream::ClusterInfoConstSharedPtr cluster_info_;
Upstream::HostDescriptionConstSharedPtr upstream_host_;
};

class FilterConfig {
Expand Down

0 comments on commit 9ef31df

Please sign in to comment.