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.

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

output amqp: Add ssl support #536

Closed
wants to merge 1 commit into from
Closed

output amqp: Add ssl support #536

wants to merge 1 commit into from

Conversation

ekini
Copy link
Contributor

@ekini ekini commented Jan 16, 2016

This commit adds ssl support for amqp output.

@ekini ekini mentioned this pull request Jan 16, 2016
@sparrc sparrc closed this in 37b96c1 Jan 17, 2016
@sparrc
Copy link
Contributor

sparrc commented Jan 17, 2016

👍 thanks much

@pdyba
Copy link

pdyba commented Jan 17, 2016

I also really appreciate it ! Thank You !

@pdyba
Copy link

pdyba commented Feb 11, 2016

Hi,
we have problem with TSL authentication via AMQP.
We have a certificate that works with Python application and with GO it does not.
When GO have both Plain and external authentication turned on GO uses only plain. When switching to external only I get
Exception (403) Reason: "SASL could not negotiate a shared mechanism"

Does URI mus be somehow special for cert-based auth external in GO ?

How were you testing that especially:

  1. What were your parameters for CA/server and cert/client cert ?
  2. How does the final URI look like in telegraf config
  3. What openssl version did You use on RMQ server and what on client ?
  4. What version of rabbit were You using ?
  5. Does the certificates needs some special permissions ?
  6. How is SSL configured (verify,verify_peer, fail_if_no_peer_cert, true_) in RMQ does it have any additional cyphers turned on ?
    Thank You in advance !
    Piotr

@ekini
Copy link
Contributor Author

ekini commented Feb 11, 2016

I haven't investigated it thoroughly, but there's an issue about this in Go amqp client: streadway/amqp#171

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