-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Launching Webkit with docker against localhost throws Error: read ECONNRESET at TCP.onStreamRead #26847
Comments
@przemek-domain For your docker container, it looks like in your container for testing ( Let me know if that works and I can look at updating our docs site to be more clear about getting this to work in CI. |
@warrensplayer thank you for getting back to me. Unfortunately, the proposed solution does not work. I receive the same error
However, I found it works when I run my CRA app over HTTPS, i.e. |
You are right. I spoke too soon. I reverted my Dockerfile-test file and rebuilt my image. When run the following:
Here is the entire output and notice the
When I run this command
The only difference is appending Can you try the same thing as I did above and see if you get the same result? |
Thank you @warrensplayer, for investing time in this. It is a very interesting finding. I noticed a difference in how we set up the network. I updated the README with info on how I run the containers. When I try your way (no
When I add the magical
ℹ️ I tried the Debug connectionPlease see README for network details. To debug the connection between containers and ran the test-conatiner as
and tested connection
and with curl
SummaryIt looks like some network setup issue. Please provide me with exact commands you ran including build, run and test so I can reproduce exactly the same on my end. |
+1 I'm having the same issue, as soon as webkit attempts to interact with the app it throws Looking forward to a resolution on this because this is stopping us from running webkit tests in our pipeline |
@adam-wood-traka Are you trying to do the same thing as the original poster with sharing a network across docker images? If not, then please file a separate issue with a reproducible example for us to look at. |
@przemek-domain Thanks for the update detail on how you setup your docker containers. I had shared my exact commands above. I had just used I will pass this along to get some more help internally and will get back with you. |
@przemek-domain I was able to reproduce your same scenario using your updated steps. I couple of updates from our team. The There are a few issues relating to using webkit in a docker container: Your issue may be related. I am forwarding this ticket to the appropriate team. They will soon evaluate the priority of this ticket and consider their capacity to pick it up. Please note that this does not guarantee that this issue will be resolved. The ticket will indicate status changes during evaluation, so we ask that you please refrain from asking for updates. Thanks! |
@warrensplayer Yes exactly the same issue sharing a network across docker images. |
Having the same issue here. |
For us it also crashes with webkit (other browsers run fine) in a normal environmnet i. e. outside
setup: As the original poster implicitly mentioned, this only happens when going against localhost. Setting baseUrl to a proper domain fixes the issue and the page loads without any issues in the cypress tests. |
Current behavior
Setup
experimentalWebKitSupport: true
<app-server-ip> server.local
Current behaviour
When running a test against CI/CD (Jenkins) or locally, I face the following error.
Desired behavior
I would like Cypress to be able to run WebKit tests against localhost
Test code to reproduce
This repo with minimal setup to reproduce the issue https://github.com/psech/cypress-network-issue
Cypress Version
12.13.0
Node version
18.16.0
Operating System
macOS 13.3.1 arm64 + docker 23.0.5
or
Jenkins agent AWS EC2 c5.9xlarge + docker
Debug Logs
Other
The above setup helps to reproduce the #24704
A place in code I suspect to be involved
cypress/packages/server/lib/browsers/webkit.ts
Lines 132 to 143 in 7fa5103
The text was updated successfully, but these errors were encountered: