krakend / krakend-amqp

AMQP compatible backend for the KrakenD framework
https://www.krakend.io
Apache License 2.0
10 stars 16 forks source link

Producer: routing_key works only for URL parameters #21

Closed lennondtps closed 10 months ago

lennondtps commented 1 year ago

Hello, Following the documentation here https://www.krakend.io/docs/backends/amqp-producer/#routing_key the routing key is expected to work with static values.

Unfortunately, this functionality is not supported by the code.

I am willing to open a Pull Request to make the code supporting this. Are you ok with including this functionality?

@kpacha @alombarte is there any alternative to make use of static routing values?

alombarte commented 1 year ago

Hello @lennondtps

Thanks for your offering. From a product perspective, the functionality sounds good. If the code ends up in the repository or not depends on the implementation and efficiency of the solution.

Feel free to send a PR for review

lennondtps commented 12 months ago

Hello @alombarte ! Feel free if you have any comments/suggestions for my PR --> https://github.com/krakend/krakend-amqp/pull/22