Skip to content

add config option to disable logging of skipped output checksum validation #2974

Closed
@lucix-aws

Description

@lucix-aws

See #2971

We have an internal switch to disable logging of skipped output checksum validation, but when we generate checksum validation for supported operations, we just hardcode it to true:

LogValidationSkipped: true,

With the advent of crc64 in S3, people are now going to see this more often as other SDKs like the CLI now PutObject w/ crc64 by default. We should forward this config switch into client Options e.g.

package s3

type Options struct {
  // By default, the client will log a warning when an output checksum cannot be validated.
  // This can be disabled with this setting.
  DisableLogSkippedChecksumValidation bool
}

Metadata

Metadata

Assignees

Labels

feature-requestA feature should be added or improved.p1This is a high priority issue

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions