Delete former credentials when rotating (granted credentials rotate)

[n3s7or]

What changed?

Added code to delete old credentials.

Why?

How did you test it?

I rotated my own AWS credentials

Potential risks

Is patch release candidate?

No

Link to relevant docs PRs

[chrnorm]

Thankyou @n3s7or!!

[chrnorm]

Actually, just looking through the PR and original issue #388 and it would be good to introduce this as opt-in behaviour. Could you please add a flag --delete to the command, and then only delete the key if the flag is set?

You can do so by adding

&cli.BoolFlag{Name: "delete", Usage: "delete the previous active key"}

to the command definition and then using

if c.Bool("delete") { }

to check if the flag is set.

[n3s7or]

Thanks for your input @chrnorm
The change is done :)

[chrnorm]

🙌 thankyou @n3s7or!