#1192 - Fix shared config loading of Duration Seconds from roles to use raw instead of int64

config/shared_config.go

@@ -613,8 +613,8 @@ func mergeSections(dst, src ini.Sections) error {
 
 		// role duration seconds key update
 		if srcSection.Has(roleDurationSecondsKey) {
-			roleDurationSeconds := srcSection.Int(roleDurationSecondsKey)
-			v, err := ini.NewIntValue(roleDurationSeconds)
+			roleDurationSeconds := srcSection.Raw(roleDurationSecondsKey)
+			v, err := ini.NewRawIntValue(roleDurationSeconds)
 			if err != nil {
 				return fmt.Errorf("error merging role duration seconds key, %w", err)
 			}

config/shared_config_test.go

@@ -9,6 +9,7 @@ import (
 	"strconv"
 	"strings"
 	"testing"
+	"time"
 
 	"github.com/aws/aws-sdk-go-v2/aws"
 	"github.com/aws/aws-sdk-go-v2/internal/ini"
@@ -302,6 +303,8 @@ func TestLoadSharedConfigFromSection(t *testing.T) {
 	if err != nil {
 		t.Fatalf("failed to load test config file, %s, %v", filename, err)
 	}
+	seconds, _ := time.ParseDuration("3601s")
+
 	cases := map[string]struct {
 		Profile  string
 		Expected SharedConfig
@@ -319,6 +322,10 @@ func TestLoadSharedConfigFromSection(t *testing.T) {
 			Profile:  "profile short_profile_name_first",
 			Expected: SharedConfig{Region: "short_profile_name_first_alt"},
 		},
+		"profile with durations": {
+			Profile:  "profile alt_durations",
+			Expected: SharedConfig{RoleDurationSeconds: &seconds},
+		},
 		"profile with partial creds": {
 			Profile:  "profile partial_creds",
 			Expected: SharedConfig{},

config/testdata/shared_config

@@ -13,6 +13,9 @@ aws_secret_access_key = SECRET
 [profile alt_profile_name]
 region = alt_profile_name_region
 
+[profile alt_durations]
+duration_seconds=3601
+
 [profile short_profile_name_first]
 region = short_profile_name_first_short
 

internal/ini/literal_tokens.go

@@ -203,6 +203,11 @@ func NewIntValue(i int64) (Value, error) {
 	return newValue(IntegerType, 10, []rune{rune(i)})
 }
 
+// NewRawIntValue returns an Integer Value type generated using a rune input.
+func NewRawIntValue(i []rune) (Value, error) {
+	return newValue(IntegerType, 10, i)
+}
+
 // Append will append values and change the type to a string
 // type.
 func (v *Value) Append(tok Token) {

internal/ini/visitor.go

@@ -253,6 +253,12 @@ func (t Section) Int(k string) int64 {
 	return t.values[k].IntValue()
 }
 
+
+// Raw returns a raw value at k
+func (t Section) Raw(k string) []rune {
+	return t.values[k].raw
+}
+
 // Float64 returns a float value at k
 func (t Section) Float64(k string) float64 {
 	return t.values[k].FloatValue()