diff options
| author | Devaev Maxim <[email protected]> | 2020-09-08 10:41:28 +0300 |
|---|---|---|
| committer | Devaev Maxim <[email protected]> | 2020-09-08 10:44:38 +0300 |
| commit | 1ea68f3e9aa2f7034e0d67e8ec90c7bc3fbd95e3 (patch) | |
| tree | 468cf64ccdb987983b258db99638928d83cb4fd6 /kvmd/apps | |
| parent | 7990bf7e54fe7655816700701524a62787956b27 (diff) | |
more validation
Diffstat (limited to 'kvmd/apps')
| -rw-r--r-- | kvmd/apps/__init__.py | 45 |
1 files changed, 22 insertions, 23 deletions
diff --git a/kvmd/apps/__init__.py b/kvmd/apps/__init__.py index b20e0f12..f7fb82de 100644 --- a/kvmd/apps/__init__.py +++ b/kvmd/apps/__init__.py @@ -81,7 +81,9 @@ from ..validators.kvm import valid_stream_fps from ..validators.kvm import valid_stream_resolution from ..validators.kvm import valid_hid_key from ..validators.kvm import valid_hid_mouse_move +from ..validators.kvm import valid_ugpio_driver from ..validators.kvm import valid_ugpio_mode +from ..validators.kvm import valid_ugpio_channel from ..validators.kvm import valid_ugpio_view_table from ..validators.hw import valid_gpio_pin @@ -181,6 +183,7 @@ def _patch_dynamic( # pylint: disable=too-many-locals "gpio": {}, **tools.rget(raw_config, "kvmd", "gpio", "drivers"), }.items(): + driver = valid_ugpio_driver(driver) driver_type = valid_stripped_string_not_empty(params.get("type", "gpio")) scheme["kvmd"]["gpio"]["drivers"][driver] = { "type": Option(driver_type, type=valid_stripped_string_not_empty), @@ -188,29 +191,25 @@ def _patch_dynamic( # pylint: disable=too-many-locals } for (channel, params) in tools.rget(raw_config, "kvmd", "gpio", "scheme").items(): - try: - mode = valid_ugpio_mode(params.get("mode", "")) - except Exception: - pass - finally: - ch_scheme: Dict = { - "driver": Option("gpio"), - "pin": Option(-1, type=valid_gpio_pin), - "mode": Option("", type=valid_ugpio_mode), - "inverted": Option(False, type=valid_bool), - } - if mode == "output": - ch_scheme.update({ - "busy_delay": Option(0.2, type=valid_float_f01), - "initial": Option(False, type=(lambda arg: (None if arg is None else valid_bool(arg)))), - "switch": Option(True, type=valid_bool), - "pulse": { - "delay": Option(0.1, type=valid_float_f0), - "min_delay": Option(0.1, type=valid_float_f01), - "max_delay": Option(0.1, type=valid_float_f01), - }, - }) - scheme["kvmd"]["gpio"]["scheme"][channel] = ch_scheme + channel = valid_ugpio_channel(channel) + mode = valid_ugpio_mode(params.get("mode", "")) + scheme["kvmd"]["gpio"]["scheme"][channel] = { + "driver": Option("gpio"), + "pin": Option(-1, type=valid_gpio_pin), + "mode": Option("", type=valid_ugpio_mode), + "inverted": Option(False, type=valid_bool), + **({ + "busy_delay": Option(0.2, type=valid_float_f01), + "initial": Option(False, type=(lambda arg: (None if arg is None else valid_bool(arg)))), + "switch": Option(True, type=valid_bool), + "pulse": { # type: ignore + "delay": Option(0.1, type=valid_float_f0), + "min_delay": Option(0.1, type=valid_float_f01), + "max_delay": Option(0.1, type=valid_float_f01), + }, + } if mode == "output" else {}) + } + rebuild = True return rebuild |