Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cookie security #880

Open
5 of 10 tasks
tablatronix opened this issue Jul 26, 2014 · 3 comments
Open
5 of 10 tasks

cookie security #880

tablatronix opened this issue Jul 26, 2014 · 3 comments

Comments

@tablatronix
Copy link
Member

tablatronix commented Jul 26, 2014

cookie security lies entirely with protection and randomness of authkey or gsusecustomsalt, as all other aspects are predictable. ( authkey is also predictable from machine state as it is just sha1 of 22 char mt_rand() at or around install timestamp). It is also not revocable unless salt changes.

  • lack of entropy in mt_rand ( not cryptographically secure)
  • Uses non delimited concatenation, splice attacks
  • uses sha1, subject to length extension attacks
  • uses loose typed comparators, datatype conversion spoofing
  • contains no time domain limiter, never expires
  • contains no password binding, never breaks when passwords change
  • sidechannel timing leakage
  • session fixation ( not sure how to avoid this stateless )
  • weak logouts, no tokens
  • Also maybe want to add a public/private computer toggle on checkbox for username and logged in status
@tablatronix
Copy link
Member Author

#844
#799

@tablatronix
Copy link
Member Author

We are hashing the cookie name, not sure what thats about, probably an unnecessary obfuscation

We are also salting / hashing the usernames, probably also unnecessary, or rather undesired , or we are going to need multiple auth keys so we are not using the same token for salts and hmac keys.

Ideally we should also have separate salts and hmac keys for each user anyway

  • implement per user keys

@tablatronix
Copy link
Member Author

tablatronix commented Feb 7, 2015

implementing some examples of better security, test branch

db0db69
1f702ae
fd64668

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

No branches or pull requests

1 participant