Skip to content

Conversation

@ajhyndman
Copy link

Hey, I took another attempt at fixing multer's support for AsyncLocalStorage.

I believe this addresses the feedback in #1119.

Fixes #814
Fixes #1046

@ajhyndman ajhyndman changed the title 814-preserve-async-hooks-context fix: Preserve async hooks context in supported environments Jul 24, 2022
@ejcheng ejcheng added the pr label Aug 15, 2022
@saifabusaleh
Copy link

Hi, Any ETA when this will be merged?

@Tyler8oliver
Copy link

Any ETA on release of this?

@rsauget
Copy link

rsauget commented Jan 10, 2024

Hi @LinusU 👋
We just experienced a production issue that would have been avoided with this PR, do you think we could merge it?

@xseman
Copy link

xseman commented May 21, 2024

Hi @LinusU @ajhyndman is there anything I can assist with to help move it forward?

@MagicIndustries
Copy link

We're experiencing the same issue, any updates on getting it merged?

@UlisesGascon UlisesGascon deleted the branch expressjs:main May 22, 2025 13:47
@UlisesGascon UlisesGascon reopened this May 22, 2025
@UlisesGascon UlisesGascon changed the base branch from master to v2 May 22, 2025 13:54
@UlisesGascon UlisesGascon added v2.x and removed pr labels May 22, 2025
@rmnmjw
Copy link

rmnmjw commented Sep 5, 2025

We had the same issue and it was quite time consuming to find the source of this bug. I added a workaround from a bug report's comments, but I'd appreciate if this pull request could be merged, so no-one else must experience this issue.

Any update on when this might be merged? I'd appreciate it a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

next() function not called in middleware, which breaks express-http-context (cls-hooked) CLS Context is lost after using multer middleware

9 participants