Merge branch 'bug/windows-path-java-override' into 'dev'

Fix window path escape on java override

See merge request crafty-controller/crafty-4!513
This commit is contained in:
Iain Powrie 2022-12-21 16:09:14 +00:00
commit 46c8c04490
2 changed files with 5 additions and 1 deletions

View File

@ -11,6 +11,7 @@ TBD
- Remove Pathlib from sub path check ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/507))
- Fix root dir selection in Upload Zip Import ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/508))
- Fix stats error on mac M1 chips ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/512))
- Fix window path escape on java override ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/513))
### Tweaks
- Make server directories non-configurable ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/511))
### Lang

View File

@ -1560,7 +1560,10 @@ class PanelHandler(BaseHandler):
return
if java_selection:
try:
execution_list = shlex.split(execution_command)
if self.helper.is_os_windows():
execution_list = shlex.split(execution_command, posix=False)
else:
execution_list = shlex.split(execution_command, posix=True)
except ValueError:
self.redirect(
"/panel/error?error=Invalid execution command. Java path"