[ 
https://issues.apache.org/jira/browse/CAMEL-16559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17337737#comment-17337737
 ] 

Dietrich Schulten commented on CAMEL-16559:
-------------------------------------------

https://github.com/apache/camel/pull/5481

> camel-consul - Consul catalog action "LIST_SERVICES" calls 
> CatalogClient.getNodes
> ---------------------------------------------------------------------------------
>
>                 Key: CAMEL-16559
>                 URL: https://issues.apache.org/jira/browse/CAMEL-16559
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-consul
>    Affects Versions: 3.9.0
>            Reporter: Dietrich Schulten
>            Priority: Minor
>             Fix For: 3.10.0
>
>
> Probably a copy-paste error in {{ConsulCatalogProducer}}. The handler for 
> {{LIST_SERVICES}} calls {{getNodes}}:
> {code:java}
> @InvokeOnHeader(ConsulCatalogActions.LIST_NODES)
> protected void listNodes(Message message) throws Exception {
>     processConsulResponse(message, 
> getClient().getNodes(buildQueryOptions(message, getConfiguration())));
> }
> @InvokeOnHeader(ConsulCatalogActions.LIST_SERVICES)
> protected void listServices(Message message) throws Exception {
>     processConsulResponse(message, 
> getClient().getNodes(buildQueryOptions(message, getConfiguration())));
> --------------------------------------------------^^^^
> }{code}
>  Should call {{getServices(queryOptions)}} instead.
>  
> Workaround: use
> {code:java}
> .to("consul:agent?action=SERVICES") {code}
> But that doesn't give you the node name where existing services are registered



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to