Mercurial Hosting > luan

annotate src/org/eclipse/jetty/server/BlockingHttpConnection.java @ 934:fe461f7cfc8e

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
simplify AsyncContinuation
author Franklin Schmidt <fschmidt@gmail.com>
date Sun, 09 Oct 2016 21:03:00 -0600
parents ad495e897c32
children 866f2e801618
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
1 //
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
2 // ========================================================================
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
3 // Copyright (c) 1995-2014 Mort Bay Consulting Pty. Ltd.
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
4 // ------------------------------------------------------------------------
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
5 // All rights reserved. This program and the accompanying materials
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
6 // are made available under the terms of the Eclipse Public License v1.0
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
7 // and Apache License v2.0 which accompanies this distribution.
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
8 //
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
9 // The Eclipse Public License is available at
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
10 // http://www.eclipse.org/legal/epl-v10.html
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
11 //
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
12 // The Apache License v2.0 is available at
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
13 // http://www.opensource.org/licenses/apache2.0.php
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
14 //
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
15 // You may elect to redistribute this code under either of these licenses.
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
16 // ========================================================================
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
17 //
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
18
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
19 package org.eclipse.jetty.server;
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
20
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
21 import java.io.IOException;
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
22
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
23 import org.eclipse.jetty.http.Generator;
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
24 import org.eclipse.jetty.http.HttpException;
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
25 import org.eclipse.jetty.http.HttpStatus;
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
26 import org.eclipse.jetty.http.Parser;
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
27 import org.eclipse.jetty.io.Connection;
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
28 import org.eclipse.jetty.io.EndPoint;
820
8e9db0bbf4f9 remove org.eclipse.jetty.util.log and upgrade slf4j
Franklin Schmidt <fschmidt@gmail.com>
parents: 802
diff changeset
29 import org.slf4j.Logger;
8e9db0bbf4f9 remove org.eclipse.jetty.util.log and upgrade slf4j
Franklin Schmidt <fschmidt@gmail.com>
parents: 802
diff changeset
30 import org.slf4j.LoggerFactory;
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
31
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
32
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
33 /* ------------------------------------------------------------ */
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
34 /** Blocking Server HTTP Connection
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
35 */
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
36 public class BlockingHttpConnection extends AbstractHttpConnection
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
37 {
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
38 private static final Logger LOG = LoggerFactory.getLogger(BlockingHttpConnection.class);
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
39
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
40 public BlockingHttpConnection(Connector connector, EndPoint endpoint, Server server)
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
41 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
42 super(connector,endpoint,server);
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
43 }
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
44
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
45 public BlockingHttpConnection(Connector connector, EndPoint endpoint, Server server, Parser parser, Generator generator, Request request)
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
46 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
47 super(connector,endpoint,server,parser,generator,request);
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
48 }
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
49
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
50 @Override
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
51 protected void handleRequest() throws IOException
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
52 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
53 super.handleRequest();
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
54 }
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
55
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
56 public Connection handle() throws IOException
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
57 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
58 Connection connection = this;
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
59
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
60 try
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
61 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
62 setCurrentConnection(this);
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
63
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
64 // do while the endpoint is open
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
65 // AND the connection has not changed
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
66 while (_endp.isOpen() && connection==this)
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
67 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
68 try
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
69 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
70 // If we are not ended then parse available
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
71 if (!_parser.isComplete() && !_endp.isInputShutdown())
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
72 _parser.parseAvailable();
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
73
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
74 // Do we have more generating to do?
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
75 // Loop here because some writes may take multiple steps and
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
76 // we need to flush them all before potentially blocking in the
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
77 // next loop.
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
78 if (_generator.isCommitted() && !_generator.isComplete() && !_endp.isOutputShutdown())
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
79 _generator.flushBuffer();
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
80
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
81 // Flush buffers
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
82 _endp.flush();
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
83 }
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
84 catch (HttpException e)
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
85 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
86 if (LOG.isDebugEnabled())
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
87 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
88 LOG.debug("uri="+_uri);
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
89 LOG.debug("fields="+_requestFields);
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
90 LOG.debug("",e);
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
91 }
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
92 _generator.sendError(e.getStatus(), e.getReason(), null, true);
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
93 _parser.reset();
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
94 _endp.shutdownOutput();
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
95 }
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
96 finally
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
97 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
98 // Is this request/response round complete and are fully flushed?
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
99 if (_parser.isComplete() && _generator.isComplete())
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
100 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
101 // Reset the parser/generator
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
102 reset();
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
103
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
104 // look for a switched connection instance?
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
105 if (_response.getStatus()==HttpStatus.SWITCHING_PROTOCOLS_101)
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
106 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
107 Connection switched=(Connection)_request.getAttribute("org.eclipse.jetty.io.Connection");
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
108 if (switched!=null)
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
109 connection=switched;
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
110 }
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
111
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
112 // TODO Is this required?
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
113 if (!_generator.isPersistent() && !_endp.isOutputShutdown())
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
114 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
115 LOG.warn("Safety net oshut!!! Please open a bugzilla");
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
116 _endp.shutdownOutput();
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
117 }
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
118 }
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
119
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
120 // If we don't have a committed response and we are not suspended
934
fe461f7cfc8e simplify AsyncContinuation
Franklin Schmidt <fschmidt@gmail.com>
parents: 874
diff changeset
121 if (_endp.isInputShutdown() && _generator.isIdle())
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
122 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
123 // then no more can happen, so close.
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
124 _endp.close();
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
125 }
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
126 }
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
127 }
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
128
874
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
129 return connection;
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
130 }
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
131 finally
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
132 {
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
133 setCurrentConnection(null);
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
134 _parser.returnBuffers();
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
135 _generator.returnBuffers();
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
136 }
Franklin Schmidt <fschmidt@gmail.com>
parents: 820
diff changeset
137 }
802
3428c60d7cfc replace jetty jars with source
Franklin Schmidt <fschmidt@gmail.com>
parents:
diff changeset
138 }