[CAMEL-16559] camel-consul - Consul catalog action "LIST_SERVICES" calls CatalogClient.getNodes

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.9.0
• Fix Version/s: 3.10.0
• Component/s: camel-consul
• Estimated Complexity: Unknown
• Assignee: Unassigned
• Reporter: Dietrich Schulten
• Created: 29/Apr/21 06:42
• Updated: 01/May/21 07:17
• Resolved: 01/May/21 07:17

Description

Probably a copy-paste error in ConsulCatalogProducer. The handler for LIST_SERVICES calls getNodes:

@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())));
--------------------------------------------------^^^^
}

 Should call getServices(queryOptions) instead.

 

Workaround: use

.to("consul:agent?action=SERVICES") 

But that doesn't give you the node name where existing services are registered

Issue Links

• links to: GitHub Pull Request #5481