[sr-dev] Suggesting fix for t_request and pua_dialoginfo

[Jasmin Schnatterbeck]

Hello List,

some months ago I reported a problem with pua_dialoginfo:
Avps declared in request route are not available anymore in branch
route, if pua_dialoginfo module is loaded.

I do not know whether there are some experiences with this problem or
not, so I looked for the reason.

pua_dialoginfo uses pua for sending PUBLISH request
pua uses tm t_request for sending PUBLISH request

t_request() is located in uac.c (tm):
- in line 411 avp list is reset
- in line 276 build_cell() is called:
  build_cell() is located in h_table.c, where lines 305-323 delete avps

If I comment out the mentioned lines, which delete avps, then avps set
in request route are also available in branch route.
But I do not know, what is also affected by this change.

So would it be reasonable, to add a parameter to build_cell() for not
changing the AVP list?

Best Regards
Jasmin