Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
@yabbat IT IS NOT CASE SENSITIVE AND THEREFOR I AM NOT SCREAMING AT YOU RIGHT NOW
-
C0D4681453yuserPassword.toLower()
Could be worse, could have the damn sql query be
# password LIKE "{userPassword}"
The shit I've seen, would haunt you for life. -
sariel85343y@theabbie but it is.
Since they're using a to lower function, I'm going to assume it's because it's stored as plaintext in the DB.
It should be hashed, and then case wouldn't matter. -
@sariel there's no reason to make that assumption. If it is lowered before it's hashed, it would still be fine.
Only security risk is drastically reducing brute force time -
@sariel It's only making it slightly less secure, but just 26 less characters won't make any difference, even if it's plaintext, it's a university database, if it gets breached all data is already lost, password won't make any difference.
-
@COD4 @heyheni understand :) The issue isn't about lowercasing passwords. It's about how it's stored, and how the auth function does the lookup. If you don't follow basic practices like hashing the password as-is, then you probably also broke rules like searching for the password field in plain text.
My next step when I see an app like that is to see if they're sanitizing my password in any way before running it through the database lookups. We might have a good case of Bobby Tables. Depending on how old Bobby was when XKCD #327 was written, its likely he's now in college :) -
so you mean it's not hashed, and directly stored on database, how terrible :|
website, address? -
It's a security fiasco.
It doesn't matter if it's case insensitive or not - it matters that I now _know_ about internal stuff.
It's the same if you e.g. print the nginx version / PHP version… or have HTTP headers like Via / X-Cache-Hit / ...
Don't allow people to know your environment - it's the simplest and most important rule in security. -
sariel85343y@theabbie so shitty security practices are ok.
Nah, I don't agree with that. Security of PII is federally regulated for educational institutions. If you don't comply, your accreditation can be in jeopardy.
Don't be a lazy piece of shit dev, at the very least hash the passwords. Make your requirements whatever, but don't store them plaintext. -
@sariel I assume it's because they want to be able to tell a student their password if they forget it, they could allow resetting it, but, who cares.
Hashing password is too simple and they won't leave it if they were aware of it, either they're unaware or it's intentional, most universities set default password, security was never a concern for universities. -
Awlex182753y@C0D4 why would you call toLower() on the password, when you could just use ILIKE on the server.
SMH my head, shorter code is always better -
LLAMS37483y@theabbie Arent you the one that leaked a load of your friends personal data through Github that one time?
Makes sense that you dont think insecure password storage is that bad. -
@LLAMS OP being a student of University and not doing anything to fix it and rather crying here is a bigger disappointment, Also, I am surprised people still remember it.
-
Maer17823ySometimes universities let their students set up university websites. Can't really blame the students - many are beginners. They might have yet to learn about security practices etc.
But the university is of course to blame. There should be a review process so student code gets checked by someone with experience - too often that's not the case.
Iirc at my university they used to allow students to set up frontends under the university domain, however any access to the backend of the university systems was prohibited via access restrictions. Students had to set up their own backend and route properly. Not really sure how that all worked, though.
The password field for our university website isn't case sensitive π€£.
rant