[micro] Add API to read Reply subject from `micro.Request` about nats.go HOT 6 CLOSED

@piotrpio I commented on the PR as well, but for our purpose, sometimes we hand messages around multiple services or handlers before responding to a client, and need to be able to pass the reply subject to the down-line code. Easy to do with nats.Msg, but not micro.

from nats.go.

@mcosta74, @themartorana Sorry it took so long and thank you for the patience. We discussed it internally and agreed that we will not consider adding public methods to interfaces a breaking change for the client. We have done it in the past with JetStreamContext so we can do the same for micro. I've created a PR adding a note about that to readme to make it official.

I'll review your PR in a moment, on first glance it looks simple and straightforward enough 😉

from nats.go.

Hey @mcosta74. It's a good idea to add the ability to check reply subject, but I'm not sure about introducing a new interface... That would force us to also add v2 of Handler interface, along with Handle method and HandlerFunc.

Another idea would be to add a RespondOpt which would allow skipping sending the response if reply is empty, but that's not ideal as well since you'd not know whether the response was sent or not.

from nats.go.

Hi @piotrpio , my need is to know if I need to prepare/Marshall the response or I can skip it because there isn't a Reply.

So your 2nd option doesn't fit what I'm looking for.

from nats.go.

That would force us to also add v2 of Handler interface, along with Handle method and HandlerFunc.

As I said, I'll be happy to implement the change of the proposal is accepted

from nats.go.

@piotrpio Can I just add a Reply() method to the Request interface? Is adding a new method the the interface considered a breaking change ?

from nats.go.