fix missing headers if t_reply_body is used inside a failure route block

[Riccardo-78]

Summary

This PR fixes append_to_reply not working when t_reply_with_body is used inside a failure_route block

Details

see linked issue

Solution

added update_cloned_msg_from_msg to the failure_route case.

Compatibility

Closing issues

closes #3422

[Riccardo-78]

I was wondering if there’s anything holding back the merge of my pull request? Let me know if I can assist with anything!

[bogdan-iancu]

Hi @Riccardo-78 - thanks for the PR here. Nevertheless, I'm not convinced this is the right fix. If we do update_cloned_msg_from_msg in the failure route, then all the changes you did in failure route will stick the transaction for all the future branches, not only for the current branch. Which is not the intended behavior (in failure route, your changes impact only the branches you create at that very moment, not all the future ones).
I think it is a deeper problem with the t_reply_with_body function - as the comment here, the problem is that the function takes as param the request from the transaction, not the current sip_msg provided from the script level - not sure why, I can check on this