roxen.lists.roxen.general

Subject Author Date
100-continue expected in Roxen 5.0 <thomas.gusenleitner < 10-01-2008
Hi,

i've noticed there is a "\r\n" missing in  protocols\http.pike method
read_to_receive()

-- original --
// Tell the client that it can start sending some more data
void ready_to_receive()
{
  // FIXME: Only send once?
  if (clientprot == "HTTP/1.1" && request_headers->expect &&
      (request_headers->expect ==  "100-continue" ||
       has_value(request_headers->expect, "100-continue" )) &&
      !my_fd_busy)
    my_fd->write("HTTP/1.1 100 Continue\r\n\");
}


-- correct --
// Tell the client that it can start sending some more data
void ready_to_receive()
{
  // FIXME: Only send once?
  if (clientprot == "HTTP/1.1" && request_headers->expect &&
      (request_headers->expect ==  "100-continue" ||
       has_value(request_headers->expect, "100-continue" )) &&
      !my_fd_busy)
    my_fd->write("HTTP/1.1 100 Continue\r\n\r\n");
}



greetings
thomas