ruanwenjun opened a new issue, #18601:
URL: https://github.com/apache/dolphinscheduler/issues/18601

   ### Search before asking
   
   - [x] I had searched in the issues and found no similar issues.
   
   ### What happened
   
   When testing an Alert Script configuration, the configured script path and 
arguments are passed through an intermediate command string. Valid paths 
containing whitespace or other special characters are therefore not handled 
literally. The test-send operation also does not apply the same 
alert-plugin-instance operation check used when creating an instance, making 
these entry points inconsistent.
   
   ### What you expected to happen
   
   The configured script path and each argument should be passed to the process 
literally. Test sending should reuse the existing alert-plugin-instance create 
operation check.
   
   ### How to reproduce
   
   1. Create an executable .sh script in a path that contains whitespace, for 
example /tmp/alert scripts/send.sh.
   2. Configure Alert Script to use that absolute path.
   3. Click Test Send.
   4. Observe that the script is not invoked from the configured literal path.
   
   ### Anything else
   
   The behavior occurs consistently and is also present on the current dev 
branch.
   
   ### Version
   
   3.4.2
   
   ### Are you willing to submit PR?
   
   - [x] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [x] I agree to follow this projects Code of Conduct.


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

Reply via email to