🐛 Bug Report — Runtime APIs - R2Conditional parses wildcard etag as strong etag

[harshal317]

Was looking into this issue here: cloudflare/workers-sdk#6411

Specifically when using the R2Conditional type when passing in an onlyIf option for R2_BINDING.put() we parse this as a strong etag regardless of the content

workerd/src/workerd/api/r2-bucket.c++

Lines 965 to 983 in 8cf4628

	R2Bucket::UnwrappedConditional::UnwrappedConditional(const Conditional& c)
	: secondsGranularity(c.secondsGranularity.orDefault(false)) {
	KJ_IF_SOME(e, c.etagMatches) {
	JSG_REQUIRE(!isQuotedEtag(e.value), TypeError,
	"Conditional ETag should not be wrapped in quotes (", e.value, ").");
	etagMatches = buildSingleStrongEtagArray(e.value);
	}
	KJ_IF_SOME(e, c.etagDoesNotMatch) {
	JSG_REQUIRE(!isQuotedEtag(e.value), TypeError,
	"Conditional ETag should not be wrapped in quotes (", e.value, ").");
	etagDoesNotMatch = buildSingleStrongEtagArray(e.value);
	}
	KJ_IF_SOME(d, c.uploadedAfter) {
	uploadedAfter = d;
	}
	KJ_IF_SOME(d, c.uploadedBefore) {
	uploadedBefore = d;
	}
	}

As a result when the conditional check is done in the R2 layer the condition which is meant to always be true when an object exists ends up being false.

Mostly creating this for tracking, I'll take a stab at updating the runtime to build a wildcard etag when we are passed a *