Skip to content

Conversation

@wirew0rm
Copy link
Member

... instead of just the leaf.

Without this, verification of the ssl connection will fail in browsers which do not have the intermediaries in their cache.

To verify run MajordomoRest_example in a directory containing a valid certificate chain in demo_public.crt and corresponding key in demo_private.key and then run
openssl s_client -showcerts -connect localhost:8080 and observe that the full chain is present instead of just the leaf.

... instead of just the leaf.

Without this, verification of the ssl connection will fail in browsers
which do not have the intermediaries in their cache.

To verify run MajordomoRest_example in a directory containing a valid
certificate chain in `demo_public.crt` and corresponding key in
`demo_private.key` and then run
`openssl s_client -showcerts -connect localhost:8080` and observe that
the full chain is present instead of just the leaf.

Signed-off-by: Alexander Krimm <A.Krimm@gsi.de>
@wirew0rm wirew0rm temporarily deployed to configure coverage January 23, 2026 17:47 — with GitHub Actions Inactive
@wirew0rm wirew0rm temporarily deployed to configure coverage January 23, 2026 17:47 — with GitHub Actions Inactive
@wirew0rm wirew0rm temporarily deployed to configure coverage January 23, 2026 17:47 — with GitHub Actions Inactive
@wirew0rm wirew0rm temporarily deployed to configure coverage January 23, 2026 17:47 — with GitHub Actions Inactive
@wirew0rm wirew0rm temporarily deployed to configure coverage January 23, 2026 17:47 — with GitHub Actions Inactive
@wirew0rm wirew0rm temporarily deployed to configure coverage January 23, 2026 17:47 — with GitHub Actions Inactive
@wirew0rm wirew0rm merged commit dd1d398 into main Jan 23, 2026
8 checks passed
@wirew0rm wirew0rm deleted the restServerCertificateChain branch January 23, 2026 18:09
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
29.1% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

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.

3 participants