failure on ErrorStr creation

[Travers]

We use a hard index when creating an ErrorStr during a failure path in the login process. Looks like we are not consistent in how we create these, so we should move to using ErrorStr(str(e)) when they are being built from Exceptions