Skip to content
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 MSW warning received under CircleCI test #1684

Open
KevinTriplett opened this issue Aug 15, 2024 · 7 comments
Open

fix MSW warning received under CircleCI test #1684

KevinTriplett opened this issue Aug 15, 2024 · 7 comments
Labels
task Code maintenance and refactoring

Comments

@KevinTriplett
Copy link
Contributor

I'm getting this error with CircleCi consistently:

[MSW] Error: captured a request without a matching request handler:

    • GET http://localhost/socket.io/
@tibetsprague
Copy link
Contributor

Yes something changed with CircleCI and we need to adjust our setup to work with sockets differently. I have not been able to look into it yet

@tibetsprague tibetsprague added the task Code maintenance and refactoring label Aug 15, 2024
@tibetsprague
Copy link
Contributor

p.s. I would recommend adding new issues to the Hylo Dev project to make sure we track them

@KevinTriplett
Copy link
Contributor Author

KevinTriplett commented Aug 16, 2024

I found this in the CircleCi documents:

Web sockets: CircleCI uses Pusher client libraries for WebSocket communication between the server and the browser, though for installs an internal server called slanger is used, so Pusher servers have no access to your instance of CircleCI, nor your source control system. This is how CircleCI updates the builds list dynamically, or shows the output of a build line-by-line as it occurs. CircleCI sends build statuses and lines of your build output through the web socket server (which unless you have configured your installation to run without SSL, is done using the same certs over SSL), so it is encrypted in transit.

@tibetsprague
Copy link
Contributor

this is one we are seeing a lot:
console.error
[MSW] Error: captured a request without a matching request handler:

    • GET http://localhost/socket.io/
  
  If you still wish to intercept this unhandled request, please create a request handler for it.
  Read more: https://mswjs.io/docs/getting-started/mocks

  at Object.error (node_modules/msw/node/lib/index.js:3168:13)
  at onUnhandledRequest (node_modules/msw/node/lib/index.js:7822:22)
  at node_modules/msw/node/lib/index.js:7858:13
  at fulfilled (node_modules/msw/node/lib/index.js:51:58)

@tibetsprague
Copy link
Contributor

@tibetsprague
Copy link
Contributor

or this thread? mswjs/msw#655

@KevinTriplett
Copy link
Contributor Author

Good links -- the puzzling aspect is why CircleCi throws the warning but our dev machines do not. Curious

@KevinTriplett KevinTriplett changed the title fix sails.io error with CircleCI fix MSW warning received under CircleCI test Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
task Code maintenance and refactoring
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants