From bf96f636491d2bdfc3c06cad98ec2f5dfd007bfe Mon Sep 17 00:00:00 2001
From: Hamid Ghaf <83242695+hghaf099@users.noreply.github.com>
Date: Thu, 4 May 2023 08:03:37 -0700
Subject: [PATCH] CLI to take days as a unit of time (#20477)

* CLI to take days as a unit of time

* CL
---
 changelog/20477.txt          | 3 +++
 command/base_flags.go        | 4 ++--
 command/token_create_test.go | 6 ++++++
 3 files changed, 11 insertions(+), 2 deletions(-)
 create mode 100644 changelog/20477.txt

diff --git a/changelog/20477.txt b/changelog/20477.txt
new file mode 100644
index 000000000..e95305a70
--- /dev/null
+++ b/changelog/20477.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+cli: CLI should take days as a unit of time for ttl like flags
+```
diff --git a/command/base_flags.go b/command/base_flags.go
index c15b7b655..3fe069fbb 100644
--- a/command/base_flags.go
+++ b/command/base_flags.go
@@ -594,7 +594,7 @@ type DurationVar struct {
 func (f *FlagSet) DurationVar(i *DurationVar) {
 	initial := i.Default
 	if v, exist := os.LookupEnv(i.EnvVar); exist {
-		if d, err := time.ParseDuration(appendDurationSuffix(v)); err == nil {
+		if d, err := parseutil.ParseDurationSecond(v); err == nil {
 			initial = d
 		}
 	}
@@ -634,7 +634,7 @@ func (d *durationValue) Set(s string) error {
 		s = "-1"
 	}
 
-	v, err := time.ParseDuration(appendDurationSuffix(s))
+	v, err := parseutil.ParseDurationSecond(s)
 	if err != nil {
 		return err
 	}
diff --git a/command/token_create_test.go b/command/token_create_test.go
index b80eec5d7..31bbd244b 100644
--- a/command/token_create_test.go
+++ b/command/token_create_test.go
@@ -71,6 +71,12 @@ func TestTokenCreateCommand_Run(t *testing.T) {
 			"not present in secret",
 			1,
 		},
+		{
+			"ttl",
+			[]string{"-ttl", "1d", "-explicit-max-ttl", "2d"},
+			"token",
+			0,
+		},
 	}
 
 	t.Run("validations", func(t *testing.T) {