@vasilvestre Is it still a draft?

@vasilvestre Is it still a draft?

Yeah totally, I had no time to work on it and I'm definitly not sure I'm on the good way. I do not really know the AMQP protocol that well and I do not even talk about AMQPS.. Anyone could open another PR or continue this one.

Thanks for the fast reply. I'm going to close then as we don't keep draft PR around. If someone wants to take over, it will still be referenced in the ticket.

Thanks for the fast reply. I'm going to close then as we don't keep draft PR around. If someone wants to take over, it will still be referenced in the ticket.

Out of this, which direction should this PR take?
I stopped working on this because I didn't know how to work on this PR. I will try to be as clear as possible by keeping updated on this.

Cases (to cover)

Case 1 (simple):
I want to set up my app with a locally installed amqp broker supporting SSL. No auth is needed, no vhost, only SSL files located in /etc/ssl/{cafile,cert.pem,private_key.pem}.

Case 2 (complex):
I use a special port, special files locations for SSL. I wan't to pass it through DSN. eg : 'amqps://cacert=/tpm/ssl/cafile&cert=/tmp/ssl/cert.pem&key=/tmp/ssl/private_key.pem

Questions (I have to anyone able to answer that !)

1. Actually AmqpTransportFactory use supports function to see if 'amqp://' is contained in dsn string and accept cacert + cert + key argument to use SSL
   How are we supposed to cover AMQPS's case in your opinion?

Here's my opinion on this so far, I've decided to use default value for SSL if nothing is passed, the first goal is to cover the case 1 (the most simple).

Observations

I think this PR should aim on :

• Having correct defaults ssl values in case of 'amqps://' eg : case 1
• Being able to pass ca_cert, cert and key file paths and verify argument. eg : case 2
• Test

Are there any plans on backporting the changes done here: #38007 to Symfony 4.4? Is that something I can create a PR for if no-one is currently planning on doing so? 4.4 still has a relatively long life ahead of it, and there's currently no way of getting AMQPS working as far as I can tell. Happy to take this on, seems perfectly transferable to the existing classes?

Are there any plans on backporting the changes done here: #38007 to Symfony 4.4? Is that something I can create a PR for if no-one is currently planning on doing so? 4.4 still has a relatively long life ahead of it, and there's currently no way of getting AMQPS working as far as I can tell. Happy to take this on, seems perfectly transferable to the existing classes?

AFAIK, no backport is supported in Symfony. But as the classes aren't final, you can override them and use your own implementation that supports AMQPS IMO