-
Notifications
You must be signed in to change notification settings - Fork 27
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
PublicQueueTask5: PIN to protect queue #21
Comments
While creating a queue, we can take as a boolean 'isPasswordProtected'. If so, a random 4 digit number is to be returned. There should also be a api for the admin to request a new random number. Similarly, while joining a queue, if the queue is password protected, the user should be asked to s enter the PIN. We should have a queue information api, that the UI can call for knowing more information on the queue. The API can return queue name, current number of people in the queue, and also if it 'isPasswordProtected'. (The APIs will be much cleaner once we have @thehamzarocks 's #16 and auth.) |
@avinashkris9 This is a nice task, you want to take it up? |
I can try. I have some doubts
|
|
How is the pin stored and validated ? |
The pin is stored in the queue table. Validation is manual, we will have to check the pin from the user against the pin from the db. The join queue form will send a queue status request when the page loads, it will have a boolean field The admin would also have a button somewhere to reset the pin with a new one. |
Just a suggestion encrypt (not encode) the pin before storing it in db , will be a good practice |
From what I think, these are the reasons why we encrypt data:
Our queue password, is a random 4 digit pin that we store. Think of it like a bluetooth pairing code. I don't see a need to encrypt it. In the event of a hack, I'll be more concerned about the name and mobile number that will get leaked. |
A queue creator can optionally choose to protect his queue by using a 4 digit pin. Then he can keep refreshing the pin whenever he wishes to.
The text was updated successfully, but these errors were encountered: