Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OkHttpServer #2916

Closed
Tracked by #2903
swankjesse opened this issue Oct 16, 2016 · 7 comments
Closed
Tracked by #2903

OkHttpServer #2916

swankjesse opened this issue Oct 16, 2016 · 7 comments
Labels
enhancement Feature not a bug
Milestone

Comments

@swankjesse
Copy link
Collaborator

We’re exploring building this. We could reuse the same request, response, and interceptor types.

@swankjesse swankjesse added this to the 4.0 milestone Oct 16, 2016
@artem-zinnatullin
Copy link
Contributor

  1. In WIP PR https://github.com/square/okhttp/pull/2912/files: <artifactId>okwebserver</artifactId>, not okhttpserver. Can you please clarify naming?
  2. Will OkHttpServer result in deletion of MockWebServer or it'll be used under the hood of MockWebServer?

@swankjesse
Copy link
Collaborator Author

TBD.

@artem-zinnatullin
Copy link
Contributor

Yeah, I understand, those questions were not like "ANSWER THEM NOW OR I'LL UNSTAR OKHTTP ON GITHUB!!" but more "Please think about that and share some thoughts when you'll have some opinion".

No pressure! 🐱

@JakeWharton
Copy link
Collaborator

JakeWharton commented Apr 15, 2017

MWS can't test SSE/long-polling because body is Buffer. This should support.

@yschimke yschimke added the enhancement Feature not a bug label Dec 30, 2017
@yschimke
Copy link
Collaborator

@swankjesse would you build this and https://github.com/cashapp/misk ?

@swankjesse
Copy link
Collaborator Author

Misk is a server framework. It currently uses Jetty though if we wanted to make the investment we could swap out Jetty for something we build ourselves.

@yschimke
Copy link
Collaborator

I'm more wondering whether we can close this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature not a bug
Projects
None yet
Development

No branches or pull requests

4 participants