Skip to content

feat(rabbit): support EXTERNAL authentication - #2953

Merged
Lancetnik merged 6 commits into
ag2ai:mainfrom
WellFREEzZ:feat/rabbit-external-auth
Jul 29, 2026
Merged

feat(rabbit): support EXTERNAL authentication#2953
Lancetnik merged 6 commits into
ag2ai:mainfrom
WellFREEzZ:feat/rabbit-external-auth

Conversation

@WellFREEzZ

Copy link
Copy Markdown
Contributor

Description

Add first-class RabbitMQ EXTERNAL authentication support.

This introduces RabbitExternalAuth, allowing users to authenticate to RabbitMQ with TLS client certificates and the auth=EXTERNAL AMQP URL parameter without mutating broker internals. The change also documents the new security mode, adds a runnable docs example, covers AsyncAPI security output, and fixes Windows-specific mypy issues in CLI supervisor signal handling.

Fixes: N/A

Type of change

  • Documentation (typos, code examples, or any documentation updates)
  • Bug fix (a non-breaking change that resolves an issue)
  • New feature (a non-breaking change that adds functionality)
  • This change requires a documentation update

Checklist

  • My code adheres to the style guidelines of this project (just lint shows no errors)
  • I have conducted a self-review of my own code
  • I have made the necessary changes to the documentation
  • My changes do not generate any new warnings
  • I have added tests to validate the effectiveness of my fix or the functionality of my new feature
  • Both new and existing unit tests pass successfully on my local environment by running just test-coverage
  • I have ensured that static analysis tests are passing by running just static-analysis
  • I have included code examples to illustrate the modifications

Verification

  • just linter passed
  • just test-rabbit tests/brokers/rabbit passed: 188 passed, 1 xfailed
  • Targeted Rabbit/AsyncAPI/docs tests passed: 20 passed
  • PYTHONUTF8=1 PYTHONIOENCODING=utf-8 just static-analysis passed
  • just zizmor passed
  • git diff --check origin/main..HEAD passed

just test-coverage currently fails locally on tests/cli/test_asyncapi_docs.py::test_serve_asyncapi_docs_from_app_with_reload. The same test fails on current main in the same environment, so it does not appear to be caused by this change.

@WellFREEzZ
WellFREEzZ requested a review from Lancetnik as a code owner July 24, 2026 13:17
@CLAassistant

CLAassistant commented Jul 24, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jul 24, 2026
@Lancetnik
Lancetnik enabled auto-merge July 29, 2026 18:30
@Lancetnik
Lancetnik added this pull request to the merge queue Jul 29, 2026
Merged via the queue into ag2ai:main with commit 2af6eb0 Jul 29, 2026
51 of 53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants