Fix typo

Merge pull request #6643 from Icinga/bugfix/command-timeout-notifications

Fix that check_timeout was used for Event/Notification commands too

Fix that check_timeout was used for Event/Notification commands too

We may add specific timeouts for event/notification commands
later, for now the original timeout inside the EventCommand/NotificationCommand
is used.

fixes #6304

Merge pull request #6642 from Icinga/bugfix/concurrent-checks-constant

Allow to override MaxConcurrentChecks constant

Merge pull request #6546 from WuerthPhoenix/fix/define-switches-not-passed-during-config-test

Overridden path constants not passed to config validation in /v1/config/stages API call

Don't inherit daemonize parameter from parent process

Fix config validation problem (startup.log) during /v1/config/stages API call

copy all arguments of parent process in AsyncTryActivateStage

Allow to override MaxConcurrentChecks constant

Merge pull request #6313 from Icinga/fix/win-check-swap

Fix wrong calculation of check_swap windows plugin

Merge pull request #6639 from Icinga/fix/windows-api-log-rename

Ensure to _unlink before renaming replay log on Windows

Fix issue 5022

Merge branch 'support/2.9'

Release version 2.9.2

Release version 2.9.2

Merge pull request #6632 from Icinga/feature/cluster-faster-reconnect

Increase the cluster reconnect frequency to 10s

Merge pull request #6633 from Icinga/feature/apilistener-conn-thread-pool

Use a dynamic thread pool for API connections

Use a dynamic thread pool for API connections

The full analysis is located in #6517.

fixes #6517

Increase the cluster reconnect frequency to 10s

This is blocked by #6517.

refs #6234

Merge pull request #6630 from Icinga/bugfix/socket-events-init

Initialize Socket Event Engine on application startup

Initialize Socket Event Engine on application startup

Previously this happened inside the TlsStream constructor
during the first connection attempt.

Update variable names

refs #6568

Ensure that config object types are committed in dependent load order

refs #6568

Shuffle items before config validation

refs #6581

Fix crash on API queries with Fedora 28 hardening and GCC 8

The actual fix is to handle nullptr references differently
for an empty filter expression. The other changes include
oob checks not necesarily involved.

fixes #6533
refs #6596

Merge pull request #6622 from Icinga/bugfix/ido-pgsql-cleanup-timestamp-utc

Ensure to use UTC timestamps for IDO PgSQL cleanup queries

Merge pull request #6616 from Icinga/feature/tls-handshake-timeout-config

Add ApiListener#tls_handshake_timeout option

Add a note about tls_handshake_timeout to upgrading docs

Merge pull request #6621 from Icinga/bugfix/ido-remove-unused-timestamp-function

Remove unused timestamp function in DB IDO

Merge pull request #6618 from Icinga/bugfix/nice-compiler-warning

Silence compiler warning for nice()

Ensure to use UTC timestamps for IDO PgSQL cleanup queries

refs #6584

Remove unused timestamp function in DB IDO

refs #6584

Silence compiler warning for nice()

Add ApiListener#tls_handshake_timeout option

This allows to specify the previously hardcoded
timeout of 10s.

refs #6517

Merge pull request #6615 from Icinga/feature/docs-distributed-monitoring

Update distributed monitoring docs for 2.10

Drop older than 2.8 references in distributed monitoring docs

With 2.10, 2.8 will be the least supported version, older doc
versions can still be found on GH if needed.

Distributed Monitoring Docs: Adopt three level scenario for CA proxy

fixes #6593

Distributed Monitoring Docs: Add versions and upgrade

refs #6612

Merge pull request #6519 from Icinga/fix/erroneous-console-exit

Reset terminal on erroneous console exit

Merge pull request #6610 from Icinga/feature/docs-technical-concepts-tls-network-io

Add "TLS Network IO" into technical concepts docs

Update workaround for custom vars

This updates #6572 which did not work

refs #6590

Add note about workaround for broken custom vars

refs #6572

Improve error logging for match/regex/cidr_match functions and unsupported dictionary usage

fixes #6442
refs #6497

Improve TLS handshake exception logging

refs #6602

Merge pull request #6527 from Icinga/feature/acknowledgment-notifications-6047

