Merge pull request #16 from numberly/token-not-renewed-correctly

SoulKyu · web-flow · commit 5eb599fd7fe9 · 2024-06-03T11:27:13.000+02:00
fix(token-was-not-correctly-renewed-for-the-renewer-and-revoker): use…
diff --git a/pkg/vault/handle_token.go b/pkg/vault/handle_token.go
@@ -390,10 +390,11 @@ func (c *Connector) StartTokenRenewal(ctx context.Context, cfg *config.Config) {
 					c.Log.Errorf("Failed to renew Vault token: %v", err)
 					c.Log.Info("Trying to reconnect to Vault")
 					newConn, err := ConnectToVault(ctx, cfg)
-					c.K8sSaVaultToken = newConn.K8sSaVaultToken
 					if err != nil {
 						c.Log.Fatalf("Can't reconnect to VAULT: %v", err)
 					}
+					c.K8sSaVaultToken = newConn.K8sSaVaultToken
+					c.SetToken(newConn.K8sSaVaultToken)
 				}
 				c.Log.Debug("Token has been renewed succefully !")
 			}

Original file line number	Diff line number	Diff line change
`@@ -390,10 +390,11 @@ func (c Connector) StartTokenRenewal(ctx context.Context, cfg config.Config) {`
`390`	`390`	`c.Log.Errorf("Failed to renew Vault token: %v", err)`
`391`	`391`	`c.Log.Info("Trying to reconnect to Vault")`
`392`	`392`	`newConn, err := ConnectToVault(ctx, cfg)`
`393`		`- c.K8sSaVaultToken = newConn.K8sSaVaultToken`
`394`	`393`	`if err != nil {`
`395`	`394`	`c.Log.Fatalf("Can't reconnect to VAULT: %v", err)`
`396`	`395`	`}`
	`396`	`+ c.K8sSaVaultToken = newConn.K8sSaVaultToken`
	`397`	`+ c.SetToken(newConn.K8sSaVaultToken)`
`397`	`398`	`}`
`398`	`399`	`c.Log.Debug("Token has been renewed succefully !")`
`399`	`400`	`}`