fmcejudo / quarkus-eureka

46 stars 24 forks source link

Parameter 'quarkus.eureka.service-url.default' can't take multiple eureka locations #61

Open LudovicPEGON opened 1 year ago

LudovicPEGON commented 1 year ago

Hello,

In some case, it would be interressed to be able to set quarkus.eureka.service-url.default with multiple eureka location. In documentation, it 's specified but it does not work

image

In ServiceLocationConfig we could do a small change: Replace public ServiceLocationConfig(Collection<String> locationsAsString) { this.locations = locationsAsString.stream().map(Location::new).collect(Collectors.toList()); } By public ServiceLocationConfig(Collection<String> locationsAsString) { this.locations = locationsAsString.stream().map(s->s.split("\\,")) .flatMap(Stream::of).map(Location::new).collect(Collectors.toList());}

fmcejudo commented 1 year ago

Hi @lpegon , thanks for reporting that. I will take a look and replace it. Cheers.