Skip to content

Commit eef6b45

Browse files
authored
fix generic interface metadata publish log error (#6746)
1 parent 039317c commit eef6b45

1 file changed

Lines changed: 11 additions & 10 deletions

File tree

dubbo-registry/dubbo-registry-api/src/main/java/org/apache/dubbo/registry/client/metadata/store/InMemoryWritableMetadataService.java

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -165,17 +165,18 @@ public boolean unsubscribeURL(URL url) {
165165
@Override
166166
public void publishServiceDefinition(URL providerUrl) {
167167
try {
168-
String interfaceName = providerUrl.getParameter(INTERFACE_KEY);
169-
if (StringUtils.isNotEmpty(interfaceName)
170-
&& !ProtocolUtils.isGeneric(providerUrl.getParameter(GENERIC_KEY))) {
171-
Class interfaceClass = Class.forName(interfaceName);
172-
ServiceDefinition serviceDefinition = ServiceDefinitionBuilder.build(interfaceClass);
173-
Gson gson = new Gson();
174-
String data = gson.toJson(serviceDefinition);
175-
serviceDefinitions.put(providerUrl.getServiceKey(), data);
176-
return;
168+
if(!ProtocolUtils.isGeneric(providerUrl.getParameter(GENERIC_KEY))){
169+
String interfaceName = providerUrl.getParameter(INTERFACE_KEY);
170+
if (StringUtils.isNotEmpty(interfaceName)) {
171+
Class interfaceClass = Class.forName(interfaceName);
172+
ServiceDefinition serviceDefinition = ServiceDefinitionBuilder.build(interfaceClass);
173+
Gson gson = new Gson();
174+
String data = gson.toJson(serviceDefinition);
175+
serviceDefinitions.put(providerUrl.getServiceKey(), data);
176+
return;
177+
}
178+
logger.error("publishProvider interfaceName is empty . providerUrl: " + providerUrl.toFullString());
177179
}
178-
logger.error("publishProvider interfaceName is empty . providerUrl: " + providerUrl.toFullString());
179180
} catch (ClassNotFoundException e) {
180181
//ignore error
181182
logger.error("publishProvider getServiceDescriptor error. providerUrl: " + providerUrl.toFullString(), e);

0 commit comments

Comments
 (0)