Skip to content

Commit

Permalink
Merge branch 'main' into fix/noop-empty-revalidate-tags
Browse files Browse the repository at this point in the history
  • Loading branch information
pieh authored Dec 19, 2024
2 parents 28339a3 + f00b542 commit d0a7f47
Show file tree
Hide file tree
Showing 6 changed files with 31 additions and 12 deletions.
2 changes: 1 addition & 1 deletion .release-please-manifest.json
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
{
".": "5.9.0"
".": "5.9.1"
}
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
# Changelog

## [5.9.1](https://github.com/opennextjs/opennextjs-netlify/compare/v5.9.0...v5.9.1) (2024-12-18)


### Bug Fixes

* use version of htmlrewriter which does not make use of asyncify, which looks to have a potential memory leak under high load ([#2721](https://github.com/opennextjs/opennextjs-netlify/issues/2721)) ([4d7ad97](https://github.com/opennextjs/opennextjs-netlify/commit/4d7ad97c3f16b01000989bde06352f86b5d526ba))

## [5.9.0](https://github.com/opennextjs/opennextjs-netlify/compare/v5.8.1...v5.9.0) (2024-12-09)


Expand Down
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@netlify/plugin-nextjs",
"version": "5.9.0",
"version": "5.9.1",
"description": "Run Next.js seamlessly on Netlify",
"main": "./dist/index.js",
"type": "module",
Expand Down
6 changes: 4 additions & 2 deletions tests/e2e/export.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -64,8 +64,10 @@ test.describe('next/image is using Netlify Image CDN', () => {

expect(nextImageResponse.status()).toBe(200)
// ensure next/image is using Image CDN
// source image is jpg, but when requesting it through Image CDN avif will be returned
expect(await nextImageResponse.headerValue('content-type')).toEqual('image/avif')
// source image is jpg, but when requesting it through Image CDN avif or webp will be returned
expect(['image/avif', 'image/webp']).toContain(
await nextImageResponse.headerValue('content-type'),
)

await expectImageWasLoaded(page.locator('img'))
})
Expand Down
22 changes: 16 additions & 6 deletions tests/e2e/simple-app.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -119,8 +119,10 @@ test.describe('next/image is using Netlify Image CDN', () => {

expect(nextImageResponse.status()).toBe(200)
// ensure next/image is using Image CDN
// source image is jpg, but when requesting it through Image CDN avif will be returned
expect(await nextImageResponse.headerValue('content-type')).toEqual('image/avif')
// source image is jpg, but when requesting it through Image CDN avif or webp will be returned
expect(['image/avif', 'image/webp']).toContain(
await nextImageResponse.headerValue('content-type'),
)

await expectImageWasLoaded(page.locator('img'))
})
Expand All @@ -142,7 +144,9 @@ test.describe('next/image is using Netlify Image CDN', () => {
)

expect(nextImageResponse.status()).toBe(200)
expect(await nextImageResponse.headerValue('content-type')).toEqual('image/avif')
expect(['image/avif', 'image/webp']).toContain(
await nextImageResponse.headerValue('content-type'),
)

await expectImageWasLoaded(page.locator('img'))
})
Expand All @@ -164,7 +168,9 @@ test.describe('next/image is using Netlify Image CDN', () => {
)

expect(nextImageResponse.status()).toBe(200)
expect(await nextImageResponse.headerValue('content-type')).toEqual('image/avif')
expect(['image/avif', 'image/webp']).toContain(
await nextImageResponse.headerValue('content-type'),
)

await expectImageWasLoaded(page.locator('img'))
})
Expand All @@ -183,7 +189,9 @@ test.describe('next/image is using Netlify Image CDN', () => {
)

expect(nextImageResponse?.status()).toBe(200)
expect(await nextImageResponse.headerValue('content-type')).toEqual('image/avif')
expect(['image/avif', 'image/webp']).toContain(
await nextImageResponse.headerValue('content-type'),
)

await expectImageWasLoaded(page.locator('img'))
})
Expand All @@ -203,7 +211,9 @@ test.describe('next/image is using Netlify Image CDN', () => {
)

expect(nextImageResponse.status()).toEqual(200)
expect(await nextImageResponse.headerValue('content-type')).toEqual('image/avif')
expect(['image/avif', 'image/webp']).toContain(
await nextImageResponse.headerValue('content-type'),
)

await expectImageWasLoaded(page.locator('img'))
})
Expand Down

0 comments on commit d0a7f47

Please sign in to comment.