Useful information. Will keep this in mind while coding. As of now, I believe a None comparision is not being done.
path operations need a kind of escaping
This needs also a test for that. We should not be able to enter a e.g. ../../ relative path or escape out of the scope of the project.
path escaping check, also directory traversal using only 'fs' for windows compatibility
Also, I apologize I didn’t comment as much. It’d make your task a lot easier. Starting next PR, please point it out if you don’t see comments. Before the first review, just briefly scan the PR and if you don’t feel it’s commented enough, let me know. This would also help during last week while writing documentation.
Add to one of the tests a UTF-8 character which isn’t in the common ASCII table. This is just to find early problems because of some modules not handle correctly utf-8.