18 Apr
2020
18 Apr
'20
3:25 a.m.
Sorry the mail was sent before ending by accident.
Thanks for looking into it and fixing it Daniel. For sure it should be
backported.
Cheers,
Federico
On Sat, Apr 18, 2020 at 9:23 AM Federico Cabiddu <federico.cabiddu(a)gmail.com>
wrote:
Hi Daniel,
you are absolutely right and that part of code is for sure wrong. Those
lines have been there since the beginning and for sure due to my lack of
understanding of transactions's internals and avps (don't remember why
sincerely).
For the records, I've been using mixed suspended/not suspended from the
beginning but I've always used separate return routes for suspended and not
suspended queries. That could explain why I never had it.
Thanks for looking into it
On Sat, Apr 18, 2020 at 3:13 AM Alex Balashov <notifications(a)github.com>
wrote:
@miconda <https://github.com/miconda> To
answer a few other questions:
1.
I always set $http_req(suspend) = 1 on async HTTP client transactions;
2.
There are no event_routes anywhere;
3.
The flow is complex, though. After this async HTTP query, there is
another *synchronous* HTTP query using the normal http_client module
+ mqueue + rtimer -- it was implemented this way because of some
problems with transaction-persistent variables (ironically) when suspending
the same TM transaction twice, and a desire for more granular control over
the request pipelines given the high and somewhat unpredictable latency of
what's being queried.
Nevertheless, all this takes place long after the crash point.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<https://github.com/kamailio/kamailio/issues/2286#issuecomment-615530081>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABO7UZJSBJ6RUK2BJR3UONLRND5DNANCNFSM4MIVPAIQ>
.
_______________________________________________
Kamailio (SER) - Development Mailing List
sr-dev(a)lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev