tobykurien / rpi_lcars

Star Trek LCARS interface for Raspberry Pi using Pygame
MIT License
658 stars 145 forks source link

Add 9 and 0 buttons. Fix len problem in PIN code. #29

Closed JDat closed 4 years ago

JDat commented 4 years ago

To be more serious we need to have all numbers, so I added 9 and 0 buttons. By changing PIN to 12345, athorise will fail, so PIN must be string and algorithm must check by string len instead of fixed value.

PS: In future I will change windows architecture and add security. From lcars.py is called main.py window. Main window will check if user is authorised. If no, call authorisation window. Authorisation window will return back to main window only on correct pin. Also authorise window will have timer for screensaver/screen blanking. PIN will be in config.py file. PIN will be encrypted. Authorize maybe will have alphanumeric keyboard. Authorize background must be redrawn so bars are controls not background for different resolution displays.

tobykurien commented 4 years ago

Thanks! This was meant to be a toy example rather than a secure implementation, but the improvements are welcome as it doesn't make the example unnecessarily complex.