The code introduced by this PR does not seem to deal properly with some valid
cases. Detecting if the URI is enclosed in `<>` fails if there are header
parameters. For example:
```
From: <al...@x.com>;tag=abc\r\n
```
The `tag` is a header parameter, is not a URI parameter. I think it should
check the `uri` field from the struct tobody, not `body`.
Furthermore, even not conform with RFC, the parser was supposed cope with
headers terminated only by `\n` to be able to test with messages forged in text
files (e.g., during development of new features that are not supported by
existing client UA).
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3935#issuecomment-2298325299
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3935/c2298325...@github.com>
_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
