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
No open projects
Vapor 3
  
Awaiting triage
Development

Successfully merging this pull request may close these issues.

None yet

2 participants