|
|
|
|
|
by aj-code
4864 days ago
|
|
This has some fairly serious security issues, which is fine for a something not designed to be seriously used (or at all). However, the readme implies you could use this and your files will be safer than with some third party. Which is dangerous, to say the least. I'll outline a few obvious issues I see: - No explicit protection against directory traversal attacks (../../etc/passwd type stuff) on upload and download. - Shell command injection on the file name on upload. - Naive authentication.
- Unsalted, fast hash sent in the URL.
- Password stored in clear text server side. - No transport security (HTTPS). This is cool as a interesting project to work on, but it should be made clear not to use this for anything just yet. |
|
I don't understand the point of hashing the password in the client anyway... The hash is as good as the password to an attacker.