go-fed / apcore

Golang ActivityPub Server Framework
GNU Affero General Public License v3.0
104 stars 10 forks source link

Bcrypt package already salts password hash #58

Open aschrijver opened 3 years ago

aschrijver commented 3 years ago

I noticed that crypto.go salts the password before sending to bcrypt, but within Crypto module salting already takes place (see Hash() func for how its added).

cjslep commented 3 years ago

Perfect! Will rip it out and save on the mental thinking overhead.