This was setting params[:redirect] to be an empty string, and I think we only want to pass redirect if there is something there.

This setting will indirectly lead to the user being redirected to the dashboard on login:
https://labs.riseup.net/code/issues/4144
Another tweak to avoid errors if version is nil, but not sure that version should ever be nil