Skip to content

Conversation

conico974
Copy link
Collaborator

Update the dependency for @opennextjs/aws to version 3.7.3, incorporating the latest changes and improvements from the release.

Copy link

changeset-bot bot commented Aug 4, 2025

🦋 Changeset detected

Latest commit: b1bdcbc

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@opennextjs/cloudflare Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

pkg-pr-new bot commented Aug 4, 2025

Open in StackBlitz

npm i https://pkg.pr.new/@opennextjs/cloudflare@838

commit: 692620d

@conico974 conico974 marked this pull request as draft August 4, 2025 08:12
@conico974
Copy link
Collaborator Author

Looks like something broke. It tries to apply some incorrect status code.
We may have to skip this release

@sommeeeer
Copy link
Collaborator

sommeeeer commented Aug 4, 2025

https://github.com/sommeeeer/opennextjs-aws/blob/2aa4b252dd83174c48bd951ded70304b7776ac0c/packages/open-next/src/http/openNextResponse.ts#L77

Sorry about that Nico, but this line needs to change. I only wanted to set the statusCode if its a number. In this case it can be NaN.

@vicb
Copy link
Contributor

vicb commented Aug 4, 2025

We also probably need to update the e2e tests with at least opennextjs/opennextjs-aws#941

@conico974 conico974 marked this pull request as ready for review August 4, 2025 14:48
@conico974 conico974 changed the title Bump @opennextjs/aws to version 3.7.3 Bump @opennextjs/aws to version 3.7.4 Aug 4, 2025
@vicb vicb force-pushed the conico/aws-3.7.3 branch from e4dfede to 692620d Compare August 4, 2025 16:39
@vicb
Copy link
Contributor

vicb commented Aug 4, 2025

Rebased on latest main, will merge when green.

Thanks for the PR!

@vicb vicb merged commit 90f451d into main Aug 4, 2025
7 checks passed
@vicb vicb deleted the conico/aws-3.7.3 branch August 4, 2025 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants