Skip to content

Issue #15514 - Log SSL context load failures - #15521

Open
DragonFSKY wants to merge 1 commit into
jetty:jetty-12.1.xfrom
DragonFSKY:fix-15514-ssl-load-warning
Open

Issue #15514 - Log SSL context load failures#15521
DragonFSKY wants to merge 1 commit into
jetty:jetty-12.1.xfrom
DragonFSKY:fix-15514-ssl-load-warning

Conversation

@DragonFSKY

Copy link
Copy Markdown

Logs exceptions thrown while loading SslContextFactory at WARN before rethrowing them. When DEBUG is enabled, the full throwable is logged; otherwise, a single-line summary is emitted.

The change preserves StopException handling and leaves subsequent configuration checks unchanged.

Fixes #15514.

Signed-off-by: dragonfsky <dragonfsky@gmail.com>
@olamy

olamy commented Jul 31, 2026

Copy link
Copy Markdown
Member

@DragonFSKY thanks for the PR. You might need to register ECA at Eclipse. Thanks

@DragonFSKY

Copy link
Copy Markdown
Author

@olamy Thanks for the reminder. I’ve signed the Eclipse Contributor Agreement and linked my GitHub account. The ECA check is now passing.

@sbordet sbordet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is the right approach, see comment in #15514.

@stefano-1973

stefano-1973 commented Aug 1, 2026

Copy link
Copy Markdown

I don't think this is the right approach, see comment in #15514.

If you read my reply to sbordet in comment #15514, I think that your pr is good and the suggestion of sborder is not correct for all the situations.

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.

Warning message is logged only if isDebugEnabled in AbstractLifeCycle. Is it wanted?

4 participants