Skip to content

Make the TLS streaming channel optional at configure time (DAQMODULES_LT_STREAMING_ENABLE_TLS) - #22

Open
viacheslauK wants to merge 1 commit into
mainfrom
tls-optional-cmake-flag
Open

Make the TLS streaming channel optional at configure time (DAQMODULES_LT_STREAMING_ENABLE_TLS)#22
viacheslauK wants to merge 1 commit into
mainfrom
tls-optional-cmake-flag

Conversation

@viacheslauK

@viacheslauK viacheslauK commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds DAQMODULES_LT_STREAMING_ENABLE_TLS (default ON) so the LT streaming modules can be built without the daq.lts:// / wss:// channel and its OpenSSL dependency. Behaviour is unchanged with the option on.

Changes

  • New CMake option, forwarded to ws-streaming as WS_STREAMING_ENABLE_TLS and published as a
    compile definition;
  • All TLS-specific code compiles out when the option is off — secure type/config factories, WSS server properties and listener, daq.lts discovery and capability, SSL error handling. A wss:// connection string then throws InvalidParameterException. A #error in common.h catches a mismatch with how ws-streaming was built.
  • Client module gains helpers for the secure/plain choice, plus isSupportedServiceName() so discovery skips unsupported services instead of throwing.
  • Tests: fixtures moved into headers, TLS-only cases split into *_module_tls.cpp built only with the option on.

@viacheslauK viacheslauK self-assigned this Aug 21, 2026
@viacheslauK
viacheslauK marked this pull request as ready for review August 21, 2026 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant