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

plugins.serveStatic Fixes GH-1864/GH-1910 #1

Merged
merged 4 commits into from
May 26, 2022
Merged

plugins.serveStatic Fixes GH-1864/GH-1910 #1

merged 4 commits into from
May 26, 2022

Conversation

kolbma
Copy link
Owner

@kolbma kolbma commented May 26, 2022

Pre-Submission Checklist

  • Opened an issue discussing these changes before opening the PR
  • Ran the linter and tests via make prepush
  • Included comprehensive and convincing tests for changes

Issues

Closes:

Introduced BadRequestErrors for problems on bad paths like urlencoded null bytes.

Added tests to check for traversals.
Reworked existing tests for making more sense.
Remove 2 doubled tests.
lints and style of bodyReader.test fixed to get prepush to run

@kolbma
chore(tests): remove 2 doubled tests
5371b40 
The 2 tests are already available at line 230.
@kolbma
fix(restifyGH-1910): plugins.serverStatic path traversal
93909da 
Fixes restify#1910
Added tests to check for traversals.
Reworked existing tests for making more sense.
@kolbma
fix(restifyGH-1864): plugins.serveStatic null byte crash
d7f32ea 
Fixes restify#1864
and propable some other BadRequest types.
@kolbma
fix(restifyGH-1864): plugins.serveStatic crash in Gzip
1180bb8 
This is also to be handled for Gzip content.
Fixes restify#1864
@kolbma kolbma merged commit 02c1497 into plugins-serveStatic-fixes May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kolbma