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.

Sign up for GitHub

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

Jump to bottom

Fixes #6603 - HTTP/2 max local stream count exceeded #6639

Merged
merged 13 commits into from Aug 30, 2021
Merged

Fixes #6603 - HTTP/2 max local stream count exceeded #6639

merged 13 commits into from Aug 30, 2021

Commits on Aug 18, 2021

  1. Fixes #6603 - HTTP/2 max local stream count exceeded 

    Made MAX_CONCURRENT_STREAMS setting work on a per-connection basis.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    @sbordet
    sbordet committed Aug 18, 2021
    Copy the full SHA
    6ea6bda View commit details
    Browse the repository at this point in the history

Commits on Aug 19, 2021

  1. Issue #6603 - HTTP/2 max local stream count exceeded 

    Updates after review.
Updated the maxMultiplex mechanism to always work on Pool.Entry, rather than on Pool.

Updated Pool javadocs.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    @sbordet
    sbordet committed Aug 19, 2021
    Copy the full SHA
    cb102d9 View commit details
    Browse the repository at this point in the history

Commits on Aug 20, 2021

  1. Thought bubble on abstracting out multiplexing from the pool

    @gregw
    gregw committed Aug 20, 2021
    Copy the full SHA
    a1297ca View commit details
    Browse the repository at this point in the history

  2. revert to the original naming

    @gregw
    gregw committed Aug 20, 2021
    Copy the full SHA
    e4d1fb9 View commit details
    Browse the repository at this point in the history

Commits on Aug 26, 2021

  1. Review suggestion 

    Use interface
    @gregw
    gregw committed Aug 26, 2021
    Copy the full SHA
    5834e5f View commit details
    Browse the repository at this point in the history

Commits on Aug 27, 2021

  1. Work In Progress

    @gregw
    gregw committed Aug 27, 2021
    Copy the full SHA
    7b78b90 View commit details
    Browse the repository at this point in the history

  2. Work In Progress

    @gregw
    gregw committed Aug 27, 2021
    Copy the full SHA
    c3a19a6 View commit details
    Browse the repository at this point in the history

  3. Fixes #6603 - HTTP/2 max local stream count exceeded 

    Made MAX_CONCURRENT_STREAMS setting work on a per-connection basis.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    @sbordet
    sbordet committed Aug 27, 2021
    Copy the full SHA
    216616a View commit details
    Browse the repository at this point in the history

  4. Issue #6603 - HTTP/2 max local stream count exceeded 

    Updates after review.
Updated the maxMultiplex mechanism to always work on Pool.Entry, rather than on Pool.

Updated Pool javadocs.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    @sbordet
    sbordet committed Aug 27, 2021
    Copy the full SHA
    9caf6fa View commit details
    Browse the repository at this point in the history

  5. Issue #6603 - HTTP/2 max local stream count exceeded 

    Fixed javadocs.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    @sbordet
    sbordet committed Aug 27, 2021
    Copy the full SHA
    e27f192 View commit details
    Browse the repository at this point in the history

  6. Issue #6603 - HTTP/2 max local stream count exceeded 

    Incorporated changes from #6648.

Now the maxMultiple value is pulled from its primary value,
i.e. HTTP2Session.maxLocalStreams, rather than being set
in multiple places.

Deprecated usages of maxMultiplex and maxUsageCount.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    @sbordet
    sbordet committed Aug 27, 2021
    Copy the full SHA
    7733d0f View commit details
    Browse the repository at this point in the history

Commits on Aug 30, 2021

  1. Issue #6603 - HTTP/2 max local stream count exceeded 

    Added comment as suggested in review.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    @sbordet
    sbordet committed Aug 30, 2021
    Copy the full SHA
    97f4ef3 View commit details
    Browse the repository at this point in the history

  2. Issue #6603 - HTTP/2 max local stream count exceeded 

    Made newEntry() private to avoid creation of more than 2 Entry subclasses.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
    @sbordet
    sbordet committed Aug 30, 2021
    Copy the full SHA
    aa3e7de View commit details
    Browse the repository at this point in the history