hiSandog opened a new pull request, #11959: URL: https://github.com/apache/seatunnel/pull/11959
### Purpose of this pull request Prevent credentials from appearing in connect dry-run failure messages when sensitive configuration keys are quoted, for example `"password": "secret"` or `'api_key': 'secret'`. The sanitizer now recognizes optional quotes around sensitive keys and keeps masking their values. A regression test covers JSON- and HOCON-style input. ### Does this PR introduce _any_ user-facing change? Yes. Dry-run CLI failure messages now replace values associated with quoted sensitive keys with `***`, matching the existing behavior for unquoted keys. ### How was this patch tested? - `./mvnw spotless:apply` - `./mvnw -pl seatunnel-common -am -Dskip.spotless=true -Dsurefire.failIfNoSpecifiedTests=false -Dtest=DryRunConnectFailureMessageSanitizerTest package` - `./mvnw -q -DskipTests verify` ### Check list * [x] No new Jar binary package is added. * [x] Documentation is not necessary for this masking fix. * [x] This PR introduces no incompatible change. * [x] This PR does not modify connector code. -- 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]
