DanielLeens commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-5551705117
Thanks @Nikk8091. I checked current dev, the existing tracker claims, and the open PR set. There is no active DingTalk validation migration PR. `DingTalkSinkFactory.optionRule()` currently requires `url` and `secret` only for presence, while both options are strings, so blank values can pass factory validation and reach the writer. This is an accepted coordination claim. Please prepare one connector-dingtalk-only PR that expresses the blank-value rule through the existing declarative validation idiom and adds focused valid, empty, and whitespace-only factory-validation coverage. Preserve option names, existing nonblank configurations, and runtime network behavior; do not add remote DingTalk validation, alter the writer, or broaden the umbrella scope. Link the focused PR here when ready. -- 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]
