484
submitted 11 months ago* (last edited 11 months ago) by [email protected] to c/[email protected]

Just take the string as bytes and hash it ffs

you are viewing a single comment's thread
view the rest of the comments
[-] [email protected] 199 points 11 months ago

There’s a special place in hell for those who set an upper limit in password lengths.

[-] [email protected] 71 points 11 months ago

I sort of get it. You don't want to allow the entire work of Shakespeare in the text field, even if your database can handle it.

16 characters is too low. I'd say a good upper limit would be 100, maybe 255 if you're feeling generous.

[-] [email protected] 91 points 11 months ago

The problem is that you (hopefully) hash the passwords, so they all end up with the same length.

[-] [email protected] 58 points 11 months ago

At minimum you need to limit the request size to avoid DOS attacks and such. But obviously that would be a much larger limit than anyone would use for a password.

[-] [email protected] 27 points 11 months ago

Also rate of the requests. A normal user isn't sending a 1 MiB password every second

[-] [email protected] 4 points 11 months ago

What's a sensible limit. 128 bytes? Maybe 64?

[-] [email protected] 8 points 11 months ago

I'd say 128 is understandable, but something like 256 or higher should be the limit. 64, however, is already bellow my default in bitwarden

[-] [email protected] 6 points 11 months ago

And sure, in theory your hashing browser-side could break if you do that. Depending on how much text the user pastes in. But at that point, it's no longer your problem but the browser's. 🦹

[-] [email protected] 21 points 11 months ago

Why are you hasing in the browser?

Also, what hashing algorithm would break with large input?

[-] [email protected] 10 points 11 months ago
[-] [email protected] 4 points 11 months ago

Damm, I legit didn't knew there bcrypt had a length limit! Thank you for another reason not to use bcrypt

[-] [email protected] 4 points 11 months ago

Scrypt has the same limit, FWIW.

It doesn't matter too much. It's well past the point where fully random passwords are impossible to brute force in this universe. Even well conceived passphrases won't get that long. If you're really bothered by it, you can sha256 the input before feeding it to bcrypt/scrypt, but it doesn't really matter.

load more comments (2 replies)
[-] [email protected] 4 points 11 months ago* (last edited 11 months ago)

Why would you not hash in the browser. Doing so makes sure the plaintext password never even gets to the server while still providing the same security.

Edit: I seem to be getting downvoted... Bitwarden does exactly what I described above and I presume they know more than y'all in terms of security https://bitwarden.com/help/what-encryption-is-used/#pbkdf2

[-] [email protected] 20 points 11 months ago* (last edited 11 months ago)

Because then the hash is the password. Someone could just send the hash instead of trying to find a password that gets the correct hash. You can't trust the client that much.

You can hash the password on both sides to make it work; though I'm not sure why you'd want to. I'm not sure what attack never having the plain text password on the server would prevent. Maybe some protection for MITM with password reuse?

[-] [email protected] 8 points 11 months ago

Because then that means you don't salt your hashes, or that you distribute your salt to the browser for the hash. That's bad.

[-] [email protected] 4 points 11 months ago

You could salt it. Distributing a unique salt doesn't help attackers much. Salt is for preventing precomputing attacks against a whole database. Attacking one password hash when you know the salt is still infeasible.

It's one of those things in security where there's no particular reason to give your attacker information, but if you've otherwise done your job, it won't be a big deal if they do.

You don't hash in the browser because it doesn't help anything.

[-] [email protected] 1 points 11 months ago

It helps against the server being able to read the password, so a bad actor (either the website itself or after a hack) could read your password. Which isn't bad if you're using good password hygiene with random passwords, but that sadly is not the norm.

[-] [email protected] 2 points 11 months ago

It doesn't. It just means the attacker can send the hash instead of the password.

[-] [email protected] 1 points 11 months ago* (last edited 11 months ago)

For that particular website yes, but a salted client side hash is worthless on a different website.

Edit: plus even unsalted it would only work if the algorithm is the same and less iterations are done

[-] [email protected] 2 points 11 months ago

