Skip to content

Commit cf383fb

Browse files
committed
Import of Cloud Functions JVM from Git-on-Borg.
- 8a7701f8965dda3907dff901ad44584f1def4fde Update converter_test to use the new API. by Éamonn McManus <[email protected]> - 8033fd268a1c772ff99660de6a44a8af0f3c335e Add a comment to HttpResponseImpl.getWriter(). by Éamonn McManus <[email protected]> PiperOrigin-RevId: 288965493
1 parent e489f4d commit cf383fb

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

invoker-core/core/src/main/java/com/google/cloud/functions/invoker/http/HttpResponseImpl.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,10 @@ public synchronized BufferedWriter getWriter() throws IOException {
7373
// PrintWriter.out field via reflection, but that sort of access to non-public fields of
7474
// platform classes is now frowned on and may draw warnings or even fail in subsequent
7575
// versions.
76+
// We could instead wrap the OutputStream, but that would require us to deduce the appropriate
77+
// Charset, using logic like this:
78+
// https://github.com/eclipse/jetty.project/blob/923ec38adf/jetty-server/src/main/java/org/eclipse/jetty/server/Response.java#L731
79+
// We may end up doing that if performance is an issue.
7680
writer = new BufferedWriter(response.getWriter());
7781
}
7882
return writer;

0 commit comments

Comments
 (0)