Skip to content

Fixes for naughty filenames #867

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

Closed
beardhatcode wants to merge 10 commits into from
Closed

Fixes for naughty filenames #867

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
1f12788
tests: Add tests for naughty filenames
beardhatcode Apr 18, 2021
2fcbda9
More challenging filename + move into folder
beardhatcode Apr 18, 2021
37cae88
Add checks for public links
beardhatcode Apr 18, 2021
e217e64
test: fix path not escaped in file upload
beardhatcode Apr 18, 2021
5325327
Properly encode the DAV path
beardhatcode Apr 18, 2021
3896bbb
Better testname
beardhatcode Apr 18, 2021
14341e9
Use more difficult emoji
beardhatcode Apr 19, 2021
195916f
It turns out that there is a built in CSS.escape
beardhatcode Apr 19, 2021
9f85553
fix: make cypress create user more informative
beardhatcode Apr 19, 2021
9ed4dda
Formatting
beardhatcode Apr 19, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use more difficult emoji 
Signed-off-by: Robbert Gurdeep Singh <[email protected]>
  • Loading branch information
@beardhatcode
beardhatcode committed Apr 19, 2021
commit 14341e93dbe7fc29da71f8aedfb9fd21010455c0
5 changes: 4 additions & 1 deletion cypress/integration/oddname.spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,10 @@ function naughtyFileName(realName) {
"realy #1's " +
'" #_+="%2520%27%22%60%25%21%23 was this called ' +
realName +
"? :p ." +
"in the" +
"☁️" +
"👩‍💻" +
"? :* ." +
ext.toUpperCase()
);
}
Expand Down