If the end user is reusing passwords. Which, granted, a lot of people do.

On the flip side, we're also forcing the use of JavaScript on the client just to handle passwords. Meanwhile, the attack we're protecting against only works for reused passwords, and the attacker is inside the server and can see the password after transport layer encryption is removed. This is a pretty marginal reason to force the complexity of JavaScript.

[-] [email protected] 4 points 11 months ago

With comments like this all over public security forums, it's no wonder we have so many password database cracks.

[-] [email protected] 3 points 11 months ago* (last edited 11 months ago)

Per your edit, you're misunderstanding what Bitwarden does and why it's different than normal web site password storage.

Bitwarden is meant to not have any insight into your stored passwords what so ever. Bitwarden never needs to verify that the passwords you've stored match your input later on. The password you type into Bitwarden to unlock it is strictly for decrypting the database, and that only happens client side. Bitwarden itself never needs to even get the master password on the server side (except for initial setup, perhaps). It'd be a breach of trust and security if they did. Their system only needs to store encrypted passwords that are never decrypted or matched on their server.

Typical website auth isn't like that. They have to actually match your transmitted password against what's in their database. If you transmitted the hashed password from the client and a bad actor on the server intercepted it, they could just send the hashed password and the server would match it as usual.

[-] [email protected] 8 points 11 months ago

If you hash in the browser it means you don't salt your hash. You should absolutely salt your hash, not doing so makes your hashes very little better than plaintext.

[-] [email protected] 6 points 11 months ago* (last edited 11 months ago)

There's nothing stopping a browser from salting a hash. Salts don't need to be kept secret, but it should be a new random salt per user.

load more comments (1 replies)
[-] [email protected] 5 points 11 months ago* (last edited 11 months ago)

Even 255 bytes with 10 million entries is only ~2.6GB of data you need to store, and if you have 10 million users the probably $1 a month extra that would cost is perfectly fine.

I suppose there may be a performance impact too since you have to read more data to check the hash, but servers are so fast now it doesn't seem like that would be significant unless your backend was poorly made.

[-] [email protected] 4 points 11 months ago

Yeah but what if I have one user with 9.9 million accounts? That bastard

[-] [email protected] 2 points 11 months ago
[-] [email protected] 53 points 11 months ago

Oh and also, "change this every four weeks please."

Okay then. NEW PASSWORD: pa$$word_Aug24

[-] [email protected] 4 points 11 months ago

Yep. Having to have requirements that doesn't flow with people very well and requiring constant updates, people WILL find shortcuts. In the office, I've seen sheets of paper with the password written down, I've seen sticky notes, I've seen people put them in notepad/word so they could just copy paste.

