Re: RFR: 8303392: Runtime.exec and ProcessBuilder.start should use System logger [v2]

2023-03-04 Thread Roger Riggs
> Runtime.exec and ProcessBuilder.start methods create a new operating system > process with the program and arguments. Many applications configure a logging > subsystem to monitor application events. Logging a process start message with > the program, arguments, and stack trace can identify the

Re: RFR: 8303392: Runtime.exec and ProcessBuilder.start should use System logger [v2]

2023-03-04 Thread Thomas Stuefe
On Sat, 4 Mar 2023 14:22:33 GMT, Roger Riggs wrote: >> Runtime.exec and ProcessBuilder.start methods create a new operating system >> process with the program and arguments. Many applications configure a >> logging subsystem to monitor application events. Logging a process start >> message wit

Re: RFR: 8303392: Runtime.exec and ProcessBuilder.start should use System logger [v2]

2023-03-06 Thread Daniel Fuchs
On Sat, 4 Mar 2023 14:22:33 GMT, Roger Riggs wrote: >> Runtime.exec and ProcessBuilder.start methods create a new operating system >> process with the program and arguments. Many applications configure a >> logging subsystem to monitor application events. Logging a process start >> message wit

Re: RFR: 8303392: Runtime.exec and ProcessBuilder.start should use System logger [v2]

2023-03-06 Thread Roger Riggs
On Mon, 6 Mar 2023 12:44:57 GMT, Daniel Fuchs wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add javadoc @implNote to Runtime.exec and ProcessBuilder methods > > src/java.base/share/classes/java/lang/ProcessBuilder

Re: RFR: 8303392: Runtime.exec and ProcessBuilder.start should use System logger [v2]

2023-03-06 Thread Roger Riggs
On Sun, 5 Mar 2023 06:14:49 GMT, Thomas Stuefe wrote: > I think to be even more useful it could make sense to print the current > directory the child is started in as well as the env var array, possibly only > with a finer logging level. There are concerns about exposing potentially sensitive

Re: RFR: 8303392: Runtime.exec and ProcessBuilder.start should use System logger [v2]

2023-03-09 Thread Roger Riggs
On Sun, 5 Mar 2023 06:14:49 GMT, Thomas Stuefe wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add javadoc @implNote to Runtime.exec and ProcessBuilder methods > > Hi Roger, > > I think to be even more useful it co