cybr conjur logon-non-interactive with environment variables failing

[David-Lang]

Describe the bug cybr conjur logon-non-interactive with environment variables failing

To Reproduce Steps to reproduce the behavior:

1. set environment variables
2. cybr conjur logon-non-interactive

Additional context Looks like the "if err != nil" conditional is missing after getClientFromEnvironmentVariable() so it always try's to getClientFromAuthenticator()

// If .conjurrc is empty, attempt to get client from environment variables if account == "" && baseURL == "" && certPath == "" { // Get client from environment variables client, loginPair, err := getClientFromEnvironmentVariable() // If partial environment variables were provided, try authenticator client, loginPair, err = getClientFromAuthenticator() // Partial environment variables were provided, assume user is attempting to use environment variables if err != nil { return &conjurapi.Client{}, &authn.LoginPair{}, err } // If no error returned, return client created from environment variables if *client != (conjurapi.Client{}) { return client, loginPair, nil } }