使用Http.Core与Http.Client 4交谈获取ConnectionClosedException?

时间:2014-08-13 09:01:06

标签: java sockets java-7 apache-httpclient-4.x apache-httpcomponents

我正在努力使用http.core&客户4.3。一般来说,它运作良好,处理起来非常愉快。但是,我在其中一个转让中收到了ConnectionClosedException,我看不出原因。就我所知,其他人工作得很好。

一切都以非常直接的方式遵循这些例子。如果没有,那就尽可能地重写,以便摆脱这种情况。

  1. 有2台服务器,都运行相同的代码[A& B]
  2. HttpClient向B
  3. 发送请求“AX”(POST)
  4. B HttpService收到“AX”帖子,处理
  5. B HttpClient在不同的端口向A发送回复“BR”(POST)
    1. 稍后这应该在A的连接关闭或尽可能接近
    2. 之后发生
    3. 现在代码实际上并不关心
  6. A收到B的回复(在不同的主题上)并做了一些事情
  7. 在问题场景中, A 作为服务器运行,B正在发送POST。对不起,它并不总是很清楚,因为在一个事务中,双方最终都运行服务器和客户端代码。

    1. A 将POST发送至 B:8080 。内联回复正确,一切正常。
    2. POST连接 B:8080 正确关闭
    3. B A 发送新的POST(如ACK)(例如... B:53991 => A :9000
      1. A 处理所有内容。没问题
      2. A rasies ConnectionClosedException
    4. 由于我不知道为什么会发生这种情况,我试图把我认为相关的一切都放在那里。我现在唯一的想法就是确保我添加/更改连接控制头有一些东西,但是我看不出它会如何影响任何东西。

      来自机器“A”的

      堆栈跟踪,当来自B的回复

      org.apache.http.ConnectionClosedException: Client closed connection
          at org.apache.http.impl.io.DefaultHttpRequestParser.parseHead(DefaultHttpRequestParser.java:133)
          at org.apache.http.impl.io.DefaultHttpRequestParser.parseHead(DefaultHttpRequestParser.java:54)
          at org.apache.http.impl.io.AbstractMessageParser.parse(AbstractMessageParser.java:260)
          at org.apache.http.impl.DefaultBHttpServerConnection.receiveRequestHeader(DefaultBHttpServerConnection.java:131)
          at org.apache.http.protocol.HttpService.handleRequest(HttpService.java:307)
          at com.me.HttpRequestHandlerThread.processConnection(HttpRequestHandlerThread.java:45)
          at com.me.net.http.HttpRequestHandlerThread.run(HttpRequestHandlerThread.java:70)
      com.me.ExceptionHolder: Client closed connection
          at com.me.log.Log.logIdiocy(Log.java:77)
          at com.me.log.Log.error(Log.java:54)
          at com.me.net.http.HttpRequestHandlerThread.run(HttpRequestHandlerThread.java:72)
      Caused by: org.apache.http.ConnectionClosedException: Client closed connection
          at org.apache.http.impl.io.DefaultHttpRequestParser.parseHead(DefaultHttpRequestParser.java:133)
          at org.apache.http.impl.io.DefaultHttpRequestParser.parseHead(DefaultHttpRequestParser.java:54)
          at org.apache.http.impl.io.AbstractMessageParser.parse(AbstractMessageParser.java:260)
          at org.apache.http.impl.DefaultBHttpServerConnection.receiveRequestHeader(DefaultBHttpServerConnection.java:131)
          at org.apache.http.protocol.HttpService.handleRequest(HttpService.java:307)
          at com.me.net.http.HttpRequestHandlerThread.processConnection(HttpRequestHandlerThread.java:45)
          at com.me.net.http.HttpRequestHandlerThread.run(HttpRequestHandlerThread.java:70)
      

      这是在 B 上运行的代码,在此方案中为“客户端”。它正在尝试发送回复,确认已正确接收来自 A 的第一个POST。实际上没有太多要传输,响应应该只是HTTP 200:

      try (CloseableHttpClient client = HttpClients.createDefault()) {
          final HttpPost post = new HttpPost(url);
          post.setHeaders(/* create application specific headers */);
      
          ByteArrayEntity entity = new ByteArrayEntity(IOUtils.toByteArray(myStream));
          post.setEntity(entity);
      
          ResponseHandler<Void> responseHandler = new ResponseHandler<Void>() {
              @Override
              public Void handleResponse(HttpResponse response) throws ClientProtocolException, IOException {
                  StatusLine status = response.getStatusLine();
                  if (!NetUtil.isValidResponseCode(response)) {
                      throw new ClientProtocolException("Unexpected Error! Oops");
                  }
                  // consume the response, if there is one, so the connection will close properly
                  EntityUtils.consumeQuietly(response.getEntity());
                  return null;
              }
          };
      
          try {
              client.execute(post, responseHandler);
          } catch (ClientProtocolException ex) { 
              // logic to queue a resend for 10 minutes later.  not triggered
              throw ex;
          }
      }
      

      A 上:这称为异步,因为响应不会通过同一http连接进入。

      主请求处理程序做了很多工作,但令人惊讶的是,在处理程序/服务器端实际控制HTTP的代码很少。伟大的图书馆...我不知何故误用了。这是实际的处理程序,一切都简化了一点,删除了验证等等。

      public class AsyncReceiverHandler implements HttpRequestHandler {
      
          @Override
          public void handle(HttpRequest request, HttpResponse response, HttpContext context) throws HttpException, IOException {
              // error if not post, other logic.  not touching http.  no errors
              DefaultBHttpServerConnection connection = (DefaultBHttpServerConnection) context.getAttribute("connection");
              Package pkg = NetUtil.createPackageFrom(connection); // just reads sender ip/port
              NetUtil.copyHttpHeaders(request, pkg); 
              try {
                  switch (recieive(request, pkg)) {
                      case EH_OK:
                          response.setStatusCode(HttpStatus.SC_OK);
                          break;
                      case OHNOES_BAD_INPUT:
                          response.setStatusCode(HttpStatus.SC_BAD_REQUEST);
                          response.setEntity(new StringEntity("No MDN entity found in request body"));
                      // bunch of other cases, but are not triggered.  xfer was a-ok
                  }
              } catch (Exception ex) {
                  //log
              } 
          }
          private MyStatus receiveMdn(HttpRequest request, Package pkg) throws Exceptions..., IOException {
              // validate request, get entity, make package, no issues
              HttpEntity resEntity = ((HttpEntityEnclosingRequest) request).getEntity();
      
              try {
                  byte[] data = EntityUtils.toByteArray(resEntity);
                  // package processing logic, validation, fairly quick, no errors thrown
              } catch (Exceptions... ex) {
                  throw ExceptionHolder(ex);
              }
          }
      }
      

      这是请求处理程序线程。这个和服务器几乎是从样本中逐字记录的。服务处理程序只启动服务,accept()是套接字。当它获得一个时,它会创建一个新副本,并调用start():

      public HttpRequestHandlerThread(final HttpService httpService, final HttpServerConnection conn, HttpReceiverModule ownerModule) {
              super();
              this.httpService = httpService;
              this.conn = (DefaultBHttpServerConnection) conn;
          }
          private void processConnection() throws IOException, HttpException {
              while (!Thread.interrupted() && this.conn.isOpen()) {
                  /* have the service create a handler and pass it the processed request/response/context */
                  HttpContext context = new BasicHttpContext(null);
                  this.httpService.handleRequest(this.conn, context);
              }
          }
          @Override
          public void run() {
              // just runs the main logic and reports exceptions. 
              try {
                  processConnection();
              } catch (ConnectionClosedException ignored) {
                  // logs error here (and others). 
              } finally {
                  try { this.conn.shutdown(); } catch (IOException ignored) {}
              }
          }
      }
      

1 个答案:

答案 0 :(得分:0)

嗯,现在这看起来很愚蠢,而且非常明显。我暂时忽略了这个问题,转而研究其他事情,并将答案从潜意识中冒出来,就像他们愿意一样。

我把这个标题添加回来,一切都清理完了:

post.setHeader("Connection", "close, TE")

以某种方式设置Connection标题的行被删除了,可能是我不小心。它们中的很多都被设置了,它仍然存在,只是在这个代码路径中出错了。基本上,服务器希望此连接立即关闭,但标头恢复为默认keep-alive。由于客户端在完成连接后立即关闭连接,这对于服务器来说是令人惊讶的,否则被告知,并且正确地说:D在反向路径中一切都很好。

由于我刚刚更改旧堆栈以使用HttpComponents我没有看到标题等等,我只是假设我使用它错了。旧堆栈并不介意。