PR#12: Update to 8.1.1 (close RHBZ#2216591) - rpms/llhttp

rpms / llhttp

#12 Update to 8.1.1 (close RHBZ#2216591)

Merged 23 days ago by music. Opened 2 months ago by music.

rpms/ music/llhttp v8.1.1 into rawhide

Update to 8.1.1 (close RHBZ#2216591)

Benjamin A. Beasley • 24 days ago

99ff5de

.gitignore

file modified

		`@@ -8,3 +8,5 @@`
		`/llhttp-8.0.0-nm-dev.tgz`
		`/llhttp-8.1.0.tar.gz`
		`/llhttp-8.1.0-nm-dev.tgz`
		`+ /llhttp-8.1.1.tar.gz`
		`+ /llhttp-8.1.1-nm-dev.tgz`

230.patch

file added

+24

		`@@ -0,0 +1,24 @@`
		`+ From 553ae7d1bda0b97a43af6b57ea2fd7ce04b6630d Mon Sep 17 00:00:00 2001`
		`+ From: "Benjamin A. Beasley" <code@musicinmybrain.net>`
		`+ Date: Tue, 27 Jun 2023 18:27:17 -0400`
		`+ Subject: [PATCH] Do not assume a particular sed implementation`
		`+`
		`+ Make the release target in the Makefile more portable.`
		`+ ---`
		`+ Makefile \| 3 +--`
		`+ 1 file changed, 1 insertion(+), 2 deletions(-)`
		`+`
		`+ diff --git a/Makefile b/Makefile`
		`+ index 5b941b3..d9c6d35 100644`
		`+ --- a/Makefile`
		`+ +++ b/Makefile`
		`+ @@ -52,8 +52,7 @@ release: clean generate`
		`+ cp -rf src/native/*.c release/src/`
		`+ cp -rf src/llhttp.gyp release/`
		`+ cp -rf src/common.gypi release/`
		`+ - cp -rf CMakeLists.txt release/`
		`+ - sed -i '' s/_RELEASE_/$(RELEASE)/ release/CMakeLists.txt`
		`+ + sed s/_RELEASE_/$(RELEASE)/ CMakeLists.txt > release/CMakeLists.txt`
		`+ cp -rf libllhttp.pc.in release/`
		`+ cp -rf README.md release/`
		`+ cp -rf LICENSE-MIT release/`

audited-null-licenses.toml

file modified

+13 -1

		`@@ -5,7 +5,7 @@`
		`[dev]`

		`# Just a module wrapper around the code in tslib, which does have a proper`
		`- # license in its package.json:`
		`+ # license (0BSD) in its package.json:`
		`# tslib/modules`
		`modules = "<unknown version>"`
		`# A “dummy” module in the tests for tslib`
		`@@ -29,3 +29,15 @@`
		`browser_field = "<unknown version>"`
		`# resolve/test/resolver/symlinked/package`
		`package = "<unknown version>"`
		`+`
		`+ # These are all part of nanoid, which is MIT-licensed.`
		`+ # nanoid/url-alphabet`
		`+ url-alphabet = "<unknown version>"`
		`+ # nanoid/non-secure`
		`+ non-secure = "<unknown version>"`
		`+ # nanoid/async`
		`+ async = "<unknown version>"`
		`+`
		`+ # This is part of yargs, which is MIT-licensed.`
		`+ # mocha/node_modules/yargs/helpers`
		`+ helpers = "<unknown version>"`

llhttp.spec

file modified

+5 -1

		`@@ -26,7 +26,7 @@`
		`# additional source even if we do not do the re-generation ourselves.`

		`Name: llhttp`
		`- Version: 8.1.0`
		`+ Version: 8.1.1`
		`%global so_version 8.1`
		`Release: %autorelease`
		`Summary: Port of http_parser to llparse`
		`@@ -54,6 +54,10 @@`
		`Source3: check-null-licenses`
		`Source4: audited-null-licenses.toml`

		`+ # Do not assume a particular sed implementation`
		`+ # https://github.com/nodejs/llhttp/pull/230`
		`+ Patch: %{url}/pull/230.patch`
		`+`
		`# The compiled RPM does not depend on NodeJS at all, but we cannot build it`
		`# on architectures without NodeJS.`
		`ExclusiveArch: %{nodejs_arches}`

sources

file modified

+2 -2

		`@@ -1,2 +1,2 @@`
		`- SHA512 (llhttp-8.1.0.tar.gz) = 363f8b3f8aa743f2529c0738b8b95dd682bacf7bd1afa1fc9b8a1d67251e1ced90b986402939ea484bc2cf511a83819b3def6345885ad59e8854c7c263d999da`
		`- SHA512 (llhttp-8.1.0-nm-dev.tgz) = 4b5aa38a4b77ade1e2f3758a156efe8698f5f0cbb7cafe9f547dd45d425f66d17bbb10b20cc98cef1268a0f8f6a8fa11f95e5e0ddde363e196425bc565f87431`
		`+ SHA512 (llhttp-8.1.1.tar.gz) = 18530d1fcfde13ba634b367852e53bf7a2dc0eb130f07fbbb5b5e02b3d4ff0ab0cd7c5a0adef8a7a0e93a0104a2544ce11d3631792539c48e1a918c5d40f7f3c`
		`+ SHA512 (llhttp-8.1.1-nm-dev.tgz) = 5348f09a4d8a7b7ecb5d4204d47b1f6c36ed7d131cd89f84feef475cba74fc651bf63bc10293fb5481c19c88e07e3ae2e44881ffda19606876e1f38de32ae80f`

music commented 2 months ago

https://github.com/nodejs/llhttp/releases/tag/release%2Fv8.1.1

https://github.com/nodejs/llhttp/releases/tag/v8.1.1

Stricter parsing of status line
Do not allow empty header separated by CR.

zuul commented 2 months ago

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/89a756785121481297c59c5bc59b46a1

check-for-arches : SUCCESS in 1m 50s
rpm-scratch-build : SUCCESS in 17m 37s
rpm-scratch-build-s390x : SUCCESS in 15m 16s (non-voting)
rpm-scratch-build-ppc64le : SUCCESS in 19m 43s (non-voting)
rpm-scratch-build-i686 : SUCCESS in 17m 11s (non-voting)
rpm-scratch-build-aarch64 : SUCCESS in 17m 19s (non-voting)
rpm-linter : SUCCESS in 4m 27s (non-voting)
rpm-rpminspect : SUCCESS in 9m 12s (non-voting)
check-for-sti-tests : SUCCESS in 14s
check-for-fmf-tests : SUCCESS in 14s
rpm-install-test : NODE_FAILURE Node request 200-0006140300 failed in 0s
Skipped 2 jobs

music commented 2 months ago

Causes the following test failure in python-aiohttp:

=================================== FAILURES ===================================
__________ test_http_response_parser_code_under_100[c-parser-pyloop] ___________

response = <aiohttp._http_parser.HttpResponseParser object at 0x7f1e4632c580>

    def test_http_response_parser_code_under_100(response) -> None:
>       msg = response.feed_data(b"HTTP/1.1 99 test\r\n\r\n")[0][0][0]

response   = <aiohttp._http_parser.HttpResponseParser object at 0x7f1e4632c580>

tests/test_http_parser.py:689:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

>   raise ex
E   aiohttp.http_exceptions.BadStatusLine: 400, message="Bad status line 'Invalid status code'"


aiohttp/_http_parser.pyx:551: BadStatusLine

Reported upstream: https://github.com/aio-libs/aiohttp/issues/7327

1 new commit added