Iirc, the attrs set to empty string was an workaround to be sure the AVPs with
attrs value do not get shifted (associated) with a wrong destination. The AVP
cannot be null, so if among selected destinations some of them were with attrs
and some without, then the number of AVPs with attrs was lower than the number
of destinations.
Maybe is no longer needed with xavps, as the fields related to a destination
are kept in an XAVP sublist. But that has to be reviewed to see if there is any
side effect. Might be better to split this chance in its own commit.
--
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/pull/1877#issuecomment-470958331
_______________________________________________
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
