[checkmk-commits] [tribe29/checkmk] 78b057: Change FetcherHeader internal representation to bytes

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 78b057e15033de9e60b695187dff75fa948746a6 https://github.com/tribe29/checkmk/commit/78b057e15033de9e60b695187dff75fa9… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-09-16 (Wed, 16 Sep 2020) Changed paths: M cmk/fetchers/controller.py M tests/unit/cmk/fetchers/test_controller.py Log Message: ----------- Change FetcherHeader internal representation to bytes The payload is now in bytes. It makes sense to have the header in bytes as well. Moreover, typing is slightly safer as we do not need to convert numbers to string and string to numbers. The code is also slightly simpler. It should also be easier to maintain and extend as we need at least one more field for the error handling. CMK-5450 Change-Id: I2dc68cd82e9db7ffc6a71c1d7e93363735b2ac54 Commit: c77b561990767836bec91f95e79006abed972919 https://github.com/tribe29/checkmk/commit/c77b561990767836bec91f95e79006abe… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-09-16 (Wed, 16 Sep 2020) Changed paths: M cmk/fetchers/controller.py M tests/unit/cmk/fetchers/test_controller.py Log Message: ----------- Add PayloadType field to FetcherHeader The new field allows us to disambiguate the type of the payload. This shall help us deserialize errors. CMK-5419 Change-Id: I38ca07fca260f5114f10977f72dbeec495181dd3 Compare: https://github.com/tribe29/checkmk/compare/5f7e0dd0eb7f...c77b56199076