First, this is not a fix, the http traffic was never supposed to be fully 
handled as sip traffic. Kamailio is not a full HTTP server/proxy. The 
workaround for creating a fake sip message was done for local processing, 
mainly to be able to use authentication and authorization functions as well as 
config variables.

The async framework is purely designed for received SIP traffic, creating a SIP 
transaction that stays in memory, runs various timer callbacks and sip-specs 
specific tasks.

You patch creates an internal sip message that does not even follow the strict 
requirements for a sip requests, like tags, via branch, ... CSeq has an 
hardcoded method even there can be different ones even in http. Also it breaks 
the $hu. These are just a few remarks upon a quick look. This PR is not 
appropriate to merge.

If you need to do some async processing in Kamailio, you have to do it with sip 
traffic. You can generate from cli with sipexer or sipsak tools.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3723#issuecomment-1906002120
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/pull/3723/c1906002...@github.com>
_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org

Reply via email to