fix data race mosquitto_loop function

Fix mosquitto_pub incorrectly reusing topic aliases when reconnecting.

Closes #2494.

Don't reuse topic alias after reconnect.

next_msg_out must be protected with the msgtime_mutex
as done everywhere else in the code else there is a data race
e.g. if mosquitto_publish is called from another thread

Signed-off-by: Christian Salvasohn <csalvasohn@gmx.de>

CMake: Use PROJECT_SOURCE_DIR to improve using mosquitto as subdirectory

When mosquitto is included as subdirectory, `CMAKE_SOURCE_DIR` does not refer to the mosquitto top level CMake file, but to the whole project top level CMake.
Use `PROJECT_SOURCE_DIR` instead to refer to the right CMake in both contextes.

Signed-off-by: Pierre Hallot <hallotpierre@gmail.com>

Requires cmake 3.1.

Fix constant restart_timeout

Closes #2401. Thanks to Diorcet Yann.

Merge branch 'fix-support-tick-with-per-listener-settings' of https://github.com/XavierDooms/mosquitto into XavierDooms-fix-support-tick-with-per-listener-settings

Closes #2420. Thanks to Frédéric Fauberteau.

fix confusing log message on connack error

tests: replace use of ssl.wrap_socket that throws warnings in Python …

Fix memory leak when modifying topics using the plugin API

The function ssl.wrap_socket() is deprecated starting Python 3.7 because
it does not support hostname matching (which is considered insecure). In
Python 3.10, the function now throws warnings at runtime, which makes
Ubuntu / Debian autopkgtest fail.

The function ssl.SSLContext.wrap_socket comes in as the replacement and
has support for SNI and hostname matching.

Replaced all uses of ssl.wrap_socket() by equivalent using
ssl.SSLContext.wrap_socket().

Signed-off-by: Olivier Gayot <olivier.gayot@canonical.com>

Signed-off-by: Abilio Marques <abiliojr@gmail.com>

Signed-off-by: Abilio Marques <abiliojr@gmail.com>

add tests for the plugin tick

Signed-off-by: Xavier Dooms <dooms.xavier@gmail.com>

The default bridge configuration uses the backoff restart configuration, however this is not cleared if only a constant timeout is desired, causing it to always use the backoff configuration with a 30 second cap.

To trigger this error, use a bridge configuration with a constant timeout (e.g restart_timeout 5). Note that the timeout value is not honoured. 

Clear the backoff configuration when applying restart_timeout.

Signed-off-by: Trevor Luscombe <trevor.luscombe@gmail.com>

Closes #2367. Thanks to Jens Alfke.

Closes #2364. Thanks to Jens Alfke.

This is undefined behaviour, and some platforms don't like it.

Issue #2355. Thanks to CJ Lee.

Set ARCHIVE DESTINATION for mosquitto library

Set ARCHIVE DESTINATION for mosquittopp library

Fix DESTINATION for mosquitto_dynamic_security MODULE

On Windows MODULE will be installed as LIBRARY component
to `lib` folder that is not prefer for dynamic loaded modules
but can be found in RUNTIME DESTINATION (`bin` folder) too.

Signed-off-by: Konstantin Podsvirov <konstantin@podsvirov.pro>