8000 fix(middleware): copy wasm files for middleware by naikaayush · Pull Request #862 · opennextjs/opennextjs-aws · GitHub
[go: up one dir, main page]

Skip to content

fix(middleware): copy wasm files for middleware #862

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 19, 2025

Conversation

naikaayush
Copy link
Contributor

prisma's wasm was not being copied into the middleware directory, which was throwing errors when prisma was being used in the middleware with the authjs database adaptor.

added the copyMiddlewareResources which is already being used in createServerBundle

Copy link
changeset-bot bot commented May 13, 2025

🦋 Changeset detected

Latest commit: 1ae56e5

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

This PR includes changesets to release 3 packages
Name Type
@opennextjs/aws Patch
app-pages-router Patch
app-router 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

@naikaayush naikaayush changed the title fix: add copyMiddlewareResources to createMiddleware fix(middleware): copy wasm files for middleware May 13, 2025
8000
Copy link
pkg-pr-new bot commented May 13, 2025

Open in StackBlitz

pnpm add https://pkg.pr.new/@opennextjs/aws@862

commit: 1ae56e5

Copy link
Contributor
@conico974 conico974 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR.
I went ahead and made the requested updates since I hadn't heard back from you

@conico974 conico974 merged commit d6889d8 into opennextjs:main May 19, 2025
3 checks passed
@github-actions github-actions bot mentioned this pull request May 18, 2025
@naikaayush
Copy link
Contributor Author

@conico974 my apologies, thank you for this!

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.

2 participants
0