fix Stick package

axiaobiao · Feb 14, 2018 · d14b713 · d14b713
1 parent a13bc57
commit d14b713
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 9 deletions.
diff --git a/WebBench/test.sh b/WebBench/test.sh
@@ -1 +1 @@
-./bin/webbench -t 60 -c 1000 -2 --get http://127.0.0.1:8888/hello
+./bin/webbench -t 60 -c 1000 -2 --get http://127.0.0.1:80/hello
diff --git a/WebServer/HttpData.cpp b/WebServer/HttpData.cpp
@@ -70,7 +70,7 @@ HttpData::HttpData(EventLoop *loop, int connfd):
 
 void HttpData::reset()
 {
- inBuffer_.clear();
+ //inBuffer_.clear();
  fileName_.clear();
  path_.clear();
  nowReadPos_ = 0;
@@ -222,13 +222,11 @@ void HttpData::handleRead()
  if (!error_ && state_ == STATE_FINISH)
  {
 
- if (keepAlive_ && connectionState_ == H_CONNECTED)
+ if ((keepAlive_ || inBuffer_.size() > 0) && connectionState_ == H_CONNECTED)
  {
  this->reset();
  events_ |= EPOLLIN;
  }
- else
- return;
  }
  else if (!error_ && connectionState_ != H_DISCONNECTED)
  events_ |= EPOLLIN;
@@ -398,7 +396,8 @@ HeaderState HttpData::parseHeaders()
  int key_start = -1, key_end = -1, value_start = -1, value_end = -1;
  int now_read_line_begin = 0;
  bool notFinish = true;
- for (size_t i = 0; i < str.size() && notFinish; ++i)
+ size_t i = 0;
+ for (; i < str.size() && notFinish; ++i)
  {
  switch(hState_)
  {
@@ -501,7 +500,7 @@ HeaderState HttpData::parseHeaders()
  }
  if (hState_ == H_END_LF)
  {
- str = str.substr(now_read_line_begin);
+ str = str.substr(i);
  return PARSE_HEADER_SUCCESS;
  }
  str = str.substr(now_read_line_begin);

diff --git a/连接的维护.md b/连接的维护.md
@@ -20,8 +20,8 @@
 
 * 对于server端，毫无疑问应该谨慎处理以上所有情况。具体说来:
 > * 出现各种关于连接的错误时，可以直接close()掉
-> * 短连接超时的请求，可以直接close()
-> * 长连接对方长时间没有请求(如果没有保活机制)，可以close()
+> * 短连接超时的请求，可以close()，也可以不关
+> * 长连接对方长时间没有请求(如果没有保活机制)，可以close()，也可以不关
 > * client发出Fin，server应当把消息发完，然后才可以close()
 > * 短连接正常结束，server可以close，也可以不close，大多数的实现是不close的(对HTTP1.1而言)