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

HTTP/2 support #2

Open
g4s8 opened this issue Apr 2, 2018 · 25 comments
Open

HTTP/2 support #2

g4s8 opened this issue Apr 2, 2018 · 25 comments
Labels

Comments

@g4s8
Copy link

g4s8 commented Apr 2, 2018

Does cactoos-http designed to support HTTP/2 protocol? If yes how does it share sockets (or clients) across requests?

@yegor256
Copy link
Owner

yegor256 commented Apr 3, 2018

@g4s8 that's a good question. Yes, we should support HTTP/2. Let's think how to do that right.

@g4s8
Copy link
Author

g4s8 commented Apr 3, 2018

@yegor256 looks like it can be HtWire implementation, e.g.:

try (final HtWire http2 = new Http2Wire("example.com")) {
  new HtResponse(
      http2,
      new JoinedText(
        "\n\r"
        "GET / HTTP/2",
        "Host: www.example.com"
      )
    );
}

also it can be helpful to see how it's implemented in other clients: https://github.com/square/okhttp/blob/master/okhttp/src/main/java/okhttp3/internal/http2/Http2Connection.java

@llorllale
Copy link
Collaborator

@g4s8 @yegor256 I have no idea how HTTP/2 works

@g4s8 can you implement a small, initial step in the right direction?

@g4s8
Copy link
Author

g4s8 commented May 15, 2018

@llorllale sure I'll have a look later

@llorllale
Copy link
Collaborator

@g4s8 you want this task?

@g4s8
Copy link
Author

g4s8 commented May 15, 2018

@llorllale yes, please assign me

@llorllale
Copy link
Collaborator

@0crat assign @g4s8

@0crat 0crat added the scope label May 15, 2018
@0crat
Copy link
Collaborator

0crat commented May 15, 2018

@0crat assign @g4s8 (here)

@llorllale The job #2 assigned to @g4s8/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this @g4s8/z is not a member of this project yet, but they can request to join, as §1 explains

@0crat
Copy link
Collaborator

0crat commented May 15, 2018

Bug was reported, see §29: +15 point(s) just awarded to @g4s8/z

@0crat
Copy link
Collaborator

0crat commented May 15, 2018

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @llorllale/z

@0crat
Copy link
Collaborator

0crat commented May 15, 2018

It is strongly discouraged to assign jobs to their creators, see §19: -15 point(s) just awarded to @llorllale/z

@llorllale
Copy link
Collaborator

@0crat refuse

@0crat
Copy link
Collaborator

0crat commented Jan 3, 2019

@0crat refuse (here)

@llorllale The user @g4s8/z resigned from #2, please stop working. Reason for job resignation: Order was cancelled

@0crat
Copy link
Collaborator

0crat commented Jan 13, 2019

The job #2 assigned to @ilyakharlamov/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@ilyakharlamov
Copy link
Contributor

ilyakharlamov commented Jan 15, 2019

@0crat refuse
ALPN is required for HTTP/2, but it isn’t available in Java until JDK 9.
OkHttp and other clients also require either Java 9 or hacked boot classes from Java 9.
Since this project is Java 8 (see pom.xml), sane implementation is not possible without JDK switching.

@0crat
Copy link
Collaborator

0crat commented Jan 15, 2019

Are you speaking to me or about me here; you must always start your message with my name if you want to address it to me, see §1

@ilyakharlamov
Copy link
Contributor

@0crat refuse

@0crat
Copy link
Collaborator

0crat commented Jan 15, 2019

@0crat refuse (here)

@ilyakharlamov The user @ilyakharlamov/z resigned from #2, please stop working. Reason for job resignation: Order was cancelled

@0crat
Copy link
Collaborator

0crat commented Jan 15, 2019

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @ilyakharlamov/z

@0crat
Copy link
Collaborator

0crat commented Jan 19, 2019

The job #2 assigned to @borysfan/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@llorllale
Copy link
Collaborator

@0crat refuse

@0crat
Copy link
Collaborator

0crat commented Mar 18, 2019

@0crat refuse (here)

@llorllale The user @borysfan/z resigned from #2, please stop working. Reason for job resignation: Order was cancelled

@0crat
Copy link
Collaborator

0crat commented Mar 18, 2019

The job #2 assigned to @mazdack/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@0crat
Copy link
Collaborator

0crat commented Sep 11, 2019

The architect of the project has changed; @llorllale/z is not at this role anymore; @paulodamaso/z is the architect now

@0crat
Copy link
Collaborator

0crat commented Jul 5, 2020

The architect of the project has changed; @paulodamaso/z is not at this role anymore; @victornoel/z is the architect now

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

No branches or pull requests

5 participants