getConsumers with QueryParameters about hop HOT 8 CLOSED

@mheidt this is open source software. If you need something, please submit a PR

from hop.

The consumers endpoint does not support pagination and I don't think it supports filtering by name/regex.

from hop.

I know. that's why I closed it. I thought, the people behind this are in contact with the people of the real API as well.

from hop.

@michaelklishin
It seems, that we have a major issue with this call.
Our productive rabbit has a shovel defined and those connections look like
{
"arguments": {},
"ack_required": true,
"active": true,
"activity_status": "up",
"channel_details": {
"connection_name": "[email protected]",
"name": "[email protected] (1)",
"node": "rabbit@10-244-108-91",
"number": 1,
"peer_host": "undefined",
"peer_port": "undefined",
"user": "shovel"
},
"consumer_tag": "amq.ctag-uLSmF70x9pQEDL34OCbkvg",
"exclusive": false,
"prefetch_count": 1000,
"queue": {
"name": "amq.gen-vIuezFJTmuEAOB2SouuafQ",
"vhost": "PROD"
}
},

The peer_port yields to
JSON parse error: Cannot deserialize value of type int from String "undefined": not a valid Integer value;

Should I make a PR, that is making ChannelDetails.peerPort a String?

from hop.

Of course nicer would be if the API (not hop) would support the filter...

from hop.

@michaelklishin: Alternatively (and not breaking the http API) it might be possible to fix this in the shovel plugin, so that either a real portnumber or "0" is set for unknown port in peer_port !?

from hop.

The management API should fix the issue, maybe a proper default value would be 0 or -1, I don't know if this happens in other cases. It's also weird the values are not set.

This should also be fixed in Hop, in case the client is used with broker versions without the fix. I'd say the simplest is to use a custom deserializer that handles the undefined value. We cannot change the type, it would be a breaking change.

from hop.

Closed in favor of #248.

from hop.