Skip to content

Commit

Permalink
fix(lambda-promtail): Remove s3 bucket ownership verification (grafan…
Browse files Browse the repository at this point in the history
…a#15383)

Co-authored-by: Christian Haudum <[email protected]>
  • Loading branch information
2 people authored and mveitas committed Jan 6, 2025
1 parent 24e25d2 commit 99fe867
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions tools/lambda-promtail/lambda-promtail/s3.go
Original file line number Diff line number Diff line change
Expand Up @@ -292,10 +292,9 @@ func processS3Event(ctx context.Context, ev *events.S3Event, pc Client, log *log
&s3.GetObjectInput{
Bucket: aws.String(labels["bucket"]),
Key: aws.String(labels["key"]),
ExpectedBucketOwner: aws.String(labels["bucket_owner"]),
})
if err != nil {
return fmt.Errorf("failed to get object %s from bucket %s on account %s, %s", labels["key"], labels["bucket"], labels["bucket_owner"], err)
return fmt.Errorf("failed to get object %s from bucket %s, %s", labels["key"], labels["bucket"], err)
}
err = parseS3Log(ctx, batch, labels, obj.Body, log)
if err != nil {
Expand Down

0 comments on commit 99fe867

Please sign in to comment.