bloomberg / amqpprox

An AMQP 0.9.1 proxy server, designed for use in front of an AMQP 0.9.1 compliant message queue broker such as RabbitMQ.
Apache License 2.0
78 stars 16 forks source link

Make MaybeSecureSocketAdapter::isSecure() private #78

Closed adamncasey closed 2 years ago

adamncasey commented 2 years ago

This is only used for something which could happen upfront, so I moved some things around to make this the case.

I made this change when I thought we'd want to template this class on whether it was tls or not due to integrating with boost::beast::basic_stream, but I've since figured out that isn't going to help us here so the next change isn't needed. I thought this commit was still useful to tidy up the interface.