Not much point, we need most of the arguments sent to the login function so it's easier to extract the whole lot as an object. So then checking for index of a constant string not much point in having a key variable that is only used once and doesn't vary. But I'll leave that as a matter of preference.