Acknowledgment notifications should only be send if problem notification has been send

Update notification and upgrading docs for recovery/acknowledgement handling

refs #6047

Only check for Problem types in user filers for Recovery/Acknowledgement notifications

refs #6047

Still notify on acknowledgement and recovery if user does not get problem notifications at all

refs #6047

Only send acknowledgement notification if user received a problem notification

refs #6047

Add "TLS Network IO" into technical concepts docs

refs #6517

Merge pull request #6607 from Icinga/feature/docs-dev-gdb-running-process

Enhance development docs with GDB backtrace and thread list

Enhance development docs with GDB backtrace and thread list

Merge pull request #6596 from Icinga/bugfix/gcc-8-f28-hardening-crash

Fix crash on API queries with Fedora 28 hardening and GCC 8

Merge pull request #6606 from Icinga/feature/enhance-contributing-docs

Enhance contributing docs

Mention 'help wanted' and 'good first issue' in contributing docs

Rename branch schema from `fix/` to `bugfix/`

Merge pull request #6573 from Icinga/bugfix/operator-docs-1

Fix operator precedence table

Fix operator precedence table

Fixes #5573

Merge pull request #6602 from Icinga/fix/improve-tls-handshake-exception-logging

Improve TLS handshake exception logging

Improve TLS handshake exception logging

Merge pull request #6595 from Icinga/feature/limit-anonymous-rpc-connections

Allow to configure anonymous clients limit inside the ApiListener object

Merge pull request #6598 from dominik-r-s/patch-1

doc/09-object-types: states filter ignored for Acknowledgements

Merge pull request #6597 from Icinga/feature/docs-development-fedora-debuginfos

Add Fedora to development docs for debuginfo packages

Add Fedora to development docs for debuginfo packages

Fix crash on API queries with Fedora 28 hardening and GCC 8

The actual fix is to handle nullptr references differently
for an empty filter expression. The other changes include
oob checks not necesarily involved.

fixes #6533

Allow to configure anonymous clients limit inside the ApiListener object

Previously this was hardcoded, and for security reasons users might want
to adjust this value. This affects CSR signing requests as well as
clients which have not yet been configured as endpoints on the current
node.

refs #6566

Raise the message size for anonymous client and pki request calls to 1MB

If one sends the full certificate chain, this previous limit of 64KB
could be hit.

09-object-types: notifications: states filter ignored for Acknowledgements!

Merge pull request #6591 from Icinga/bugfix/lto-builds-static-initialize-namespaces

Fix static initializer priority for namespaces in LTO builds

Merge pull request #6590 from Icinga/fix/custom-var-workaround

Update workaround for custom vars

Fix static initializer priority for namespaces in LTO builds

fixes #6575

Merge pull request #6356 from sourcejedi/fix/systemd

Fix logging under systemd

Merge pull request #6568 from Icinga/feature/commit-order

Ensure that config object types are committed in dependent load order

Update workaround for custom vars

This updates #6572 which did not work

Update variable names

Ensure that config object types are committed in dependent load order

Merge pull request #6581 from Icinga/fix/speedup-config

Shuffle items before config validation

Shuffle items before config validation

Merge pull request #6588 from Icinga/bugfix/prepare-dirs-safe-reload-full-path

Fix using full path in prepare-dirs/safe-reload scripts

Fix using full path in prepare-dirs/safe-reload scripts

This won't be visible with packages and overridden paths,
just source builds which invoke the scripts manually.

refs #6506

Merge pull request #6583 from maxswjeon/master

Update PostgreSQL library path variable in INSTALL.md

Merge pull request #6586 from Icinga/bugfix/centos-7-buildfix-non-unity-builds

Fix non-unity builds on CentOS 7 with std::shared_ptr

Fix non-unity builds on CentOS 7 with std::shared_ptr

refs #6509

Merge pull request #6574 from gunnarbeutner/fix/move-downtime-constants

Move new downtime constants into the Icinga namespace

Edited INSTALL.md File

Edited the Mistake of the PostgreSQL_LIBRARY_DIR comment error.
#6582

Move new downtime constants into the Icinga namespace

