12084 Commits

Author SHA1 Message Date
Alexander A. Klimov
516fd21fec ConfigObjectUtility::GetObjectConfigPath(): hash names of not already existing objects
... to avoid too long file names.

refs #8022
2021-06-15 10:41:27 +02:00
Julian Brost
685cc5c155
Merge pull request #8801 from Icinga/feature/release-v2.12.4
Version update and changelog for v2.12.4
v2.12.4
2021-05-27 09:26:57 +02:00
Alexander Aleksandrovič Klimov
b7ff749c77
Merge pull request #8809 from Icinga/bugfix/mysql-one-transaction-for-programstatus-2.12
IDO: Use own transaction for program status and make sure InternalNewTransaction() gets executed
2021-05-26 17:50:29 +02:00
Alexander Aleksandrovič Klimov
f2cd24b437
Merge pull request #8771 from Icinga/bugfix/retry-rename-on-windows-2.12
Retry file rename operations on Windows for some errors (2.12)
2021-05-26 17:50:22 +02:00
Alexander Aleksandrovič Klimov
6ceba45739
Merge pull request #8777 from Icinga/bugfix/runworker-exceptions-2.12
Improve handling of exceptions thrown by RunWorker (2.12)
2021-05-26 16:44:54 +02:00
Julian Brost
c1cee03833 Version update and changelog for v2.12.4 2021-05-26 16:09:30 +02:00
Alexander Aleksandrovič Klimov
8f0b7910b9
Merge pull request #8775 from Icinga/bugfix/problem-notification-at-downtime-end-2.12
Send problem notifications after downtime end for checkables in child zones (2.12)
2021-05-26 11:04:56 +02:00
Alexander Aleksandrovič Klimov
bb4356cc83
Merge pull request #8782 from Icinga/bugfix/concurent-notification-send-and-delete-2.12
Fix crash when notifications are sent while the notification object is deleted (2.12)
2021-05-26 11:01:55 +02:00
Alexander Aleksandrovič Klimov
c18cb90f27
Merge pull request #8785 from Icinga/bugfix/concurrent-schedule-downtime-delete-host-2.12
Fix null pointer dereferences when deleting objects while scheduling downtimes (2.12)
2021-05-26 11:00:03 +02:00
Alexander Aleksandrovič Klimov
204bac296e
Merge pull request #8787 from Icinga/bugfix/api-put-object-deletion-212-7726
ConfigObjectUtility#CreateObject(): check config objects for duplicates
2021-05-26 10:44:38 +02:00
Alexander Aleksandrovič Klimov
195ae0a2ce
Merge pull request #8808 from Icinga/bugfix/reminder-notification-after-timeperiod-with-interval-0-2.12
Make sure not to send reminder notification, if problem notification has been suppressed by TimePeriod
2021-05-26 10:44:21 +02:00
Noah Hilverling
24cbd0685b IDO-MySQL: Make sure InternalNewTransaction() and FinishAsyncQueries() get executed during high load 2021-05-25 17:22:26 +02:00
Noah Hilverling
7d8be5ee69 IDO: Use own transaction for programstatus 2021-05-25 17:13:21 +02:00
Noah Hilverling
511f6bd6b5 Notifications: Make sure not to send reminder notification, if problem notification has been suppressed 2021-05-25 16:59:38 +02:00
Alexander A. Klimov
817fe3fcac ConfigObjectUtility#CreateObject(): check config objects for duplicates
... not to delete already existing objects during a trial of re-creation.

refs #7726
2021-05-25 16:29:58 +02:00
Julian Brost
35c9c656a0 AddDowntime: return Downtime::Ptr instead of String containing the name
At numerous places in the code, something like this is performed:

    String name = Downtime::AddDowntime(...);
    Downtime::Ptr downtime = Downtime::GetByName(name);

However, `downtime` can be a `nullptr` after this as it is possible that
the downtime is deleted in between.

