Zjianru opened a new pull request, #16082:
URL: https://github.com/apache/dubbo/pull/16082

   ## Description
   This PR removes the unnecessary pattern check for `password` in 
`ConfigValidationUtils.checkParameterName`.
   
   **Problem:**
   Previously, `checkParameterName` validated all parameters (except `backup`) 
using `PATTERN_NAME_HAS_SYMBOL`, which disallowed many special characters 
commonly used in passwords (e.g., `@`, `!`, `#`). This caused issues when using 
Nacos with complex passwords configured via parameters.
   
   **Fix:**
   Added `PASSWORD_KEY` to the `ignoreCheckKeys` list in `checkParameterName`, 
ensuring passwords are exempted from pattern validation (length check in 
`validateRegistryConfig` remains).
   
   **Fixes:** #16073
   
   ## 描述 (Chinese)
   修复 `ConfigValidationUtils` 中对 `password` 参数进行不必要的正则校验问题。
   
   **问题:**
   此前 `checkParameterName` 会对所有参数(除 `backup` 外)使用 `PATTERN_NAME_HAS_SYMBOL` 
进行校验,该正则不支持密码中常见的特殊字符(如 `@`、`!`、`#`),导致 Nacos 等注册中心使用复杂密码时校验失败。
   
   **修复:**
   将 `PASSWORD_KEY` 加入 `checkParameterName` 的忽略列表,跳过正则检查(长度检查仍保留)。
   
   **修复:** #16073


-- 
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]

Reply via email to