safeps icon indicating copy to clipboard operation
safeps copied to clipboard

Use execFile(Sync) instead of exec(Sync) in isExecutable(Sync).

Open max-schaefer opened this issue 2 years ago • 3 comments

This is arguably slightly tidier and perhaps safer, since the command isn't run through a shell.

max-schaefer avatar Mar 31 '22 11:03 max-schaefer

Nifty, I never knew of that method.

In your PR there is an incorrect space prefix of the version arg string, which would cause a failure.

I'll be tearing apart safeps in the coming months, into multiple promise and purpose based packages.

Are you actively using safeps?

balupton avatar Nov 24 '23 13:11 balupton

I am not currently using safeps, so feel free to close this.

max-schaefer avatar Nov 27 '23 08:11 max-schaefer

Thanks for the PR update. I'll merge when I get to the pr queue on my current roadmap, such that I can validate the changes in @docpad: https://github.com/orgs/bevry/discussions/1

balupton avatar Nov 29 '23 23:11 balupton