search

amqphub / amqp-10-jms-spring-boot

AMQP 1.0 JMS Binding for Spring Boot
Apache License 2.0
35 stars 25 forks source link

Polish configuration properties #3

Closed snicoll closed 7 years ago

snicoll commented 7 years ago

This commit adds documentation to each configuration key and makes default values explicit so that the generated meta-data is complete.

The remoteURL property has been renamed to brokerUrl for consistency with other messaging-based auto-configuration. Also, not suffixing it with URL prevents the canonical property name to be remote-u-r-l.

Finally, blacklist and whitelist are lists so they are now defined as such in the configuration. That way it can be configured hierarchically with yml.

snicoll commented 7 years ago

Now my initial request in #1 has been fixed, there are merge conflicts on this one. Let me know if you want me to fix them.

tabish121 commented 7 years ago

If you could rebase against master that'd be great