fixed how Errors are handled

pkg/vault/auth.go

@@ -15,7 +15,7 @@ import (
 
 // Login will authenticate the user with Vault
 // Will detect if user needs to re-login
-func (v *Vault) Login() error {
+func (v *Vault) Login() Error {
 	// get the token from the user's environment
 	if v.client.Token() != "" {
 		v.log.Debug("Reading token from environment 'VAULT_TOKEN'")
@@ -24,7 +24,7 @@ func (v *Vault) Login() error {
 		v.tokenHelper = token.InternalTokenHelper{}
 		token, err := v.tokenHelper.Get()
 		if err != nil {
-			return err
+			return v.parseError(err)
 		}
 
 		if token != "" {

pkg/vault/error.go

@@ -11,22 +11,39 @@ import (
 	"syscall"
 )
 
-// Error is the custom error type for this package
-type Error struct {
+// CustomVaultError is the custom error type for this package
+type CustomVaultError struct {
 	MessageParts  []string
 	OriginalError error
 }
 
+// Error is custom error interface with one method that will block other functions
+// from using this Error. This is not interchangeable with the standard error.
+type Error interface {
+	Error() string
+	blockInterface()
+}
+
 // Error returns the error string
-func (verr Error) Error() string {
-	return fmt.Sprintf("Vault Error: %s", strings.Join(verr.MessageParts, "; "))
+func (verr CustomVaultError) Error() string {
+	return fmt.Sprintf("Vault: %s", strings.Join(verr.MessageParts, "; "))
+}
+
+//
+func (verr CustomVaultError) blockInterface() {
 }
 
 // parseError parses known errors into more user-friendly messages
 func (v *Vault) parseError(err error) Error {
 
-	var verr Error
-	verr.OriginalError = err
+	// Provent parseError from calling parseError again
+	if serr, ok := err.(Error); ok {
+		return serr
+	}
+
+	// var verr CustomVaultError
+	verr := &CustomVaultError{OriginalError: err}
+	// verr.OriginalError = err
 
 	// Catch some known HTTP errors
 	if uerr, ok := err.(*url.Error); ok {

pkg/vault/secrets.go

@@ -1,8 +1,9 @@
 package vault
 
 import (
-	"github.com/hashicorp/vault/api"
 	"path/filepath"
+
+	"github.com/hashicorp/vault/api"
 )
 
 // Using Vaults Logical client:
@@ -81,7 +82,7 @@ func (v *Vault) ListSecrets(path string) ([]string, error) {
 func (v *Vault) GetSecret(path string) (*api.Secret, error) {
 	secret, err := v.client.Logical().Read(path)
 	if err != nil {
-		return nil, err
+		return nil, v.parseError(err)
 	}
 
 	return secret, nil

pkg/vault/status.go

@@ -5,11 +5,11 @@ import (
 	"time"
 )
 
-func (v *Vault) isVaultHealthy() (bool, error) {
+func (v *Vault) isVaultHealthy() (bool, Error) {
 
 	result, err := v.client.Sys().Health()
 	if err != nil {
-		return false, v.parseError(err)
+		return false, v.parseError(err).(Error)
 	}
 
 	v.log.Debug("Vault server info from (" + v.client.Address() + ")")

pkg/vault/token.go

@@ -11,7 +11,7 @@ func (v *Vault) GetCurrentTokenTTL() (time.Duration, error) {
 	// Get the token info from Vault
 	secret, err := v.client.Auth().Token().LookupSelf()
 	if err != nil {
-		return 0, err
+		return 0, v.parseError(err)
 	}
 
 	// Get our TTL from the Vault secret interface{}

pkg/vault/vault.go

@@ -66,13 +66,13 @@ func New(config *Config) (*Vault, error) {
 	// Ensure Vault is up and Healthy
 	_, err = v.isVaultHealthy()
 	if err != nil {
-		return nil, v.parseError(err)
+		return nil, err
 	}
 
 	// Run Login logic
 	err = v.Login()
 	if err != nil {
-		return nil, v.parseError(err)
+		return nil, err
 	}
 
 	// If user wants, extend the token timeout
@@ -81,13 +81,13 @@ func New(config *Config) (*Vault, error) {
 			v.log.Debug("Attempting to set token duration to ", v.config.InitialTokenDuration)
 			_, err = v.client.Auth().Token().RenewSelf(int(v.config.InitialTokenDuration.Seconds()))
 			if err != nil {
-				return nil, err
+				return nil, v.parseError(err)
 			}
 
 			// Show the actual TTL and warn if different from requested
 			actualDuration, err := v.GetCurrentTokenTTL()
 			if err != nil {
-				return nil, v.parseError(err)
+				return nil, err
 			}
 			v.log.Debug("Current token is valid for {}", actualDuration.String())
 

stim/vault.go

@@ -30,6 +30,8 @@ func (stim *Stim) Vault() *vault.Vault {
 			timeInDuration = time.Duration(0)
 		}
 
+		stim.log.Debug("Vault Address: ({})", stim.GetConfig("vault-address"))
+
 		// Create the Vault object and pass in the needed address
 		vault, err := vault.New(&vault.Config{
 			Address:              stim.GetConfig("vault-address"), // Default is 127.0.0.1