hutiefang76 commented on PR #12635: URL: https://github.com/apache/seatunnel/pull/12635#issuecomment-6235437102
Updated in 965d8fce74. The matcher now redacts the complete single-line concatenation, including `""s3cret`, quoted-plus-bare values and reference-plus-literal values. Pure substitutions reuse the same grammar, so padded/quoted paths and multiple quoted reference groups survive save/reload unchanged. Attached comments are preserved. I also replaced the misleading test and documented conservative handling of prose after a bare credential. The full CLI suite passes: 392 tests and 3 subtests; Ruff 0.16.7 and root `./mvnw -q -DskipTests verify` pass. Multiline triple-quoted bodies remain outside the line-bounded matcher. I checked the suggested CI route: the fork's Build workflow is enabled but only accepts push events, with no workflow_dispatch entry. This real code push should supply that event; I will report the actual checks separately, rather than treating the local build as CI. AI assistance: DeepSeek and Codex assisted this follow-up and its validation. -- 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]
