Well, I'd love a pull request on this. It should be a simple matter of adding the header here:

https://github.com/davehorton/drachtio-fn-b2b-sugar/blob/28e26f210feeb6e78c03994982aabaf462eafd90/lib/call-manager.js#L177

Hi @davehorton ,

Very happy to give it a go, but there's no way to pass headers into cancel function on a request, and I can't figure out how else to do it?

https://github.com/davehorton/drachtio-srf/blob/44f9848a43dba3faedb043255e2d86d87af6fe32/lib/request.js#L70 is the cancel on Request. It delegates the cancel to _agent though to me its not clear who this agent is and where to look next.

Can you get me on track?

Thanks,
Steve

ah, good point!. I will have to push a change to drachtio-srf to allow the standard opts on a cancel.

@davehorton is that just a standard someone else can do a PR for? Or will there be underlying changes needed to drachtio-server?

This is a change only to drachtio-srf. No changes required to drachtio server. I'm actually testing it now should be in shortly...

hmm, scratch that. It looks like changes may be needed to drachtio server as well..

ok as of:

[email protected], and
[email protected]

you can now attach headers to a cancel request, e.g.

req.cancel({
   headers: {
      'Reason':  'SIP;cause=200;text="Call completed elsewhere"''
   }
});

So please test, and if it looks good send me a PR for the simring change