5846: Fixed the Login,Password field character limit in login page

inclusive-design / AChecker

Automated interactive Web content accessibility checker.

https://achecker.ca

GNU General Public License v2.0

69 stars 61 forks source link

5846: Fixed the Login,Password field character limit in login page #70

Closed pranjaljn97 closed 6 years ago

pranjaljn97 commented 6 years ago

Restrict HTML maxlength and size validation on login and password field in themes/default/login.tmpl.php

cindyli commented 6 years ago

Thanks for the pull request, @pranjaljn97.

The validation of field lengths should also be verified at the server side before saving into the database for security reason, for example, when requests are not sent through the login page but via other means such as hacking scripts or curl commands.

pranjaljn97 commented 6 years ago

My Pleasure @cindyli. I have done the server side validation too on the login as well as register functionality.Also added error handling part. Though, password is encrypting on client side only.So, I cant validate length on the server side. Thank You. Looking Forward to your Feedback :)