Missing return provokes wrong metrics delivery

[eduardogr]

We missed a return statement after the line 156 of the backend/vault.go file:

• https://github.com/tuenti/secrets-manager/blob/master/backend/vault.go#L156

and this is provoking that we are sending successful login metrics when we shouldn't:

c.logger.Info("login successful, got a new vault token")

[dannyk81]

yes, indeed 😅 @fcgravalos actually caught that last week when we had a sealed Vault.

We'll do a quick PR to fix this, thanks for noticing this too 👍

[eduardogr]

i've just created the issue because he told me about that, i'm doing a quick fix, as soon as i upload it i create the PR