Update instrumentation/servlet/servlet-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/servlet/v3_0/snippet/SnippetInjectingResponseWrapper.java

siyuniu-ms · trask · web-flow · commit 85c43e19a6fc · 2023-02-17T18:41:00.000-08:00
Co-authored-by: Trask Stalnaker &lt;trask.stalnaker@gmail.com&gt;
diff --git a/instrumentation/servlet/servlet-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/servlet/v3_0/snippet/SnippetInjectingResponseWrapper.java b/instrumentation/servlet/servlet-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/servlet/v3_0/snippet/SnippetInjectingResponseWrapper.java
@@ -178,7 +178,7 @@ public void updateContentLengthIfPreviouslySet() {
 
   public boolean isNotSafeToInject() {
     // if content-length was set and response was already committed (headers sent to the client),
-    // then not safe to inject because the content-length header cannot be updated to account for
+    // then it is not safe to inject because the content-length header cannot be updated to account for
     // the snippet length
     return isCommitted() && (contentLength != UNSET);
   }

Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,7 @@ public void updateContentLengthIfPreviouslySet() {`
`178`	`178`
`179`	`179`	`public boolean isNotSafeToInject() {`
`180`	`180`	`// if content-length was set and response was already committed (headers sent to the client),`
`181`		`- // then not safe to inject because the content-length header cannot be updated to account for`
	`181`	`+ // then it is not safe to inject because the content-length header cannot be updated to account for`
`182`	`182`	`// the snippet length`
`183`	`183`	`return isCommitted() && (contentLength != UNSET);`
`184`	`184`	`}`