Skip to content

Commit

Permalink
fix(lambda-promtail): Remove s3 bucket ownership verification (#15383)
Browse files Browse the repository at this point in the history
Co-authored-by: Christian Haudum <christian.haudum@gmail.com>
  • Loading branch information
milenpankov-bosch and chaudum authored Dec 23, 2024
1 parent a3808b6 commit bf73014
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 bf73014

Please sign in to comment.