Commit 8315c98
committed
entrypoint-aws-batch: Keep ../ path parts in ZIP archive members during extraction
The default of stripping ../ parts in member paths is a (good!)
restriction for safety and security, but such paths do not pose any
(additional) risk in the context of our Nextstrain runtime containers.
We're already downloading and executing arbitrary user-supplied code, so
the ability to overwrite system files with ZIP archive members is not
any additional privilege.
Keeping the ../ parts will allow Nextstrain CLI to construct ZIP archives
for jobs which write to sibling paths of /nextstrain/build in the
container. This will be used for including pathogen workflow source
separate (e.g. in /nextstrain/pathogen) from the analysis working
directory (/nextstrain/build). As a side-effect, this will also allow
Nextstrain CLI's --augur, --auspice, etc. overlays to start working with
AWS Batch when previously they did not.
Note that Nextstrain CLI does *not* permit ../ path parts when
extracting from these same ZIP archives (e.g. after a job completes to
download results), as that *would* be additional risk. Currently it
strips ../ parts, like unzip's default behaviour, but that will change
soon to entirely skip archive members containing ../ parts.1 parent 3ea8f75 commit 8315c98
1 file changed
+1
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | | - | |
| 11 | + | |
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| |||
0 commit comments