Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

POSTing user credentials isn't secure #168

Closed
ilanbiala opened this issue Sep 2, 2014 · 5 comments
Closed

POSTing user credentials isn't secure #168

ilanbiala opened this issue Sep 2, 2014 · 5 comments

Comments

@ilanbiala
Copy link
Member

In the $scope.signin function definition, there is a POST request that just sends someth:

{
  user: 'johndoe',
  password: 'jdoe123'
}

Is there no better way to transport this data, because users can just open dev tools and copy the user/pass combo or set up an HTTP request tracker and get that from a victim's computer.

@yoneal
Copy link

yoneal commented Sep 6, 2014

Encryption is usually handled by tunneling transport through HTTPS.

@jloveland
Copy link
Contributor

I have a PR out for HTTPS. Please take a look.

There is a salt and hash function for the user schema. We should look into making this stronger. Such as

@ilanbiala
Copy link
Member Author

@jloveland what is wrong with the hashing we are using now? I get that it isn't the best, but the tradeoff probably isn't worth it for most people and they can change it if they want.

@jloveland
Copy link
Contributor

@ilanbiala, I didn't review all the code. I see there's nothing wrong with the way the hashing is working...looks like mean is using crypto.pbkdf2Sync here. This is great because it's following the NIST Special Publication 800-132 guidelines.

I agree that if people want different crypto, they can just change it.

@jloveland
Copy link
Contributor

Pull Request #140 has been merged. @roieki , can this be closed or are there other concerns?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants