finefuture commented on code in PR #14077:
URL: https://github.com/apache/dubbo/pull/14077#discussion_r1604300867
##########
dubbo-rpc/dubbo-rpc-triple/src/main/java/org/apache/dubbo/rpc/protocol/tri/TripleHttp2Protocol.java:
##########
@@ -204,10 +205,9 @@ private Http2FrameCodec buildHttp2FrameCodec(URL url) {
}
private TripleConfig getTripleConfig(URL url) {
- return url.getOrDefaultApplicationModel()
- .getApplicationConfigManager()
- .getProtocol(url.getProtocol())
- .orElseThrow(() -> new IllegalStateException("There's no
ProtocolConfig specified."))
- .getTriple();
+ return
url.getOrDefaultApplicationModel().getApplicationConfigManager().getDefaultProtocols().stream()
Review Comment:
If this application is a pure consumer, no ProtocolConfig will be
configured, but if it needs to call the downstream triple service,
configClientPipeline requires a TripleConfig.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
