-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Investigate RangeError [ERR_SOCKET_BAD_PORT]: Port should be >= 0 and < 65536. Received type number (NaN)
within firefox system-tests
#30352
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
Comments
I am seeing this still on the #30324, so this is something we need to handle. Its difficult to reproduce, but we can likely handle the issue in the case a CDP address is undefined and send a retry event to relaunch the browser. We send the capability, but it does not get set |
@AtofStryker would you mind running tests with the Firefox preference |
@whimboo I can. I have it enabled in this circleCI run which is this commit. It happens sporadically so I might not be able to reproduce it on first try (also going to be some failures from printing additional logs) but ill see if I can get a run where it fails with this issue. |
Yes, understood. Let me know when it happened next and I'll take a look at it. Thanks! |
@AtofStryker did this issue actually happen on specific platforms only? Maybe there is some more data to localize the issue? |
I think so far I've only seen it on linux but there is a larger sample size there. Still working on getting a reproduction in a Circle run. |
Released in This comment thread has been locked. If you are still experiencing this issue after upgrading to |
Current behavior
Currently, sometimes in CI after the merging of #30250, firefox fails to start with the error
RangeError [ERR_SOCKET_BAD_PORT]: Port should be >= 0 and < 65536. Received type number (NaN)
. It looks like we are failing to parse the cdp port here. we should wrap this logic and handle the error to see whatmoz:debuggerAddress
capability returns within our system-tests to see if we can fix the error or handle it more gracefully.Below is a screenshot of the error, which can be seen in this job

Desired behavior
Once we see where the error is coming from, we can either resolve it completely or handle it more gracefully
Test code to reproduce
N/A
Cypress Version
N/A
Node version
N/A
Operating System
N/A
Debug Logs
No response
Other
No response
The text was updated successfully, but these errors were encountered: