[dubbo] branch master updated: throw exception on path+version not found when decoding request (#8357)

[ho...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

horizonzy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/dubbo.git


The following commit(s) were added to refs/heads/master by this push:
     new ca794b6  throw exception on path+version not found when decoding request (#8357)
ca794b6 is described below

commit ca794b627cd7ce9634ce215d39bc757d49e5810f
Author: ken.lj <ke...@gmail.com>
AuthorDate: Tue Jul 27 20:21:21 2021 +0800

    throw exception on path+version not found when decoding request (#8357)
---
 .../main/java/org/apache/dubbo/remoting/transport/CodecSupport.java  | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/dubbo-remoting/dubbo-remoting-api/src/main/java/org/apache/dubbo/remoting/transport/CodecSupport.java b/dubbo-remoting/dubbo-remoting-api/src/main/java/org/apache/dubbo/remoting/transport/CodecSupport.java
index d4beb50..ba1eaee 100644
--- a/dubbo-remoting/dubbo-remoting-api/src/main/java/org/apache/dubbo/remoting/transport/CodecSupport.java
+++ b/dubbo-remoting/dubbo-remoting-api/src/main/java/org/apache/dubbo/remoting/transport/CodecSupport.java
@@ -162,10 +162,7 @@ public class CodecSupport {
         ServiceRepository repository = ApplicationModel.getServiceRepository();
         ProviderModel providerModel = repository.lookupExportedServiceWithoutGroup(path + ":" + version);
         if (providerModel == null) {
-            if (logger.isWarnEnabled()) {
-                logger.warn("Serialization security check is enabled but cannot work as expected because " +
-                        "there's no matched provider model for path " + path + ", version " + version);
-            }
+            throw new IOException("Service " + path + " with version " + version + " not found, invocation rejected.");
         } else {
             List<URL> urls = providerModel.getServiceConfig().getExportedUrls();
             if (CollectionUtils.isNotEmpty(urls)) {