@ConditionalOnProperty target property with no metadata

[snicoll]

Several auto-configurations use a condition on a property that has no metadata. As a result, trying to use it to customize the auto-configuration lead to a warning in an IDE.

The report detected the following:

org.springframework.cloud.zookeeper.config.ZookeeperConfigAutoConfiguration [spring.cloud.zookeeper.config.enabled]
org.springframework.cloud.zookeeper.config.ZookeeperConfigAutoConfiguration.ZkRefreshConfiguration#configWatcher [spring.cloud.zookeeper.config.watcher.enabled]
org.springframework.cloud.zookeeper.discovery.ConditionalOnRibbonZookeeper [ribbon.zookeeper.enabled]
org.springframework.cloud.zookeeper.discovery.ZookeeperRibbonClientConfiguration#dependenciesBasedLoadBalancer [spring.cloud.zookeeper.dependency.ribbon.loadbalancer]
org.springframework.cloud.zookeeper.discovery.dependency.DependencyFeignClientAutoConfiguration [spring.cloud.zookeeper.dependency.headers.enabled]
org.springframework.cloud.zookeeper.discovery.dependency.DependencyRestTemplateAutoConfiguration [spring.cloud.zookeeper.dependency.resttemplate.enabled]
org.springframework.cloud.zookeeper.discovery.dependency.DependencyRibbonAutoConfiguration#loadBalancerClient [spring.cloud.zookeeper.dependency.ribbon.enabled]
org.springframework.cloud.zookeeper.serviceregistry.ZookeeperServiceRegistryAutoConfiguration [spring.cloud.service-registry.enabled]

[hu-chia]

BTW, spring-cloud-zookeeper-config:2.2.1-RELEASE do not have spring-autoconfigure-metadata and additional-spring-autoconfigure-metadata.

[ryanjbaxter]

Fixed via 009ee54

[snicoll]

@ryanjbaxter I don't think that commit fixed the problem I reported.

Taking the first example in the description above (updated with recent code): spring.cloud.zookeeper.enabled is used in ConditionalOnZookeeperEnabled but there is no metadata for that property. It is flagged as unresolved as far as I know.

[ryanjbaxter]

Sorry @snicoll I didnt check the issues the PR claimed it fixed, thanks for reopening.