miconda left a comment (kamailio/kamailio#4304)

I am not sure I understand exactly the use case. Is it about a REGISTER request getting to a SIP proxy that does parallel forking to many downstream SIP registrars? If yes, what do you want to happen with replies having the code >= 300? To be forwarded back to the initial sender of the REGISTER request?

If it is about authentication, why not only 401/407? Shall a 400 be forwarded as well?

Also, the description does not seem right, for the INVITE, if the transaction uas status is >= 200, then only 2xx responses are forwarded (in the code it does not seem that >=300 are still forwarded for the INVITEs). Or I am missing something?

Looking at the overall diff at this moment:

https://patch-diff.githubusercontent.com/raw/kamailio/kamailio/pull/4304.diff

The first changed line to the LM_DBG() does not seem useful, just breaks the code formatting. Also, a non-necessary empty line is added.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.