[resent - devel mailing list address was invalid]

For the moment I will just get it walking via headers list, with no shortcut in sip_msg_t as hdr_field_t*. The reply doesn't have many headers anyhow.

But, if I am not wrong, I think Reason header can occur many times. Or at least I saw it two times, one time like:

Reason: Q.850; ...

and in the same message, another one like:

Reason: SIP;...

Overall, I think we should review which shortcut pointers to headers makes sense to keep inside sip_msg_t. Some of them are very rare in sip messages, probably can be found easily by walking the list of headers when needed.

Cheers,
Daniel

On 30/05/14 17:52, Juha Heinanen wrote:
i'll move this to devel. acc module is not trivial one.

should parsing of reason header be added to parser and struct hdr_field*
reason to struct sip_mgs?

-- juha

--
Daniel-Constantin Mierla - http://www.asipto.com
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda


_______________________________________________
sr-dev mailing list
sr-dev@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to