Skip to content

Conversation

@mar-cf
Copy link
Contributor

@mar-cf mar-cf commented Oct 7, 2025

No description provided.

@jmorrell-cloudflare
Copy link
Contributor

jmorrell-cloudflare commented Oct 7, 2025

I don't think multiple checksums are allowed: https://developers.cloudflare.com/r2/api/workers/workers-api-reference/#r2putoptions

Note
Only a single hashing algorithm can be specified at once.

That said, this would be better if we ever relaxed that requirement in the future. However I think it's harder to query, so I'm a little torn.

@mar-cf
Copy link
Contributor Author

mar-cf commented Oct 10, 2025

Alternatively, we could just set the first seen checked in a preset order.

And/or post under a different tag, checksum2 or something along those lines.

Only a single hashing algorithm can be specified at once.

That may be true, but I noticed that doesn't hold all the time.

@jmorrell-cloudflare LMK what you prefer

Copy link
Contributor

@jmorrell-cloudflare jmorrell-cloudflare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you still want to land this go for it

@mar-cf mar-cf marked this pull request as ready for review October 29, 2025 17:38
@mar-cf mar-cf requested review from a team as code owners October 29, 2025 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants