owners: Include suniltheta under AWS files (envoyproxy#30728)

Update few TODO comments
Plus removed suniltheta from zipkin extension

Signed-off-by: Sunil Narasimhamurthy <[email protected]>

CODEOWNERS

@@ -84,8 +84,8 @@ extensions/filters/common/original_src @klarose @mattklein123
 /*/extensions/filters/http/cache @toddmgreer @jmarantz @penguingao @mpwarres @capoferro
 /*/extensions/http/cache/simple_http_cache @toddmgreer @jmarantz @penguingao @mpwarres @capoferro
 # aws_iam grpc credentials
-/*/extensions/grpc_credentials/aws_iam @lavignes @mattklein123
-/*/extensions/common/aws @lavignes @mattklein123
+/*/extensions/grpc_credentials/aws_iam @suniltheta @lavignes @mattklein123
+/*/extensions/common/aws @suniltheta @lavignes @mattklein123
 # adaptive concurrency limit extension.
 /*/extensions/filters/http/adaptive_concurrency @tonya11en @mattklein123
 # admission control extension.
@@ -147,8 +147,8 @@ extensions/filters/common/original_src @klarose @mattklein123
 # support for on-demand VHDS requests
 /*/extensions/filters/http/on_demand @dmitri-d @htuch @kyessenov
 /*/extensions/filters/network/connection_limit @mattklein123 @alyssawilk @delong-coder
-/*/extensions/filters/http/aws_request_signing @derekargueta @mattklein123 @marcomagdy
-/*/extensions/filters/http/aws_lambda @mattklein123 @marcomagdy @lavignes
+/*/extensions/filters/http/aws_request_signing @derekargueta @suniltheta @mattklein123 @marcomagdy
+/*/extensions/filters/http/aws_lambda @suniltheta @mattklein123 @marcomagdy @lavignes
 /*/extensions/filters/http/buffer @alyssawilk @mattklein123
 /*/extensions/transport_sockets/raw_buffer @alyssawilk @mattklein123
 # Watchdog Extensions
@@ -264,7 +264,7 @@ extensions/filters/http/oauth2 @derekargueta @mattklein123
 /*/extensions/http/stateful_session/header @ramaraochavali @wbpcode @cpakulski
 /*/extensions/filters/http/stateful_session @wbpcode @cpakulski
 # tracers
-/*/extensions/tracers/zipkin @wbpcode @Shikugawa @suniltheta @basvanbeek
+/*/extensions/tracers/zipkin @wbpcode @Shikugawa @basvanbeek
 /*/extensions/tracers/dynamic_ot @wbpcode @Shikugawa @basvanbeek
 /*/extensions/tracers/opencensus @wbpcode @Shikugawa @basvanbeek
 /*/extensions/tracers/common @wbpcode @Shikugawa @basvanbeek

source/extensions/common/aws/utility.cc

@@ -64,7 +64,7 @@ Utility::canonicalizeHeaders(const Http::RequestHeaderMap& headers,
       });
   // The AWS SDK has a quirk where it removes "default ports" (80, 443) from the host headers
   // Additionally, we canonicalize the :authority header as "host"
-  // TODO(lavignes): This may need to be tweaked to canonicalize :authority for HTTP/2 requests
+  // TODO(suniltheta): This may need to be tweaked to canonicalize :authority for HTTP/2 requests
   const absl::string_view authority_header = headers.getHostValue();
   if (!authority_header.empty()) {
     const auto parts = StringUtil::splitToken(authority_header, ":");

source/extensions/tracers/xray/daemon_broker.cc

@@ -43,7 +43,7 @@ void DaemonBrokerImpl::send(const std::string& data) const {
                                                   nullptr /*local_ip*/, *address_);
 
   if (rc.return_value_ != payload.length()) {
-    // TODO(marcomagdy): report this in stats
+    // TODO(suniltheta): report this in stats
     ENVOY_LOG_TO_LOGGER(logger, debug, "Failed to send trace payload to the X-Ray daemon.");
   }
 }

source/extensions/tracers/xray/xray_tracer_impl.cc

@@ -75,7 +75,7 @@ Tracing::SpanPtr Driver::startSpan(const Tracing::Config& config,
   // If we have a XRay TraceID in the headers, then we create a SpanContext to pass that trace-id
   // around if no TraceID (which means no x-ray header) then this is a brand new span.
 
-  // TODO(marcomagdy) - how do we factor this into the logic above
+  // TODO(suniltheta) - how do we factor this into the logic above
   UNREFERENCED_PARAMETER(tracing_decision);
   const auto header = trace_context.getByKey(XRayTraceHeader);
   absl::optional<bool> should_trace;