Handle parsing of timestamps in time.DateTime format. Fix gruntwork-i…

…o#670

util/time.go

@@ -1,10 +1,11 @@
 package util
 
 import (
+	"time"
+
 	"github.com/aws/aws-sdk-go/aws"
 	"github.com/gruntwork-io/cloud-nuke/logging"
 	"github.com/gruntwork-io/go-commons/errors"
-	"time"
 )
 
 const (
@@ -16,7 +17,8 @@ const (
 	FirstSeenTagKey = "cloud-nuke-first-seen"
 
 	// The time format of the `firstSeenTagKey` tag value.
-	firstSeenTimeFormat = time.RFC3339
+	firstSeenTimeFormat       = time.RFC3339
+	firstSeenTimeFormatLegacy = time.DateTime
 )
 
 func IsFirstSeenTag(key *string) bool {
@@ -26,9 +28,12 @@ func IsFirstSeenTag(key *string) bool {
 func ParseTimestamp(timestamp *string) (*time.Time, error) {
 	parsed, err := time.Parse(firstSeenTimeFormat, aws.StringValue(timestamp))
 	if err != nil {
-		logging.Debugf("Error parsing the timestamp into a `RFC3339` Time format")
-		return nil, errors.WithStackTrace(err)
-
+		logging.Debugf("Error parsing the timestamp into a `RFC3339` Time format. Trying parsing the timestamp using the legacy `time.DateTime` format.")
+		parsed, err = time.Parse(firstSeenTimeFormatLegacy, aws.StringValue(timestamp))
+		if err != nil {
+			logging.Debugf("Error parsing the timestamp into legacy `time.DateTime` Time format")
+			return nil, errors.WithStackTrace(err)
+		}
 	}
 
 	return &parsed, nil

util/time_test.go

@@ -19,9 +19,15 @@ func TestParseTimestamp(t *testing.T) {
 		wantErr bool
 	}{
 		{
-			"it should parse \"2023-12-19 10:38:44\" correctly",
+			"it should parse legacy firstSeenTag value \"2023-12-19 10:38:44\" correctly",
 			args{timestamp: aws.String("2023-12-19 10:38:44")},
-			&time.Time{2023, 12, 19, 10, 38, 44, 0, time.UTC},
+			newTime(time.Date(2023, 12, 19, 10, 38, 44, 0, time.UTC)),
+			false,
+		},
+		{
+			"it should parse RFC3339 firstSeenTag value \"2024-04-12T15:18:05Z\" correctly",
+			args{timestamp: aws.String("2024-04-12T15:18:05Z")},
+			newTime(time.Date(2024, 4, 12, 15, 18, 5, 0, time.UTC)),
 			false,
 		},
 	}
@@ -33,8 +39,12 @@ func TestParseTimestamp(t *testing.T) {
 				return
 			}
 			if !reflect.DeepEqual(got, tt.want) {
-				t.Errorf("ParseTimestamp() = %v, want %v", got, tt.want)
+				t.Errorf("ParseTimestamp() = %v, want %v", got, &tt.want)
 			}
 		})
 	}
 }
+
+func newTime(t time.Time) *time.Time {
+	return &t
+}