archive/rsnext
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

5MB -> 4MB body size limit (#26887)

Browse source 
This decreases the body size limit that triggers a warning from 5MB -> 4MB, which provides a little more wiggle room. Certain things like using base64 on body, headers, path, etc can cause the response to be larger than initially calculated. 

Initial PR: https://github.com/vercel/next.js/pull/26831
This commit is contained in:
Maia Teegarden 2021-07-02 16:17:56 -07:00 committed by GitHub
parent b4089998e6
commit b8f3a6d2d9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
errors/api-routes-body-size-limit.md
View file

@ -1,12 +1,12 @@
# API Routes Body Size Limited to 5mb
# API Routes Body Size Limited to 4MB
#### Why This Error Occurred
API Routes are meant to respond quickly and are not intended to support responding with large amounts of data. The maximum size of responses is 5 MB.
API Routes are meant to respond quickly and are not intended to support responding with large amounts of data. The maximum size of responses is 4 MB.
#### Possible Ways to Fix It
Limit your API Route responses to less than 5 MB. If you need to support sending large files to the client, you should consider using a dedicated media host for those assets. See link below for suggestions.
Limit your API Route responses to less than 4 MB. If you need to support sending large files to the client, you should consider using a dedicated media host for those assets. See link below for suggestions.
### Useful Links

4
packages/next/server/api-utils.ts
View file

@ -75,9 +75,9 @@ export async function apiResolver(
contentLength += Buffer.byteLength(args[0])
}
if (contentLength >= 5 * 1024 * 1024) {
if (contentLength >= 4 * 1024 * 1024) {
console.warn(
`API response for ${req.url} exceeds 5MB. This will cause the request to fail in a future version. https://nextjs.org/docs/messages/api-routes-body-size-limit`
`API response for ${req.url} exceeds 4MB. This will cause the request to fail in a future version. https://nextjs.org/docs/messages/api-routes-body-size-limit`
)
}

6
test/integration/api-support/test/index.test.js
View file

@ -398,17 +398,17 @@ function runTests(dev = false) {
expect(data).toBe('hi')
})
it('should warn if response body is larger than 5MB', async () => {
it('should warn if response body is larger than 4MB', async () => {
let res = await fetchViaHTTP(appPort, '/api/large-response')
expect(res.ok).toBeTruthy()
expect(stderr).toContain(
'API response for /api/large-response exceeds 5MB. This will cause the request to fail in a future version.'
'API response for /api/large-response exceeds 4MB. This will cause the request to fail in a future version.'
)
res = await fetchViaHTTP(appPort, '/api/large-chunked-response')
expect(res.ok).toBeTruthy()
expect(stderr).toContain(
'API response for /api/large-chunked-response exceeds 5MB. This will cause the request to fail in a future version.'
'API response for /api/large-chunked-response exceeds 4MB. This will cause the request to fail in a future version.'
)
})