fix: handle special characters in postinstall script
#2649
+2
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Improve handling of shell metacharacters in folder names in the
postinstallscript by replacingexecSyncwithexecFileSync.For example, spaces in folder names are now allowed:
Current behaviour
Installing
mswwith amsw.workerDirectoryset in yourpackage.jsonfile fails if the folder structure has shell special characters (e.g. spaces) in it.Proposed
Installing
mswwith amsw.workerDirectoryset in yourpackage.jsonfile works, even if your folder structure has spaces in it.Future work
The
config/scripts/patch-ts.jsscript is still vulnerable to this issue too, but since that only affects developers of this library, not users, I've ignored it for this PR:msw/config/scripts/patch-ts.js
Lines 91 to 96 in e0a4643