rename userd -> user_details for consistency.

Plus minor comment fixes
push all session related setups through same code path

valid and tempsessions and normal sessions were using same calls but sometimes running overlapping code.  There are three endpoints - tempsession or anonymous browsing, autosession or testing, valid or valruse endpioit.  All use same codepaths to setup sessions
autopep8