-
-
Notifications
You must be signed in to change notification settings - Fork 518
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
fix: prevent instanceof
handler check failures between different MSW versions
#2349
Conversation
do you also want to use this pattern for filtering the websocket handlers? https://github.com/mswjs/msw/blob/main/src/core/ws/handleWebSocketEvent.ts#L33 |
@robbtraister, thanks, that's a fantastic point. Will adjust the solution. |
b6c5375
to
6b1904d
Compare
this is awesome! I really appreciate the effort on this. |
commit: |
@patricklafrance @robbtraister can you please give this preview release a try and let me know if the issue is fixed on your end? I would publish the changes then. Thanks. |
@kettanaito it works for me! 🎉 |
@kettanaito I am not sure what's going on, I'll have to investigate but at the moment NO requests at all are being intercepted, not even those for the HOST app request handlers. The issue might be on on my side though, I'll have to investigate later today. |
@patricklafrance, you mentioned you have different frontends loading different versions of MSW. If that's the case, none of the older versions will include I highly encourage you to update MSW across the board. I presume you are at v2 already so it should be a breeze. |
@kettanaito they are all on the same version in my sample application that I used for testing this new version. |
Released: v2.6.1 🎉This has been released in v2.6.1! Make sure to always update to the latest version ( Predictable release automation by @ossjs/release. |
Oh, that's interesting! Is that reproducible in the example you shared with me? |
@kettanaito I think it was something about how I installed the PR package (https://pkg.pr.new/msw@2349). Not sure what happened but i tried with |
It's fixed with |
instanceof
handlers check (dual publish) #2346Roadmap