swift-server / http

:warning: Historical HTTP API - please use https://github.com/swift-server/async-http-client instead
Apache License 2.0
703 stars 48 forks source link

Move PoCSocket code into its own directory/target #65

Closed carlbrown closed 7 years ago

carlbrown commented 7 years ago

I'm going to throw this out here, although we don't have to do this.

Based on some of the feedback I've seen since the 0.1.0 announcement, there's some confusion about the difference between the API and the Proof of Concept implementation. This might make that more clear (although it could totally backfire - it wouldn't be the first time).

Thoughts?

tanner0101 commented 7 years ago

I'm all for this 👍

seabaylea commented 7 years ago

This would break the API is make it non-intuative for a user trying to start a server, so I don't think its a change we should make.

The underlying implementation is a "Proof of Concept" - i.e. the design/architecture/approach is far from fixed. In fact if someone wanted to propose a full drop-in replacement then thats something that would definitely be interesting.