I'm pretty sure that `(login)` will logout first if the user to be logged in is different from the one already logged in.  So `(logout)` here is unnecessary