fix comments

Signed-off-by: Xuyang Tao <[email protected]>
envoyproxy · Jan 18, 2021 · d50f567 · d50f567
1 parent ff21619
commit d50f567
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/docs/root/configuration/observability/access_log/usage.rst b/docs/root/configuration/observability/access_log/usage.rst
@@ -165,7 +165,7 @@ The following command operators are supported:
 
 %REQUEST_HEADERS_BYTES%
   HTTP
-    Request headers bytes.
+    Uncompressed bytes of request headers.
 
   TCP
     Not implemented (0).
@@ -222,14 +222,14 @@ The following command operators are supported:
 
 %RESPONSE_HEADERS_BYTES%
   HTTP
-    Response headers bytes.
+    Uncompressed bytes of response headers.
 
   TCP
     Not implemented (0).
 
 %RESPONSE_TRAILERS_BYTES%
   HTTP
-    Response trailers bytes.
+    Uncompressed bytes of response trailers.
 
   TCP
     Not implemented (0).

diff --git a/source/common/formatter/substitution_formatter.h b/source/common/formatter/substitution_formatter.h
@@ -233,7 +233,7 @@ class HeaderFormatter {
  */
 class HeadersByteSizeFormatter : public FormatterProvider {
 public:
-  // TODO(htuch): Add RequestTrailers here.
+  // TODO(taoxuy): Add RequestTrailers here.
   enum class HeaderType { RequestHeaders, ResponseHeaders, ResponseTrailers };
 
   HeadersByteSizeFormatter(const HeaderType header_type);

diff --git a/test/common/access_log/access_log_impl_test.cc b/test/common/access_log/access_log_impl_test.cc
@@ -159,12 +159,12 @@ name: accesslog
   response_headers_.addCopy("response_header_key", "response_header_val");
   response_trailers_.addCopy("response_trailer_key", "response_trailer_val");
 
-  //    request headers:
-  //    :method: GET
-  //    :path: /
-  //    request_header_key: request_header_val
-
-  //    response headers:
+  // request headers:
+  // :method: GET
+  // :path: /
+  // request_header_key: request_header_val
+  //
+  // response headers:
   // response_header_key: response_header_val
   //
   // response trailers: