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.

Sign up for GitHub

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

Jump to bottom

Qpid transport cannot use EXTERNAL as a login_method #501

Closed
bmbouter opened this issue Aug 4, 2015 · 1 comment
Closed

Qpid transport cannot use EXTERNAL as a login_method #501

bmbouter opened this issue Aug 4, 2015 · 1 comment
Assignees
@bmbouter
Labels
Component: Qpid Broker

Comments

@bmbouter
Copy link
Contributor

bmbouter commented Aug 4, 2015

The docs indicate that login_method can be passed as a keyword argument to the Connection object and this is expected to override the SASL selection mechanism. The Qpid transport does not respect this argument and ignores it. The precludes Kombu from using EXTERNAL SASL mechanism which is required for some deployments.
@bmbouter bmbouter self-assigned this Aug 4, 2015
bmbouter added a commit to bmbouter/kombu that referenced this issue Aug 4, 2015
@bmbouter
Adds support, tests, and docs for login_method with Qpid transport.
7c04c87 
Fixes celery#501
Fixes celery#499
bmbouter added a commit to bmbouter/kombu that referenced this issue Aug 4, 2015
@bmbouter
Adds support, tests, and docs for login_method with Qpid transport.
5b29e9b 
Fixes celery#501
Fixes celery#499
bmbouter added a commit to bmbouter/kombu that referenced this issue Aug 4, 2015
@bmbouter
Adds support, tests, and docs for login_method with Qpid transport.
3cb9671 
Fixes celery#501
Fixes celery#499
bmbouter added a commit to bmbouter/kombu that referenced this issue Aug 4, 2015
@bmbouter
Adds support, tests, and docs for login_method with Qpid transport.
dc4e30d 
Fixes celery#501
Fixes celery#499
@bmbouter bmbouter mentioned this issue Aug 4, 2015
Merged
@bmbouter
Copy link
Contributor Author

bmbouter commented Aug 4, 2015

This has been fixed with dc4e30d and will be available in the next Kombu release which I expect to be 3.0.27.

@bmbouter bmbouter closed this as completed Aug 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bmbouter bmbouter

Labels
Component: Qpid Broker
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@thedrow @bmbouter