zhuyijun/nacos
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[ISSUE #11968] Buf fix: Use put() instead of putIfAbsent() for setArg() in ConfigChangeRequest (#11971)

Browse Source 
* Fixed an issue where the maximum number of anti-fragile plug-ins implemented by default in Nacos is one more than the actual number of connections.

* bug fix : 修改删除空服务实例时，服务名和分组名没有正确解析的问题

* Update ConfigChangeAspect.java

修改configChangeRequest.setArg()的key为serviceType

* Update ConfigChangeAspect.java

* Update ConfigChangeRequest.java
This commit is contained in:
Happy-26 2024-04-22 10:53:11 +08:00 committed by GitHub
parent 2bb7193d1b
commit 96814ff00d
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
config/src/main/java/com/alibaba/nacos/config/server/aspect/ConfigChangeAspect.java
View File

@ -311,7 +311,7 @@ public class ConfigChangeAspect {
for (ConfigChangePluginService ccs : beforeExecutePluginServices) {
final String serviceType = ccs.getServiceType().toLowerCase(Locale.ROOT);
final Properties properties = configChangeConfigs.getPluginProperties(serviceType);
configChangeRequest.setArg("pluginProperties", properties);
configChangeRequest.setArg(ConfigChangeConstants.PLUGIN_PROPERTIES, properties);
ccs.execute(configChangeRequest, configChangeResponse);
if (null != configChangeResponse.getArgs()) {
// update args by filter with whitelist

2
plugin/config/src/main/java/com/alibaba/nacos/plugin/config/model/ConfigChangeRequest.java
View File

@ -40,7 +40,7 @@ public class ConfigChangeRequest {
}
public void setArg(String key, Object value) {
requestArgs.putIfAbsent(key, value);
requestArgs.put(key, value);
}
public Object getArg(String key) {