Fix logging under systemd

icinga2.service used `-e ${ICINGA2_ERROR_LOG}`, but this is documented
as having no effect without `-d`.  Furthermore, icinga2 under systemd
unconditionally logged everything to the system log (but without setting
the log level etc), which contradicted the documentation.  (Issue #6339)

Stop icinga2 on systemd from logging to stdout - and hence the system log -
once it has finished starting up.  Just like when you start icinga2 from a
terminal using `-d`.  And just like -d, we stop logging fatal errors to
stderr, and instead write to the log file passed with `-e`.

As per docs, mainlog (icinga2.log) is already enabled by default.  And
pre-startup messages including config errors will still appear in the
system log.

This uses a new option --close-stdio, which has the same effect on logging as
--daemonize, but does not fork or call setsid().

For this purpose, I moved setsid() up and into Daemonize().

Consequence of that last point: if anyone is weird enough to specify a TTY
device file as the fatal error log (-e option), that will become icinga's
controlling terminal, which you generally don't want as a daemon.  This
makes it consistent with the existing behaviour for icinga mainlog.  For
this reason you're supposed to use O_NOCTTY in Linux daemons.  But I wasn't
sure where icinga would want to put the ugly `#ifdef _WIN32 ... #else ...`.

Daemonize(): use one error convention, not three

Standardize on exit() / _exit() (this depends whether we are considered
to be the "main" fork, which should run anything registered with atexit()).
Exclude `return false` and throwing exceptions.

This fixes the error path for fork().  Daemonize() would return false, but
the `return false` error convention was not tested in the caller.

It also fixes the error message for fork() to show the error code.
Everyone loves `strace`, but sysadmins should not have to rerun their
daemons under it just to see an error code.

Also in case an exception is thrown, show its diagnostic information
instead of dropping it on the floor.  In the log message, I mention why we
are uninitializing and then initializing the app at this point.

For the reader, it pushes all the weirdness into the error convention of
Daemonize().  This comes back to the exit() / _exit() distinction.  Once
we have forked, we technically don't want to allow the parent process to
exit(), so we don't want to return to the caller.

fix "Console" log to flush

It's called "Console", which would be line-buffered anyway.  But, it's
implemented as std::cout.  This might be piped to a logger, as in
daemontools or systemd.  In this case it will not be a TTY, and log lines
should be flushed without too much delay.  Let's just flush each message.

Let's not introduce a static instance of StreamLogger (flushed by interval
timer).  That's too stressful to read, because static instances are really
annoying to order.  Example citation: "Yay, our static destructors are
pretty much beyond repair at this point." -- application.cpp.

I don't know if there will be any need to optimize logging syscalls.  The
init script uses `--daemonize`.  I think the systemd service should also
avoid using the "Console" log after startup (see next commit).  The
documentation does not warn that the syslog feature is less efficient
in system calls than mainlog; deferred flusing does not seem to be a highly
prominent feature.  There's no cool comment in the code about how much the
syscalls were slowing down some use case (or qualifying that this
optimization can only eliminate syscalls on platforms with both mutexes and
clocks that can work without syscalls).

Merge pull request #6532 from Icinga/feature/allow-down-times-to-apply-on-child-hosts-3935

Add child_options to ScheduledDowntime

Add upgrading docs for API schedule-downtime 'child_options'

Merge pull request #6502 from jenslink/patch-2

Update 17-language-reference.md

Merge pull request #6501 from jenslink/patch-1

Update 03-monitoring-basics.md

Merge pull request #6510 from Icinga/feature/windows-build-scripts

Update Appveyor build scripts

Add child_options to ScheduledDowntime

refs #3935

Merge pull request #6509 from gunnarbeutner/feature/real-constants

Implement support for namespaces

Update upgrading docs for v2.10 and namespaces

Merge pull request #6572 from Icinga/bugfix/29X-custom-vars-upgrade

Add note about workaround for broken custom vars

Add note about workaround for broken custom vars

Merge pull request #6570 from Icinga/bugfix/tls-anonymous-clients-limit

Increase limit for simultaneously connected anonymous TLS clients

Increase limit for simultaneously connected anonymous TLS clients