Fix: Aws::S3::Errors::InvalidRequest (MAYBE-RAILS-CT)

[revise-dev]

The error "Aws::S3::Errors::InvalidRequest: You can only specify one checksum at a time" occurs because the Cloudflare R2 storage configuration is attempting to specify multiple checksum validations simultaneously. While Cloudflare R2 uses the S3 API, it has slightly different requirements for checksums.

The original configuration had two checksum-related settings:

request_checksum_calculation: "when_required"
response_checksum_validation: "when_required"

These settings were likely intended to ensure data integrity during file uploads and downloads. However, they conflict with each other when making requests to the S3-compatible API, as only one checksum validation can be active at a time.

The fix is to simplify the checksum configuration by:

1. Removing both existing checksum settings
2. Adding a single checksum: nil setting to explicitly disable checksums

This change maintains compatibility with Cloudflare R2 while preventing the multiple checksum error. The nil value tells Active Storage not to include any checksum in the request, which is a valid configuration for S3-compatible storage services.

Testing this change is important since it affects file upload functionality. The existing test suite indicates that file attachments are used throughout the application, particularly for user profile images. The tests verify that files can be uploaded and stored correctly without checksum errors.

Tip

You can make revisions or ask questions of Revise.dev by using /revise in any comment or review!

• /revise Add a comment above the method to explain why we're making this change.
• /revise Why did you choose to make this change specifically?

Important

If something doesn't look right, click to retry this interaction.

[zachgoll]

Fixed via 0476f25