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

Bugfix/use write head instead of implicit header #170

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Bugfix/use write head instead of implicit header #170

wants to merge 17 commits into from

Commits on Jul 30, 2022

  1. Add failing tests for http2

    @maritz @Icehunter
    maritz authored and Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    f160f44 View commit details
    Browse the repository at this point in the history

  2. Add support for http2 

    The change just removes the usage of undocumented http API and instead uses the proper writeHead method
    @maritz @Icehunter
    maritz authored and Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    2c8ed82 View commit details
    Browse the repository at this point in the history

  3. Remove arrow function in tests

    @maritz @Icehunter
    maritz authored and Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    8eae006 View commit details
    Browse the repository at this point in the history

  4. Conditionally run http2 tests if http2 is available

    @maritz @Icehunter
    maritz authored and Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    8c68bc1 View commit details
    Browse the repository at this point in the history

  5. Fix port in tests to be assigned automatically

    @maritz @Icehunter
    maritz authored and Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    5122916 View commit details
    Browse the repository at this point in the history

  6. Change http2 test usage to describe.skip if no http2 available

    @maritz @Icehunter
    maritz authored and Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    7b7f506 View commit details
    Browse the repository at this point in the history

  7. Fix closing the http2 connections to prevent possible exceptions

    @maritz @Icehunter
    maritz authored and Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    8ba8100 View commit details
    Browse the repository at this point in the history

  8. Fix closing the request first, then the client, then the server

    @maritz @Icehunter
    maritz authored and Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    8ea806d View commit details
    Browse the repository at this point in the history

  9. Fix closing for v8.x and v9.x

    @maritz @Icehunter
    maritz authored and Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    a562056 View commit details
    Browse the repository at this point in the history

  10. fix tests not draining data for http2 requests, resulting in timeouts… 

    … in node v10.4 onwards
    @maritz @Icehunter
    maritz authored and Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    feddec1 View commit details
    Browse the repository at this point in the history

  11. fix: 🐛 assert.equal error

    @Icehunter
    Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    5124eb0 View commit details
    Browse the repository at this point in the history

  12. fix: 🐛 remove console.log's and timeout, let build fail

    @Icehunter
    Icehunter committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    094316e View commit details
    Browse the repository at this point in the history

  13. Apply suggestions from code review 

    Co-authored-by: Lam Wei Li <lam_wei_li@hotmail.com>
    @Icehunter @lamweili
    Icehunter and lamweili committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    d85e6e8 View commit details
    Browse the repository at this point in the history

Commits on Aug 2, 2022

  1. fixed lint

    @Icehunter @lamweili
    Icehunter authored and lamweili committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    d3016e2 View commit details
    Browse the repository at this point in the history

  2. fix: an issue where test hangs when assertion fails in http2 as http2… 

    …server is not closed
    @lamweili
    lamweili committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    ebae4e9 View commit details
    Browse the repository at this point in the history

  3. refactor: use http2.constants instead of hard-coded strings in http2 … 

    …test
    @lamweili
    lamweili committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    78cac20 View commit details
    Browse the repository at this point in the history

  4. Node.js 0.8 compatible

    @lamweili
    lamweili committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    4c1d27b View commit details
    Browse the repository at this point in the history