[reSIProcate] Bug in handleBadRequest

Byron Campen bcampen at estacado.net
Tue Mar 13 12:29:10 CDT 2007


	Yeah, I noticed this. I have sent Robert a patch that fixes this  
problem, so he can re-run the test. I don't know if we could be  
having a problem with the Content-Length being wrong though, since  
those errors are accompanied by a logging statement. If the Content- 
Length is malformed, we don't log, but it looks fine.

Best regards,
Byron Campen
> Byron,
>
>
>
> Looking at Roberts problem, I noticed the following – probably why  
> we can’t figure out why the 400.  I suspect the 400 is due to a bad  
> content length – but fixing this will help determine.
>
>
>
> Scott
>
>
>
>
>
> error->getReason should read badReg->getReason
>
>
>
> bool
>
> TransactionState::handleBadRequest(const resip::SipMessage& badReq,  
> TransactionController& controller)
>
> {
>
>    assert(badReq.isRequest() && badReq.method() != ACK);
>
>    try
>
>    {
>
>       SipMessage* error = Helper::makeResponse(badReq,400);
>
>       error->header(h_StatusLine).reason()+="(" + error->getReason 
> () + ")";
>
>       Tuple target(badReq.getSource());
>
>
>
>
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.resiprocate.org/pipermail/resiprocate-devel/attachments/20070313/12fab6fd/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 2423 bytes
Desc: not available
URL: <http://list.resiprocate.org/pipermail/resiprocate-devel/attachments/20070313/12fab6fd/attachment.bin>


More information about the resiprocate-devel mailing list