Skip to content

Commit 88bff0f

Browse files
committed
fix: handling legacy requests
1 parent c3e9ce0 commit 88bff0f

File tree

2 files changed

+13
-7
lines changed

2 files changed

+13
-7
lines changed

src/middleware/withCarParkFetch.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,10 +54,12 @@ const MAX_BATCH_SIZE = 20 * 1024 * 1024
5454
*/
5555
export function withCarParkFetch (handler) {
5656
return async (request, env, ctx) => {
57-
// if carpark public bucket is not set, just use default
58-
if (!env.CARPARK_PUBLIC_BUCKET_URL) {
57+
const url = new URL(request.url)
58+
const legacyRequest = url.searchParams.get('legacyReq') === 'true'
59+
if (!env.CARPARK_PUBLIC_BUCKET_URL || legacyRequest) {
5960
return handler(request, env, { ...ctx, fetch: globalThis.fetch })
6061
}
62+
6163
const bucket = new TraceBucket(/** @type {import('@web3-storage/public-bucket').Bucket} */ (env.CARPARK))
6264
const bucketHandler = createHandler({ bucket, maxBatchSize: MAX_BATCH_SIZE })
6365

src/middleware/withLocator.js

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,12 +22,16 @@ import { trace } from '@opentelemetry/api'
2222
*/
2323
export function withLocator (handler) {
2424
return async (request, env, ctx) => {
25+
const url = new URL(request.url)
26+
const legacyRequest = url.searchParams.get('legacyReq') === 'true'
2527
const useIndexingService = isIndexingServiceEnabled(request, env)
2628
const span = trace.getActiveSpan()
2729
if (span) {
2830
span.setAttribute('useIndexingService', useIndexingService)
31+
span.setAttribute('legacyRequest', legacyRequest)
2932
}
30-
const client = useIndexingService
33+
34+
const client = !legacyRequest && useIndexingService
3135
? new Client({
3236
serviceURL: env.INDEXING_SERVICE_URL
3337
? new URL(env.INDEXING_SERVICE_URL)
@@ -37,10 +41,10 @@ export function withLocator (handler) {
3741
serviceURL: env.CONTENT_CLAIMS_SERVICE_URL
3842
? new URL(env.CONTENT_CLAIMS_SERVICE_URL)
3943
: undefined,
40-
carpark: env.CARPARK,
41-
carparkPublicBucketURL: env.CARPARK_PUBLIC_BUCKET_URL
42-
? new URL(env.CARPARK_PUBLIC_BUCKET_URL)
43-
: undefined
44+
...(legacyRequest ? {} : {
45+
carpark: env.CARPARK,
46+
carparkPublicBucketURL: env.CARPARK_PUBLIC_BUCKET_URL ? new URL(env.CARPARK_PUBLIC_BUCKET_URL) : undefined
47+
})
4448
})
4549

4650
const locator = Locator.create({ client })

0 commit comments

Comments
 (0)