Support http2 client (SSRF) (2) #372

timokoessler · 2024-09-16T12:40:40Z

This pr requires the new hook system because we need to modify a return value of a method.

Commits cherry-picked from #360

Required because we need to modify return value

codecov · 2024-09-16T14:03:19Z

Codecov Report

Attention: Patch coverage is 86.69276% with 68 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
library/sinks/http2-request/wrapResponseHandler.ts	66.66%	31 Missing ⚠️
library/sinks/http2-request/wrapRequestMethod.ts	72.72%	21 Missing ⚠️
library/sinks/HTTP2Request.ts	93.00%	10 Missing ⚠️
library/sinks/http2-request/getUrlFromRequest.ts	89.74%	4 Missing ⚠️
library/helpers/http2Request.ts	97.01%	2 Missing ⚠️

📢 Thoughts on this report? Let us know!

timokoessler added 8 commits September 16, 2024 14:21

Wrap http2.connect call

a9f8273

Extend http2 tests

7d6a65b

Improve http2.connect arg parsing

1f3ae45

Inspect http2.connect dns lookups

e091cd0

Refactor http2 request arg parsing

6fffe8e

Modify http2 client test for Node v16

6509906

Increase test coverage

678d263

Migrate pr to new hook system

a25325e

Required because we need to modify return value

timokoessler mentioned this pull request Sep 16, 2024

Support http2 client (SSRF) #360

Closed

4 tasks

Wrap http2.request and extract url

99186a5

timokoessler added 4 commits September 16, 2024 16:29

Wrap http2 response handlers

0b8ff49

Block http2 ssrf redirect

c4a822c

Update SSRF redirect test urls

29440ac

Add http2 redirect test

159ce43

Base automatically changed from new-hooks-system to beta October 14, 2024 16:56

Base automatically changed from beta to main December 3, 2024 15:56