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-1604 #2

Merged
merged 4 commits into from
May 26, 2022
Merged

plugins.serveStatic Fixes GH-1604 #2

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:

There should be no dependency on the dirname for specific path.

kolbma added 4 commits May 25, 2022 19:27
@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-1604): plugins.serveStatic no dep dirname
c85da14 
Fixes restify#1604
There should be no dependency on the dirname for specific path.
@kolbma kolbma merged commit 7f36ae1 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