search

leostratus / interdisciplinary-programming

The site, yo.
MIT License
3 stars 1 forks source link

Error 1102: Login failed after fresh signup #23

Closed leostratus closed 10 years ago

leostratus commented 10 years ago

Need to investigate repro conditions, unclear.

leostratus commented 10 years ago

Cannot POST /logout - may be related, can't click this 'back to safety' after this thing bails.

Also, no output from the app. Maybe good ol' fashioned explicit logging?

bkatiemills commented 10 years ago

Some server-side console logs now spit out info about what is going wrong during login after profile creation; plz post logs here next time this is seen.

bkatiemills commented 10 years ago

Console logging indicates a login error and user respectively:

[Error: Failed to serialize user into session]
null

It seems that a race condition was being created between inserting the new user into the DB, and looking for that user in the DB to log them in. Moving login into the callback to DB insertion seems like a good move; 22 consecutive profile creations were successful after doing this. This error seemed to happen on the ~10% level previously, so p(22 success & 0 failure | bug still active) < 10% ; going to close now, reopen if needed.

bkatiemills commented 10 years ago

Oops, that last commit didn't have the right stuff added to it - the next one does.