[checkmk-commits] [tribe29/checkmk] 680214: Decouple authorization from MonitoringCode, part 1.

Branch: refs/heads/2.0.0 Home: https://github.com/tribe29/checkmk Commit: 680214114dcd0a0a6c9b6ee298e591b02b27f12a https://github.com/tribe29/checkmk/commit/680214114dcd0a0a6c9b6ee298e591b02… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2021-02-23 (Tue, 23 Feb 2021) Changed paths: M livestatus/src/HostListColumn.cc M livestatus/src/HostListState.cc M livestatus/src/HostListState.h M livestatus/src/NagiosCore.cc M livestatus/src/ServiceGroupMembersColumn.cc M livestatus/src/ServiceListColumn.cc M livestatus/src/ServiceListState.cc M livestatus/src/ServiceListState.h M livestatus/src/TableComments.cc M livestatus/src/TableDowntimes.cc M livestatus/src/TableHosts.cc M livestatus/src/TableHostsByGroup.cc M livestatus/src/TableLog.cc M livestatus/src/TableServices.cc M livestatus/src/TableServicesByGroup.cc M livestatus/src/TableServicesByHostGroup.cc M livestatus/src/TableStateHistory.cc M livestatus/src/auth.cc M livestatus/src/auth.h Log Message: ----------- Decouple authorization from MonitoringCode, part 1. is_authorized_for() doesn't need the whole MonitoringCore, just the AuthorizationKind for services. We can push this refactoring further, which will be done in upcoming commits, this one is already big enough. Change-Id: I077461370fe0e541069469cc867af6d06ed39a9b Commit: 0f5842c9116bbd1ae72288b1cb3e12dbf582ec66 https://github.com/tribe29/checkmk/commit/0f5842c9116bbd1ae72288b1cb3e12dbf… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2021-02-23 (Tue, 23 Feb 2021) Changed paths: M livestatus/src/HostGroupsColumn.cc M livestatus/src/TableHostGroups.cc M livestatus/src/TableHostsByGroup.cc M livestatus/src/auth.cc M livestatus/src/auth.h Log Message: ----------- Decouple authorization from MonitoringCode, part 2. is_authorized_for_host_group() doesn't need the whole MonitoringCore, just the AuthorizationKind for groups and services. Change-Id: Ia9564f66b7ba4f612bb38ed0688dd4731b90f8c8 Commit: 0e1d44c302c42f9b510de5a95597a1b972098e14 https://github.com/tribe29/checkmk/commit/0e1d44c302c42f9b510de5a95597a1b97… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2021-02-23 (Tue, 23 Feb 2021) Changed paths: M livestatus/src/ServiceGroupsColumn.cc M livestatus/src/TableServiceGroups.cc M livestatus/src/TableServicesByGroup.cc M livestatus/src/auth.cc M livestatus/src/auth.h Log Message: ----------- Decouple authorization from MonitoringCode, part 3. is_authorized_for_service_group() doesn't need the whole MonitoringCore, just the AuthorizationKind for groups and services. Change-Id: I9271dbfccee52355e9a190f588aa3d8b2f374707 Commit: 0ce791c807c9300f7eee1b35c14a680f0c6fde4e https://github.com/tribe29/checkmk/commit/0ce791c807c9300f7eee1b35c14a680f0… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2021-02-23 (Tue, 23 Feb 2021) Changed paths: M livestatus/src/HostListState.cc M livestatus/src/ServiceListState.cc M livestatus/src/ServiceListState.h M livestatus/src/TableHosts.cc M livestatus/src/TableServiceGroups.cc Log Message: ----------- Decouple authorization from MonitoringCode, part 4. ServiceListState::getValueFromServices() doesn't need the whole MonitoringCore, just the AuthorizationKind for services. As a consequence, we can decouple the constructor, too. Change-Id: Ic625e59dee78708e05f1c666a6df4e4653d3e5b6 Commit: 1412581732b7fae4b038cf95dcc77e5dfc097a1e https://github.com/tribe29/checkmk/commit/1412581732b7fae4b038cf95dcc77e5df… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2021-02-23 (Tue, 23 Feb 2021) Changed paths: M livestatus/src/HostListState.cc M livestatus/src/HostListState.h M livestatus/src/TableHostGroups.cc Log Message: ----------- Decouple authorization from MonitoringCode, part 5. HostListState doesn't need the whole MonitoringCore, just the AuthorizationKind for services. Change-Id: I82078397bace206c6f036cbea6f79f2479382a26 Compare: https://github.com/tribe29/checkmk/compare/d81f9a69d200...1412581732b7