Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP #9956

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

WIP #9956

wants to merge 6 commits into from

Conversation

Al2Klimov
Copy link
Member

fixes #9955

@Al2Klimov Al2Klimov self-assigned this Jan 3, 2024
@cla-bot cla-bot bot added the cla/signed label Jan 3, 2024
@icinga-probot icinga-probot bot added area/api REST API area/distributed Distributed monitoring (master, satellites, clients) area/windows Windows agent and plugins labels Jan 3, 2024
@Al2Klimov
Copy link
Member Author

Al2Klimov commented Jan 9, 2024

@laudrup's Boost.WinTLS seems not to compile as CERT_CHAIN_ENGINE_CONFIG lacks hExclusiveRoot. That member only exists if NTDDI_VERSION >= NTDDI_WIN7 (0x06010000). The only explanation: NTDDI_VERSION <= NTDDI_WS08SP4 (0x06000400), i.e. we kinda unofficially support Windows Server 2008.

@RincewindsHat Didn't you mention customers with a such OS once?

Enables CERT_CHAIN_ENGINE_CONFIG#hExclusiveRoot used by Boost.WinTLS.
Icinga 2 already doesn't start on the original Server 2008.
@laudrup
Copy link

laudrup commented Jan 10, 2024

Boost.WinTLS seems not to compile as CERT_CHAIN_ENGINE_CONFIG lacks hExclusiveRoot. That member only exists if NTDDI_VERSION >= NTDDI_WIN7 (0x06010000). The only explanation: NTDDI_VERSION <= NTDDI_WS08SP4 (0x06000400), i.e. we kinda unofficially support Windows Server 2008.

Not sure if this is relevant for you anymore and I cannot really see that this project uses Boost.WinTLS, but I haven't really thought about what requirements there could be for the platform where Boost.WinTLS is actually used so thanks for pointing that out.

As I don't use Windows myself and mainly build and test this with Github actions where older versions of Windows are not supported I'm not sure I can figure out what to do about this, but thanks again anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api REST API area/distributed Distributed monitoring (master, satellites, clients) area/windows Windows agent and plugins cla/signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

On Windows use SChannel, not OpenSSL, for TLS connections
2 participants