yhs0092 commented on code in PR #4875:
URL:
https://github.com/apache/servicecomb-java-chassis/pull/4875#discussion_r2264642082
##########
transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/http/RestClientInvocation.java:
##########
@@ -218,6 +220,11 @@ protected void handleResponse(HttpClientResponse
httpClientResponse) {
return;
}
+ if (restOperationMeta != null && restOperationMeta.isServerSendEvents()) {
+
processFlowableResponseBody(FlowableHelper.toFlowable(httpClientResponse));
Review Comment:
这里可能需要再确认一下, 如果微服务客户端调用服务端的SSE接口方法, 但是服务端报错了, 比如返回 403 状态码的
`application/json` 响应, 那么这里是否就不应该以 SSE 的形式去处理 response body 了呢?
换句话说, 这里的 if 判断条件应该以服务端返回的实际 `Content-Type` 为准会更合适一些?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
