alexvanin/restic
1
0
Fork 0
forked from TrueCloudLab/restic
Code Pull requests Activity

Don't crash if SecretString is uninitialized

Browse source
This commit is contained in:
Michael Eischer 2022-07-02 19:44:28 +02:00
parent bb5f196b09
commit cdaf9b4f26
2 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
internal/options/secret_string.go
View file

@ -13,12 +13,15 @@ func (s SecretString) GoString() string {
} }
func (s SecretString) String() string { func (s SecretString) String() string {
if len(*s.s) == 0 { if s.s == nil || len(*s.s) == 0 {
return `` return ``
} }
return `**redacted**` return `**redacted**`
} }
func (s *SecretString) Unwrap() string { func (s *SecretString) Unwrap() string {
if s.s == nil {
return ""
}
return *s.s return *s.s
} }

8
internal/options/secret_string_test.go
View file

@ -53,3 +53,11 @@ func TestSecretStringEmpty(t *testing.T) {
test.Equals(t, `""`, fmt.Sprintf("%#v", secret)) test.Equals(t, `""`, fmt.Sprintf("%#v", secret))
test.Equals(t, keyStr, secret.Unwrap()) test.Equals(t, keyStr, secret.Unwrap())
} }
func TestSecretStringDefault(t *testing.T) {
secretStruct := &secretTest{}
test.Equals(t, "", secretStruct.str.String())
test.Equals(t, `""`, secretStruct.str.GoString())
test.Equals(t, "", secretStruct.str.Unwrap())
}