[checkmk-commits] [Checkmk/checkmk] 69376c: 16112 FIX Refer to a time period by its name inste...

Branch: refs/heads/master Home: https://github.com/Checkmk/checkmk Commit: 69376c5edb202699d4828a0d36a95c082ac92c33 https://github.com/Checkmk/checkmk/commit/69376c5edb202699d4828a0d36a95c082… Author: Leonardo Petrora <leonardo.petrora(a)checkmk.com> Date: 2023-10-05 (Thu, 05 Oct 2023) Changed paths: A .werks/16112 M cmk/gui/plugins/openapi/endpoints/time_periods/__init__.py M cmk/gui/plugins/openapi/endpoints/time_periods/request_schemas.py M cmk/gui/plugins/openapi/endpoints/time_periods/response_schemas.py M tests/unit/cmk/gui/plugins/openapi/test_openapi_time_period.py Log Message: ----------- 16112 FIX Refer to a time period by its name instead of its alias in the REST API Before this Werk, a time period to be excluded was referred to by its alias and not its name. This Werk changes that behaviour and now time periods to exclude must be referred to by their name. Change-Id: I0933c4f3b993c5aed26d41db9eba702650ca87ca Commit: 6db9651a8f6c016e3832796736d52c8ec5e1c7c2 https://github.com/Checkmk/checkmk/commit/6db9651a8f6c016e3832796736d52c8ec… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2023-10-05 (Thu, 05 Oct 2023) Changed paths: A .werks/16139 M cmk/gui/plugins/openapi/endpoints/notification_rules/common_schemas.py M cmk/gui/plugins/wato/notifications.py M tests/unit/cmk/gui/plugins/openapi/test_notification_rules.py Log Message: ----------- 16139 FIX notifications: adjust regex pattern This werk addresses an issues found by the QA team with regards to the regex pattern we use to match valid PushOverPlugin api_keys and also the user group key. Previously we used [a-zA-Z0-9]{30} which will match any string that has 30 or more characters and numbers. This is partial match. We now do the following ^[a-zA-Z0-9]{30,40}$ which will match the whole string. We only allow characters and numbers with the whole string having to be of length min 30 and max 40 characters. Change-Id: I2a0f9e7a2e098241eddc75969c6a51f6bf955cb9 Commit: 3275c9e993189363fab057957c52ec4fb1c3d68d https://github.com/Checkmk/checkmk/commit/3275c9e993189363fab057957c52ec4fb… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2023-10-05 (Thu, 05 Oct 2023) Changed paths: A .werks/16152 M cmk/gui/plugins/openapi/endpoints/host_config/request_schemas.py M tests/unit/cmk/gui/plugins/openapi/test_openapi_host_config.py Log Message: ----------- 16152 FIX host_config: bulk delete hosts requires at least 1 entry This werk addresses an issue found by our QA team when bulk deleting hosts via the following endpoint. /domain-types/host_config/actions/bulk-delete/invoke The body for this endpoint requires that the entries fields is given a list of host names that the client wants to delete. However, when passing in an empty list, this was causing problems. To get around this, we've set a mininum value of items for the entries field to 1. CMK-14259 Change-Id: Iee08cb74b238660fd48b178a3720df3cd81ed16e Compare: https://github.com/Checkmk/checkmk/compare/0fbc39ee7327...3275c9e99318