[sr-dev] [kamailio/kamailio] async_http_client crash (#2286)

[Federico Cabiddu]

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 at 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 at 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 at lists.kamailio.org
>> https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kamailio.org/pipermail/sr-dev/attachments/20200418/669aead3/attachment-0002.html>