Performance regression in 13.6.3 when using service workers that don't proxy every request

[ryanthemanuel]

Current behavior

We fixed an issue in 13.6.3 involving service workers to ensure that we could capture those requests properly for Test Replay and intercepts. In the process, we broke situations where service workers are present but don't necessarily proxy all network traffic from a given page. This will cause those requests that don't go through the service worker to time out at 2 seconds.

Desired behavior

No requests should time out.

Test code to reproduce

This can be reproduced by using a service worker that does the following:

self.addEventListener('fetch', function() {
  return
})

or

self.addEventListener('fetch', function(event) {
  if (event.request.url.includes('<some pattern>') {
     event.respondWith(<something>)
  }
  return
})

Note that this issue can be resolved for now if all traffic is routed through the service worker (i.e. the above turns into)

self.addEventListener('fetch', function(event) {
  event.respondWith(fetch(event.request))
})

or

self.addEventListener('fetch', function(event) {
  if (event.request.url.includes('<some pattern>') {
     event.respondWith(<something>)
  } else {
	 event.respondWith(fetch(event.request))
  }
})

Cypress Version

13.6.4

Node version

all

Operating System

all

Debug Logs

No response

Other

No response

[cypress-bot]

Released in 13.7.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v13.7.0, please open a new issue.