spring-cloud / spring-cloud-consul

Spring Cloud Consul
http://cloud.spring.io/spring-cloud-consul/
Apache License 2.0
808 stars 540 forks source link

spring.cloud.consul.config.prefixes documentation update #746

Open shan916 opened 2 years ago

shan916 commented 2 years ago

The documentation for spring-cloud-consul has no mention of the deprecation of spring.cloud.consul.config.prefix in favor of spring.cloud.consul.config.prefixes. I had to do dig up the code to see if the new (plural) property is a YAML list or if it is a comma separated String. The documentation at the below URL should be updated with an explanation.

https://docs.spring.io/spring-cloud-consul/docs/current/reference/html/#customizing

mario45211 commented 1 year ago

up - I wasted few hours trying to figured out why my configuration has not been resolving because of this confusing doc... 👎

spencergibb commented 1 year ago

This typically wouldn't go in the documentation but in release notes. This was missed, our bad. Both properties were supported in 3.x and was removed in 4.0

spencergibb commented 1 year ago

My mistake, prefix support hasn't been removed and it should still work. @mario45211 prefix should still work.