slachiewicz commented on issue #268: URL: https://github.com/apache/maven-shared-utils/issues/268#issuecomment-5631371822
Direction: fix in `CmdShell`, conditional on the `cmd.exe /?` special set, and #286's parentheses case is folded in here. https://github.com/apache/maven-shared-utils/pull/434 implements it: any executable or argument containing whitespace or one of `& < > ( ) @ ^ |` is double-quoted, items already quoted are left alone, and `setQuotedArgumentsEnabled(false)` opts out. Two notes for anyone tracking this from maven-jarsigner or maven-invoker. The `^&` workaround mentioned above will need to be removed once this ships, since `^` is literal inside quotes and the password would then arrive as `^&`. And master currently lacks even the space quoting that 3.4.2 had, which ef89c5a removed; the PR restores that too. *This comment was created with AI assistance.* -- 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]
