[sr-dev] [kamailio/kamailio] Branch ordering behavior changed between v5.3 and v5.4 (#2449)

[Daniel-Constantin Mierla]

Not using t_load_contacts() was not giving me a chance earlier to investigate deeply, but finally I got some time and I went on troubleshooting by diff-ing the code and behaviour between the versions and came up with the commit referenced above. 

The result of debug messages now looks like in the branch 5.3, based on the tests done before with output posted in one of my comments above, so I haven't spent more time on understanding the changes introduced by proportional mode with the use of q_index field.

Now it prints:

```
 1(5011) WARNING: <script>: Route order:
 1(5011) INFO: <script>: ==== 1: sip:101 at 9.9.9.9
 1(5011) INFO: <script>: ==== 2: sip:101 at 8.8.8.8
 1(5011) INFO: <script>: ==== 3: sip:101 at 7.7.7.7
 1(5011) INFO: <script>: ==== 4: sip:101 at 6.6.6.6
 1(5011) INFO: <script>: ==== 5: sip:101 at 5.5.5.5
 1(5011) INFO: <script>: ==== 6: sip:101 at 4.4.4.4
 1(5011) INFO: <script>: ==== 7: sip:101 at 3.3.3.3
 1(5011) INFO: <script>: ==== 8: sip:101 at 2.2.2.2
 1(5011) INFO: <script>: ==== 9: sip:101 at 1.1.1.1
 1(5011) INFO: <script>: ==== 10: sip:101 at 127.0.0.1
```

The patch is only in master branch for now.

@cleung-tpn -- if you can test and confirm works as you expect, then I will backport to branch 5.3.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2449#issuecomment-729673535
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kamailio.org/pipermail/sr-dev/attachments/20201118/4581cc67/attachment.htm>