<p></p>
<p>I am not really against, but also not seeing a real value on this approach, instead it can make troubleshooting harder for upstream, because it hides the downstream reply codes. In this case is about 500 or 408, but that can show a difference between a busy server and an unresponsive server.</p>
<p>Of course, there are cases when one wants to hide the downstream codes, but it is more likely a case per case specific.</p>
<p>Typically one has to edit that part of config to tune the list of reply codes for which to do re-routing.</p>
<p>On the other hand, the patch has syntax error, the opening <code>{</code> for <code>else</code> is missing.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/pull/2602#issuecomment-759374155">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABO7UZIKMMJNRGHPDJRDTL3SZV4PZANCNFSM4WAR3Q6Q">unsubscribe</a>.<img src="https://github.com/notifications/beacon/ABO7UZOAEQLUPSUJGY6I5M3SZV4PZA5CNFSM4WAR3Q62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFVBSCSY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/kamailio/kamailio/pull/2602#issuecomment-759374155",
"url": "https://github.com/kamailio/kamailio/pull/2602#issuecomment-759374155",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>