Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

add support for enabling HTTP pipelining #320

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tanner0101
Copy link
Member

  • Adds a new option supportPipelining to HTTPServer.start.
  • Fixes http server does not support pipelined requests #318
  • Adds a debug-only assertion when pipelining is detected in HTTPServerHandler.
  • Changes TLS test to vapor.codes since Amazon's TLS seems to be unreliable.

@@ -126,6 +128,9 @@ private final class HTTPServerHandler<R>: ChannelInboundHandler where R: HTTPSer

/// Optional server header.
private let serverHeader: String?

/// If true, we are waiting for a response to be sent.
var awaitingReponse: Bool
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
var awaitingReponse: Bool
private(set) var awaitingReponse: Bool

?

@t089
Copy link

t089 commented Dec 13, 2018

Super, looks good to me. I've just opened #321 to add a test case for pipelining.

Base automatically changed from master to main March 12, 2021 02:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants