[v1.7.x] Fix securityProtocol sasl config in consumer

[gajosu]

Hi, I'm testing the package with a Sasl connection and in the consumer builder, when the securityProtocol setting is specified, argument 4 of the Sasl class is not taken into account, so the only way to make it work is to use the withSecurityProtocol. method, this PR solves the problem

[iprastha]

Hi i'm having this issue as well, and obtained the following error

Disconnected: verify that security.protocol is correctly configured, broker might require SASL authentication

If this is not merged yet, means when building the consumer , i should add the following

->withSecurityProtocol('SASL_PLAINTEXT')

is this correct ?

[mateusjunges]

Thank you @gajosu. Just a quick note: with this method, the protocol defined in the sasl config takes priority over withSecurityProtocol method.

Hi i'm having this issue as well, and obtained the following error

Disconnected: verify that security.protocol is correctly configured, broker might require SASL authentication

If this is not merged yet, means when building the consumer , i should add the following

->withSecurityProtocol('SASL_PLAINTEXT')

is this correct ?

Yes

[gajosu]

Thank you @gajosu. Just a quick note: with this method, the protocol defined in the sasl config takes priority over withSecurityProtocol method.

Hello, yes, if the ->withSasl() method is set, it will have priority, because in the Sasl class it already defines a security protocol

[mateusjunges]

@gajosu @iprastha just released this fix on v1.7.7. Thank you!