hutiefang76 commented on PR #12443: URL: https://github.com/apache/seatunnel/pull/12443#issuecomment-5987902130
Thanks @SEZ9. > Could you confirm the current changed-file list Confirmed against unchanged head `971fb98587b1f5e673fd65912a8e66cc53b69f11`: 11 files, comprising four production files, five tests and two docs. Production (API and core-starter): ```text seatunnel-api/src/main/java/org/apache/seatunnel/api/configuration/ReadonlyConfig.java seatunnel-api/src/main/java/org/apache/seatunnel/api/configuration/util/ConfigMapPathUtils.java seatunnel-core/seatunnel-core-starter/src/main/java/org/apache/seatunnel/core/starter/utils/ConfigBuilder.java seatunnel-core/seatunnel-core-starter/src/main/java/org/apache/seatunnel/core/starter/utils/ConfigShadeUtils.java ``` Tests: ```text seatunnel-api/src/test/java/org/apache/seatunnel/api/configuration/ReadableConfigTest.java seatunnel-core/seatunnel-core-starter/src/test/java/org/apache/seatunnel/core/starter/utils/ConfigBuilderTest.java seatunnel-e2e/seatunnel-engine-e2e/connector-seatunnel-e2e-base/src/test/java/org/apache/seatunnel/engine/e2e/RestApiIT.java seatunnel-engine/seatunnel-engine-server/src/test/java/org/apache/seatunnel/engine/server/rest/RestApiSubmitJobConfigShadeDecryptTest.java seatunnel-engine/seatunnel-engine-server/src/test/java/org/apache/seatunnel/engine/server/utils/RestUtilTest.java ``` Docs: ```text docs/en/engines/zeta/rest-api-v2.md docs/zh/engines/zeta/rest-api-v2.md ``` There is no production `RestUtil.java` change in the current diff. Engine-server changes are tests; the guard lives in the shared API/core-starter conversion code, so the title remains `[Fix][Core]`. > Please also state explicitly in the PR description which test covers the decrypt-path issue (F1) Added a “Current changed modules and decrypt regression” section to the description. It names `ConfigBuilderTest#testInvalidPathKeysSurviveConfigShadeRoundTrip` ([source](https://github.com/apache/seatunnel/blob/971fb98587b1f5e673fd65912a8e66cc53b69f11/seatunnel-core/seatunnel-core-starter/src/test/java/org/apache/seatunnel/core/starter/utils/ConfigBuilderTest.java#L78-L103)): `ConfigShadeUtils.decryptConfig(ConfigBuilder.of(configMap))` followed by `Assertions.assertEquals(fields, actualFields)`, with the regex and reserved/escaped keys in the complete fields map. The existing normalization boundary, EN/ZH docs and MetadataProviderManager/AssertSink exclusions remain explicit. > please trigger a fresh remote CI run on the current head Re-ran **all jobs** of the fork Build workflow at that exact head: [run 36778785525](https://github.com/hutiefang76/seatunnel/actions/runs/36778785525), attempt **3**, triggered Oct 5 at **04:02 UTC**. The earlier attempt 2 succeeded; attempt 3 is newly started and has no final result yet. No code or head commit changed in this pass. -- 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]
