Skip to content

Commit 242f472

Browse files
erikologicclaude
andcommitted
Fix uploadBlob rate limit type error with discriminated union
The RateLimitsConfig is a discriminated union - when enabled is false, the custom properties don't exist. Use the discriminant to narrow the type. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent ddb9972 commit 242f472

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

packages/pds/src/api/com/atproto/repo/uploadBlob.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
1+
import { DAY } from '@atproto/common'
12
import { UpstreamTimeoutError, parseReqEncoding } from '@atproto/xrpc-server'
23
import { BlobMetadata } from '../../../../actor-store/blob/transactor'
34
import { AppContext } from '../../../../context'
45
import { Server } from '../../../../lexicon'
56

67
export default function (server: Server, ctx: AppContext) {
8+
const rateLimits = ctx.cfg.rateLimits
79
server.com.atproto.repo.uploadBlob({
810
auth: ctx.authVerifier.authorizationOrUserServiceAuth({
911
checkTakedown: true,
@@ -13,8 +15,8 @@ export default function (server: Server, ctx: AppContext) {
1315
},
1416
}),
1517
rateLimit: {
16-
durationMs: ctx.cfg.rateLimits.repoUploadBlobRateLimitDuration,
17-
points: ctx.cfg.rateLimits.repoUploadBlobRateLimitPoints,
18+
durationMs: rateLimits.enabled ? rateLimits.repoUploadBlobRateLimitDuration : DAY,
19+
points: rateLimits.enabled ? rateLimits.repoUploadBlobRateLimitPoints : 1000,
1820
},
1921
handler: async ({ auth, input }) => {
2022
const requester = auth.credentials.did

0 commit comments

Comments
 (0)