You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Shouldn't it be this.rabbitProperties.getSsl().determineEnabled() instead? The problem is if spring.rabbitmq.ssl.enabled is not specified in application.yml this line fails with NPE.
Also if virtualHost, username and/or password are specified on addresses instead of separate fields current implementation will ignore them, use of determine* alternatives might help here also (It looks like I should file these separately, but I'm not sure it makes sense).
@vkochnev commented on Wed Nov 03 2021
https://github.com/spring-cloud/spring-cloud-stream-binder-rabbit/blob/09c8b9bdc5a172b4034b32ff4d70b094ccb616b7/spring-cloud-stream-binder-rabbit/src/main/java/org/springframework/cloud/stream/binder/rabbit/RabbitMessageChannelBinder.java#L262
Shouldn't it be
this.rabbitProperties.getSsl().determineEnabled()
instead? The problem is ifspring.rabbitmq.ssl.enabled
is not specified inapplication.yml
this line fails with NPE.Also if
virtualHost
,username
and/orpassword
are specified on addresses instead of separate fields current implementation will ignore them, use ofdetermine*
alternatives might help here also (It looks like I should file these separately, but I'm not sure it makes sense).More comprehensive solution might be discussed in spring-projects/spring-amqp#1384
The text was updated successfully, but these errors were encountered: