TypeError: This API doesn't support dates before 1687.

[sentry]

Sentry Issue: DIST-WORKER-PROD-2Y

TypeError: This API doesn't support dates before 1687.
  at None (worker.js:26142:39)
  at retryWrapper (worker.js:16910:28)
  at R2Provider.getFile (worker.js:26140:20)
  at getFile (worker.js:26411:14)
  at Object.handle (worker.js:13091:18)
...
(3 additional frame(s) were not displayed)

This issue was automatically created by Sentry via Node.js Worker Default Rule

[MattIPv4]

if-modified-since: 128 coming from bingbot is causing this, due to the 128 being passed into new Date (resulting in a date in the year 128) in

release-cloudflare-worker/src/utils/request.ts

Lines 60 to 67 in 7805f93

	function getDateFromHeader(dateString: string | null): Date | undefined {
	if (dateString === null) {
	return undefined;
	}
	const date = new Date(dateString);
	return !isNaN(date.getTime()) ? date : undefined;
	}

release-cloudflare-worker/src/utils/request.ts

Lines 32 to 33 in 7805f93

	export function parseConditionalHeaders(headers: Headers): ConditionalHeaders {
	const ifModifiedSince = getDateFromHeader(headers.get('if-modified-since'));

release-cloudflare-worker/src/middleware/r2Middleware.ts

Lines 117 to 119 in 7805f93

	result = await provider.getFile(r2Path, {
	conditionalHeaders: parseConditionalHeaders(request.headers),
	});

[MattIPv4]

As to why this is then causing the R2 binding to error looks to be related to the opposite end of the range that cloudflare/workerd#1687 is reporting issues with

[targos]

https://httpwg.org/specs/rfc9110.html#field.if-modified-since

A recipient MUST ignore the If-Modified-Since header field if the received field value is not a valid HTTP-date

128 is not a valid HTTP-date: https://httpwg.org/specs/rfc9110.html#http.date

[flakey5]

new Date() is very lenient with what it allows, I think we should switch to something similar to what undici does for parsing dates (src)