[checkmk-commits] [tribe29/checkmk] 51f208: Protocol: Split into Serializer and Deserializer

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 51f208b93d9218250b8da311bdee3cc9f2ae555e https://github.com/tribe29/checkmk/commit/51f208b93d9218250b8da311bdee3cc9f… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2022-01-19 (Wed, 19 Jan 2022) Changed paths: M cmk/core_helpers/protocol.py M cmk/utils/type_defs/protocol.py Log Message: ----------- Protocol: Split into Serializer and Deserializer Change-Id: I59c2eaba2dd445973114ece492cd99f452dc21a0 Commit: 6fb8a5360bd2c1c2305707961d24a9bdeb8a3ee0 https://github.com/tribe29/checkmk/commit/6fb8a5360bd2c1c2305707961d24a9bde… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2022-01-19 (Wed, 19 Jan 2022) Changed paths: A .werks/13197 M cmk/utils/regex.py A tests/unit/cmk/gui/utils/test_init.py Log Message: ----------- 13197 FIX Add space as valid URL character With Werk 13195 a stricter URL validation was introduced. In the original implementation the space character was not allowed. Unfortunately servicenames with spaces are not encoded properly and spaces used to work. Since this is also common behaviour and does not impose security risks the space is allowed again. It seems that we are quite lax with URL encoding our links and browsers are very tolerant. The RFC also asks for robustness for user-typed URIs. Therefore allowing the space character as valid URI character. Change-Id: I1044e7cd84aa3acf145cbdd493cdd3873393c2c7 Compare: https://github.com/tribe29/checkmk/compare/c93816bec5b4...6fb8a5360bd2