This commit changes the return type of `Downtime::AddDowntime` to return
a Downtime::Ptr instead of the full name of the downtime. `AddDowntime`
performs the very same `GetByName()` operation internally, but handles
the `nullptr` case correctly and throws an exception.
2021-05-25 15:59:55 +02:00
Julian Brost
0cef85cf83 Properly handle service downtime referencing a deleted host
Only two out of three cases were handled properly by the code: host
downtimes referencing a deleted host and service downtimes referencing a
deleted service worked fine. However, if a service downtime references a
deleted host, `Host::GetByName()` returns `nullptr` which isn't
accounted for. Use `Service::GetByNamePair()` instead as this performs a
check for the host being null internally.
2021-05-25 15:59:55 +02:00
Julian Brost
30b7a1e65b Use reference-counted pointer in notification callback
`this` could be deleted after `Notification::BeginExecuteNotification`
exited and before `Notification::ExecuteNotificationHelper` finished.
This is fixed by constructing a `Notification::Ptr` and operate on that
one as it is properly reference-counted.
2021-05-25 15:59:29 +02:00
Julian Brost
c4e58477a0 Try to log useful information for exceptions thrown by RunWorker 2021-05-25 15:58:53 +02:00
Julian Brost
b98a96258c Catch exceptions thrown by RunWorker on Windows
The same is already done on other platforms in line 529.
2021-05-25 15:58:53 +02:00
Julian Brost
ddafaa0add Only handle event::SetSuppressed{Notifications,NotificationTypes} within the local zone
Note that even when passing `nullptr` as target zone to `RelayMessage()`, the
cluster message will still be sent to the parent zone. These incoming messages
will now be rejected by the parent nodes. At the moment, there's no way to only
send within the local zone.
2021-05-25 15:58:17 +02:00
Julian Brost
d109dfcb78 Retry file rename operations on Windows for some errors 2021-05-25 15:57:41 +02:00
Alexander Aleksandrovič Klimov
da1dd05dca
Merge pull request #8792 from Icinga/bugfix/212boost174
Support Boost 1.74
2021-05-25 15:51:13 +02:00
Julian Brost
31077fa99f
Merge pull request #8806 from Icinga/probot/sync-changelog/86c43e86255aa35813284f80af5cbda6165c01a7
CHANGELOG.md: add v2.11.8
2021-05-25 15:25:21 +02:00
Noah Hilverling
bc5ad5d79a CHANGELOG.md: add v2.11.8 2021-05-25 13:24:34 +00:00
Julian Brost
86c43e8625
Merge pull request #8805 from Icinga/probot/sync-changelog/badbbe29192cb18bf9347b7837ef5a623243a7e4
CHANGELOG.md: add v2.11.7
2021-05-25 15:24:21 +02:00
Julian Brost
e1c3731915 CHANGELOG.md: add v2.11.7 2021-05-25 13:23:29 +00:00
Julian Brost
badbbe2919
Merge pull request #8804 from Icinga/probot/sync-changelog/8a6a452a78dedd88d57ec522c3962c637db578f7
CHANGELOG.md: add v2.11.6
2021-05-25 15:23:17 +02:00
Julian Brost
bff9c6385f CHANGELOG.md: add v2.11.6 2021-05-25 13:22:32 +00:00
Julian Brost
8a6a452a78
Merge pull request #8803 from Icinga/probot/sync-changelog/be7b238ef4c33ffc38f12cd5908b278dcd90233c
CHANGELOG.md: add v2.11.5
2021-05-25 15:22:17 +02:00
Alexander A. Klimov
1e4f9e3766 CHANGELOG.md: add v2.11.5 2021-05-25 12:59:38 +00:00
Julian Brost
be7b238ef4
Merge pull request #8797 from Icinga/probot/sync-changelog/b11d6f37b78c9b09018a49ec5072b290d7110547
CHANGELOG.md: add v2.11.4
2021-05-25 14:59:23 +02:00
Alexander Aleksandrovič Klimov
e6bbacbfeb
Merge pull request #8790 from Icinga/feature/gha212
GitHub actions: update distros
2021-05-25 12:08:51 +02:00
Noah Hilverling
a4fabb2898 CHANGELOG.md: add v2.11.4 2021-05-25 08:17:04 +00:00
Alexander Aleksandrovič Klimov
b11d6f37b7
Merge pull request #8795 from Icinga/probot/sync-changelog/0cba2d73a5d1288b9cdc164135fd472e41dfdbf1
CHANGELOG.md: add v2.10.7
2021-05-25 10:16:44 +02:00
Michael Friedrich
db3af6db15 CHANGELOG.md: add v2.10.7 2021-05-21 08:22:41 +00:00
Alexander Aleksandrovič Klimov
0cba2d73a5
Merge pull request #8794 from Icinga/probot/sync-changelog/d404a563ec554594f9dc948ac0d1f90b350eb9e8
CHANGELOG.md: add v2.10.6
2021-05-21 10:22:27 +02:00
Michael Friedrich
08fa73ac64 CHANGELOG.md: add v2.10.6 2021-05-20 16:27:48 +00:00
Alexander Aleksandrovič Klimov
d404a563ec
Merge pull request #8774 from Icinga/probot/sync-changelog/2cb995e93714ab82914e8cace050749ba6359213
CHANGELOG.md: add v2.9.3
2021-05-20 18:27:34 +02:00
Julian Brost
cf4dd94b97 Provide a conversion function from icinga::String to boost::string_view
Boost.Beast changed the signature of
boost::beast::http::basic_fields::set in version 1.74 so that no longer
allows passing an icinga::String instance as value. This adds a
conversion function so that it works again.
2021-05-20 16:27:51 +02:00
Julian Brost
58416c3c61 Use content_length method for setting the Content-Length header
Boost.Beast changed the signature of the previously used generic `set`
method so that it no longer accepts integer types, however there is
alreay a more specific method for setting the Content-Length header, so
use this one instead.
2021-05-20 16:27:51 +02:00
Alexander A. Klimov
612e4a5ef7 Define BOOST_ASIO_USE_TS_EXECUTOR_AS_DEFAULT
... to enable compiling with Boost v1.74.

refs #8185
2021-05-20 16:27:51 +02:00
Louis Sautier
04f4c132ff Fix ‘fs::copy_option’ has not been declared with boost 1.74.0
It was deprecated in
f199152b7d
2021-05-20 16:27:51 +02:00
Alexander A. Klimov
a41838547c GitHub actions: update distros 2021-05-20 15:43:49 +02:00
Alexander A. Klimov
ff9cf5a573 GitHub actions: don't cancel not yet failed jobs on a failure 2021-05-20 15:38:43 +02:00
Alexander A. Klimov
216dac1985 GitHub actions: don't upload any artifacts 2021-05-20 15:38:43 +02:00
Michael Friedrich
7329618a39 CHANGELOG.md: add v2.9.3 2021-05-20 11:11:00 +00:00
Noah Hilverling
2cb995e937
Merge pull request from GHSA-pcmr-2p2f-r7j6
Verify certificates against CRL before renewing them (2.12)
v2.12.3
2020-12-15 12:30:19 +01:00
Noah Hilverling
a995dc0577
Merge pull request #8542 from Icinga/bugfix/openssl-error-buffer-2.12
Use proper buffer size for OpenSSL error messages (support/2.12)
2020-12-15 12:13:56 +01:00
Noah Hilverling
61c78311b7 Add security fix to CHANGELOG 2020-12-15 10:49:01 +01:00