diff --git a/CHANGELOG.md b/CHANGELOG.md index 24bb7ed..91bc7e4 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,9 +2,10 @@ Note: Breaking changes between versions are indicated by "💥". -## unreleased +## Unreleased -- [Bugfix] Fix 502 error on request to lms with header larger than the maximum uwsgi buffer size +- [Bugfix] Fix crash in `local quickstart -p` command. +- [Bugfix] Fix 502 error on request to lms with header larger than the maximum uwsgi buffer size. ## v11.2.8 (2021-04-27) diff --git a/tests/test_bindmounts.py b/tests/test_bindmounts.py index 35fe1cb..dbc4b4b 100644 --- a/tests/test_bindmounts.py +++ b/tests/test_bindmounts.py @@ -29,3 +29,8 @@ class BindMountsTests(unittest.TestCase): ) self.assertEqual(("/openedx/venv", "/tmp/openedx:/openedx"), volume_args) self.assertEqual(("run", "lms", "echo", "boom"), non_volume_args) + + def test_parse_volumes_empty_list(self) -> None: + volume_args, non_volume_args = bindmounts.parse_volumes([]) + self.assertEqual((), volume_args) + self.assertEqual((), non_volume_args) diff --git a/tutor/bindmounts.py b/tutor/bindmounts.py index d733283..2324d61 100644 --- a/tutor/bindmounts.py +++ b/tutor/bindmounts.py @@ -75,7 +75,7 @@ def parse_volumes(docker_compose_args: List[str]) -> Tuple[List[str], List[str]] @click.command(context_settings={"ignore_unknown_options": True}) @click.option("-v", "--volume", "volumes", multiple=True) - @click.argument("args", nargs=-1, required=True) + @click.argument("args", nargs=-1) def custom_docker_compose( volumes: List[str], args: List[str] ) -> None: # pylint: disable=unused-argument diff --git a/tutor/commands/compose.py b/tutor/commands/compose.py index 6860802..782420d 100644 --- a/tutor/commands/compose.py +++ b/tutor/commands/compose.py @@ -276,7 +276,7 @@ def logs(context: click.Context, follow: bool, tail: bool, service: str) -> None name="dc", ) @click.argument("command") -@click.argument("args", nargs=-1, required=True) +@click.argument("args", nargs=-1) @click.pass_obj def dc_command(context: Context, command: str, args: List[str]) -> None: config = tutor_config.load(context.root)