OPSnet / Logchecker

Logchecker for parsing and scoring logs coming from CD ripping programs
The Unlicense
54 stars 10 forks source link

Use symfony/process instead of shell_exec #10

Closed datagutten closed 4 years ago

datagutten commented 4 years ago

The shell_exec calls reported that the log file was not found even if a PHP file_exists said that it was there. I guess there is something with the escaping of the shell argument, so I added symfony/process to project and used that instead of shell_exec and escapeshellarg. My experience with symfony/process is that the escaping works very well and it fixed the problem here.

itismadness commented 4 years ago

Merged in 7e119aa8e3539167a9b81952d9f1f9cdb89a4d8a. Thanks!