Skip to content

Commit b8d0f98

Browse files
author
Alan Shaw
authored
fix: include R2 key in error message (#2326)
Include the R2 key in the error message for better visibility when debugging. We're getting rate limits for put with the same key...
1 parent aaee05d commit b8d0f98

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

packages/api/src/utils/uploader/r2-uploader.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ export class R2Uploader {
5454
return { key, url }
5555
} catch (cause) {
5656
// @ts-expect-error wen ts understand Error object?
57-
throw new Error('Failed to upload CAR to R2', { cause })
57+
throw new Error(`Failed to upload CAR to R2: ${key}`, { cause })
5858
}
5959
}
6060

@@ -74,7 +74,7 @@ export class R2Uploader {
7474
})
7575
} catch (cause) {
7676
// @ts-expect-error error.cause is legit.
77-
throw new Error('Failed to write satnav index to R2', { cause })
77+
throw new Error(`Failed to write satnav index to R2: ${key}`, { cause })
7878
}
7979
}
8080

@@ -91,7 +91,9 @@ export class R2Uploader {
9191
})
9292
} catch (cause) {
9393
// @ts-expect-error error.cause is legit.
94-
throw new Error('Failed to write dudewhere index to R2', { cause })
94+
throw new Error(`Failed to write dudewhere index to R2: ${key}`, {
95+
cause,
96+
})
9597
}
9698
}
9799
}

0 commit comments

Comments
 (0)