Skip to content

Commit 60bbda3

Browse files
committed
More error details
1 parent df507f3 commit 60bbda3

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

s3secrets-helper/s3/s3.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ func (c *Client) Get(key string) ([]byte, error) {
100100
}
101101
}
102102

103-
return nil, fmt.Errorf("Could not GetObject (%s) in bucket (%s): (%v)", key, c.bucket, err)
103+
return nil, fmt.Errorf("Could not GetObject (%s) in bucket (%s). Ensure your IAM Identity has s3:GetObject permission for this key and bucket. (%v)", key, c.bucket, err)
104104
}
105105
defer out.Body.Close()
106106
// we probably should return io.Reader or io.ReadCloser rather than []byte,
@@ -114,7 +114,7 @@ func (c *Client) Get(key string) ([]byte, error) {
114114
// Other errors result in false with an error.
115115
func (c *Client) BucketExists() (bool, error) {
116116
if _, err := c.s3.HeadBucket(context.TODO(), &s3.HeadBucketInput{Bucket: &c.bucket}); err != nil {
117-
return false, fmt.Errorf("Could not HeadBucket (%s): (%v)", c.bucket, err)
117+
return false, fmt.Errorf("Could not HeadBucket (%s). Ensure your IAM Identity has s3:ListBucket permission for this bucket. (%v)", c.bucket, err)
118118
}
119119
return true, nil
120120
}

0 commit comments

Comments
 (0)