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.

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

Simple security feature added #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

BrianMwangi21
Copy link
Contributor

@BrianMwangi21 BrianMwangi21 commented Nov 4, 2017

Since it's a quick upload -> download web app, I thought of a simple security feature.
On upload, a 4-digit random number is generated which can be memorized or written down.
On download, the 4-digit number is requested, if correct, download proceeds.

This helps when :

  1. You are not immediately downloading your file,
  2. Someone else has the url and you don't want them to download the file and delete it from the system. Hence you can't lose it

Though it works to that point, it would still be nice to redirect the user back to index.php after download is complete. I have tried but to no avail. Any way to help?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant