Skip to content

Add support for client certificate authentication for AMQP 1.0#2223

Merged
thjaeckle merged 2 commits intoeclipse-ditto:masterfrom
ATNoG:amqp-connection-client-cert
Sep 16, 2025
Merged

Add support for client certificate authentication for AMQP 1.0#2223
thjaeckle merged 2 commits intoeclipse-ditto:masterfrom
ATNoG:amqp-connection-client-cert

Conversation

@JCapucho
Copy link
Contributor

Adds support for using client certificate authentication in AMQP 1.0 connections by properly configuring the SSL context to use the connection certificate and keys and by setting the EXTERNAL authentication mechanism.

@thjaeckle
Copy link
Member

Thanks a lot for the contribution @JCapucho 👍
Really appreciate it ..

Did you successfully verify that this work with an AMQP 1.0 endpoint and client certificates?
Looks almost too simple. :)

@thjaeckle thjaeckle added this to the 3.8.0 milestone Sep 10, 2025
@thjaeckle thjaeckle moved this to Waiting for Approval in Ditto Planning Sep 10, 2025
@JCapucho
Copy link
Contributor Author

JCapucho commented Sep 10, 2025

Hi @thjaeckle, I tested this against an Apache Qpid Dispatch router, and could confirm that the client certificate authentication was being used (without certificates it didn't work and even with certificates I was having problems because of the problem fixed in #2222) and that with these changes client certificates were exchanged and the handshake succeeded.

@thjaeckle
Copy link
Member

thjaeckle commented Sep 10, 2025

Started a systemTests run at: https://github.com/eclipse-ditto/ditto/actions/runs/17616067613
*edit: this is acutally for PR #2222

@thjaeckle
Copy link
Member

Started system-tests run at: https://github.com/eclipse-ditto/ditto/actions/runs/17771740291

Copy link
Member

@thjaeckle thjaeckle left a comment

Choose a reason for hiding this comment

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

System tests look good 👍

LGTM - thanks for the contribution :)

@thjaeckle thjaeckle merged commit 8d15ab6 into eclipse-ditto:master Sep 16, 2025
3 checks passed
@github-project-automation github-project-automation bot moved this from Waiting for Approval to Done in Ditto Planning Sep 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants