-
Notifications
You must be signed in to change notification settings - Fork 741
Add zip bomb tests #1300
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
Merged
Merged
Add zip bomb tests #1300
Changes from all commits
Commits
Show all changes
85 commits
Select commit
Hold shift + click to select a range
0b089ef
WIP
e4e513b
remove max size from NewZstdCompressor
ef84d36
WIP support multiple compression types
4a659b0
rename compressionType to Type
c6de927
fix metric
13eb890
WIP remove CompressionEnabled and add --network-compression-type
bc55ea4
rename types
400a918
add zstd compression/decompression metrics
6f8bd3e
Merge remote-tracking branch 'origin/dev' into add-zstd-compression
38e7a12
don't allow 2 network compression flags
3850c79
remove benchmark
6065be8
cleanup
d7c5d52
don't use zstd until v1.10
385ba27
tweak error message
37bcf39
nit
81afb71
nits
4da5f21
nits
d8a9ccf
Merge remote-tracking branch 'origin/dev' into add-zstd-compression
d9b778c
flag wording nit
41bc6c5
add zstd tests; fix bugs
11c171d
consolidate metrics
ddd35ee
remove old todo
6dd3f25
update test
eab4879
update tests
1ad19c1
add tests
6b29cfc
appease linter
7c150cd
Merge branch 'dev' into add-zstd-compression
874632a
Merge remote-tracking branch 'origin/dev' into add-zstd-compression
a9a96f6
Merge branch 'add-zstd-compression' of github.com:ava-labs/avalancheg…
9526ba9
nits
e1f0d55
Merge branch 'dev' into add-zstd-compression
c000a83
Merge branch 'dev' into add-zstd-compression
483f040
use default compression type in tests
041c37e
Merge branch 'add-zstd-compression' of github.com:ava-labs/avalancheg…
6b3f9a3
flag nit
5325c6b
only allow zstd after cortina
4345b1e
Merge remote-tracking branch 'upstream/dev' into add-zstd-compression
51d3cbb
merged
StephenButtolph 9dc7108
Merge remote-tracking branch 'upstream/dev' into add-zstd-compression
b401ea6
address PR comments
b0225da
remove switch
45d9b3f
Merge branch 'add-zstd-compression' of github.com:ava-labs/avalancheg…
8472760
add max message size test
0e61411
add max message size to zstd
32d8d37
add test
72a0b7c
Merge remote-tracking branch 'upstream/dev' into add-zstd-compression
db6504a
remove switch
53df4c6
use stream interface for Decompress to avoid unzip bomb
adbc82a
fix copy pasta bug
e510398
move switch case to default
82d7f68
remove impossible switch case
bdcf189
appease linter
b91159d
make reader a local var
c259199
return nit
9105741
add invalid max size check to zstd compressor creation
8ad8fe6
Merge branch 'dev' into add-zstd-compression
StephenButtolph eaec3ef
Parallelize gzip compression
StephenButtolph c6c6fdf
merged
StephenButtolph c6f4800
log warning for unknown op during metrics observation
8ad1cb6
use sync.Pool of gzip writer
8c499af
gzipWriter.Flush() --> gzipWriter.Close()
812a37d
test cleanup
084be58
Merge remote-tracking branch 'upstream/add-zstd-compression' into sim…
a0c1dda
appease linter
c1d7a98
Merge remote-tracking branch 'upstream/add-zstd-compression' into sim…
26e7235
remove magic number
f3f7785
Merge remote-tracking branch 'upstream/add-zstd-compression' into sim…
a2cc314
Merge branch 'dev' into add-zstd-compression
StephenButtolph 436685b
imports nit
StephenButtolph d30e0a9
Merge remote-tracking branch 'upstream/add-zstd-compression' into sim…
b40e14b
nit
StephenButtolph 0c0a396
test nits
422a840
Merge remote-tracking branch 'upstream/add-zstd-compression' into sim…
0fca08a
nits
StephenButtolph dfc909d
nit
StephenButtolph 781023e
Merge branch 'add-zstd-compression' into simplify-gzip-compression
StephenButtolph e769d97
Add zipbomb tests
StephenButtolph c2fe2e4
nit
StephenButtolph e550864
merged
StephenButtolph dda9596
Embed the zip bombs
StephenButtolph 65ec891
nits
StephenButtolph 7afff07
Merge branch 'simplify-gzip-compression' into add-zip-bomb-tests
StephenButtolph 0f43f37
Merge branch 'dev' into simplify-gzip-compression
StephenButtolph a074713
Merge branch 'simplify-gzip-compression' into add-zip-bomb-tests
StephenButtolph 37ed76c
Merge branch 'dev' into add-zip-bomb-tests
StephenButtolph File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file not shown.
Binary file not shown.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kinda interesting that you have to make this at least 6 to pass 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's just because of how the underlying bytes buffer grows afaict.