This is made worse, because you have to go out of your way for a password manager, which means you need to know what that is. And you need a good one because there has been (and I'm going to generalize here) problems with some password managers in the past. And for work, they have to allow a password manager for that to even be an option. Which you then end up with this security theater.

[-] [email protected] 3 points 10 months ago

And you need a good one because there has been problems with some password managers in the past.

coughLastPasscough

“Problems”. What an delightfully understated term to use.

[-] [email protected] 3 points 11 months ago

the password cannot contains the same sequences of characters as the old password.

and i have seen this requirement in a service that requires changing it every month for some reasons.

and this is to manage a government digital identity that allows to log it in all governments websites.

[-] [email protected] 2 points 10 months ago* (last edited 10 months ago)

the password cannot contains the same sequences of characters as the old password.

That's a weird way to say "we store your password in plaintext"

[-] [email protected] 3 points 10 months ago

Not necessarily. Presumably the change password form requires entering the old and new password at the same time. Then they can compare the two as plain text and hash the old password to make sure it matches, then if so, hash the new password and overwrite it. Passwords stored hashed, comparison only during the change process. A theme on this is checking password complexity rules during the login process and advising to update to something more secure. It's possible because you're sending the password as plain text (hopefully over a secure connection), so it can be analysed before computing the hash. This even works if the hash is salt and peppered.

[-] [email protected] 18 points 11 months ago

Reasonable upper limits are OK. But FFS, the limit should be enough to have a passphrase with 4 or 5 words in it.

[-] [email protected] 10 points 11 months ago

Just opened a PayPal account and their limit is 20. Plus the only 2fa option is sms 🙃.

[-] [email protected] 10 points 10 months ago

I just double checked and I have TOTP enabled for my PayPal account so it should be an option.

I just found this support article of theirs and it says it can only be enabled through their website and not through the app (why?!) so you might be running into that?

https://www.paypal.com/uk/cshelp/article/what-is-2-step-verification-and-how-do-i-turn-it-on-or-off-help167

[-] [email protected] 5 points 10 months ago

Probably people would struggle to scan the QR Code with their smartphone. I think most apps can scan it from a image but obviously this would be unsafe, especially when people sync their screenshot to the cloud.

I can 100% confirm totp exist for PayPal, because I'm using it.

load more comments (1 replies)
[-] [email protected] 6 points 10 months ago* (last edited 10 months ago)

"Your password needs to be less than 65k characters long" >:(

[-] [email protected] 5 points 10 months ago

Darn, can't use the entire Bee Movie on Blu-Ray as my password then.

load more comments (1 replies)
[-] [email protected] 5 points 11 months ago

Basically guaranteed to be a clear text offender

[-] [email protected] 3 points 10 months ago

Especially since it takes more effort to limit it than leave it wide open for whatever length of password a user wants to use.

nvarchar(max) is perfect to store the hashed copy.

this post was submitted on 26 Aug 2024
484 points (100.0% liked)

Mildly Infuriating

41103 readers
582 users here now

Home to all things "Mildly Infuriating" Not infuriating, not enraging. Mildly Infuriating. All posts should reflect that.

I want my day mildly ruined, not completely ruined. Please remember to refrain from reposting old content. If you post a post from reddit it is good practice to include a link and credit the OP. I'm not about stealing content!

It's just good to get something in this website for casual viewing whilst refreshing original content is added overtime.


Rules:

1. Be Respectful


Refrain from using harmful language pertaining to a protected characteristic: e.g. race, gender, sexuality, disability or religion.

Refrain from being argumentative when responding or commenting to posts/replies. Personal attacks are not welcome here.

...


2. No Illegal Content


Content that violates the law. Any post/comment found to be in breach of common law will be removed and given to the authorities if required.

That means: -No promoting violence/threats against any individuals

-No CSA content or Revenge Porn

-No sharing private/personal information (Doxxing)

...


3. No Spam


Posting the same post, no matter the intent is against the rules.

-If you have posted content, please refrain from re-posting said content within this community.

-Do not spam posts with intent to harass, annoy, bully, advertise, scam or harm this community.

-No posting Scams/Advertisements/Phishing Links/IP Grabbers

-No Bots, Bots will be banned from the community.

...


4. No Porn/ExplicitContent


-Do not post explicit content. Lemmy.World is not the instance for NSFW content.

-Do not post Gore or Shock Content.

...


5. No Enciting Harassment,Brigading, Doxxing or Witch Hunts


-Do not Brigade other Communities

-No calls to action against other communities/users within Lemmy or outside of Lemmy.

-No Witch Hunts against users/communities.

-No content that harasses members within or outside of the community.

...


6. NSFW should be behind NSFW tags.


-Content that is NSFW should be behind NSFW tags.

-Content that might be distressing should be kept behind NSFW tags.

...


7. Content should match the theme of this community.


-Content should be Mildly infuriating.

-The Community !actuallyinfuriating has been born so that's where you should post the big stuff.

...


8. Reposting of Reddit content is permitted, try to credit the OC.


-Please consider crediting the OC when reposting content. A name of the user or a link to the original post is sufficient.

...

...


Also check out:

Partnered Communities:

1.Lemmy Review

2.Lemmy Be Wholesome

3.Lemmy Shitpost

4.No Stupid Questions

5.You Should Know

6.Credible Defense


Reach out to LillianVS for inclusion on the sidebar.

All communities included on the sidebar are to be made in compliance with the instance rules.

founded 2 years ago
MODERATORS