-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
New 3.9.2 not work on iPadOS, Safari #2975
Comments
Update:
sudo sed -i "s/req.headers\[\"sec-websocket-extensions\"\]/false/g" \
/usr/lib/code-server/out/node/routes/vscode.js
sudo sed -i "s/responseHeaders.push(\"Sec-WebSocket-Extensions/\/\/responseHeaders.push(\"Sec-WebSocket-Extensions/g" \
/usr/lib/code-server/out/node/routes/vscode.js |
using chrome on ipadOS gets the same error,just blank. But everything goes well on Android OS. |
Oh no! Thank you @springhack for the detailed report and the patches. And thank you @sh1luo for your extra notes as well! We'll see what we can do to fix this (and prevent it from happening again). cc @code-asher |
@jsjoeio I can confirm this happens in Safari on macOS 11. |
Just upgraded to
|
The root of the error seems to be:
Anybody else confirm? Running Edit: Also broken with the same error in |
sed: can't read /usr/lib/code-server/out/node/routes/vscode.js: No such file or directory. Oh,,,,,no effect |
@sh1luo I had a similar issue. Used: find / -name vscode.js to find out my files were in a slightly different path. The patch worked for me after adjusting the path to the vscode.js file. |
thank you.Using your method, after editing this file everything goes well on iOS. |
Took very long to find this issue post. This fix worked for me too. |
Thank You , I had the same issue. |
Thanks for the feedback! I'll pin the issue until we can fix it. |
Hey folks! Update here:
|
@jsjoeio was this only related to Safari? Because, I am seeing this running |
Ahh, my apologies @nodesocket - I overlooked your comment. We originally thought it was Safari only. Let me see if I can reproduce your issue using a similar setup. |
@nodesocket did any of the suggested workarounds above fix your issue by chance? |
@jsjoeio I got a Bash shell into the Kubernetes pod running kubectl exec -it code-server-6784d969bd-hldrf -- /bin/bash
coder@code-server-6784d969bd-hldrf:~$ You want me to just try running: sudo sed -i "s/req.headers\[\"sec-websocket-extensions\"\]/false/g" \
/usr/lib/code-server/out/node/routes/vscode.js
sudo sed -i "s/responseHeaders.push(\"Sec-WebSocket-Extensions/\/\/responseHeaders.push(\"Sec-WebSocket-Extensions/g" \
/usr/lib/code-server/out/node/routes/vscode.js |
Oh interesting! Yeah, if you're comfortable! I'm curious if that fixes it. |
Just running those commands did not fix it. Do I need to restart the process by chance? |
Ah, shoot. I think so! Worth a shot! |
@nodesocket Bummer. Well, we will look into this further. Thanks for the screenshot! |
@nodesocket I feel like it makes sense to move this into a separate issue. Doesn't seem like your issue is the same as the OP. |
Hello, I know issue is closed but issues #3004 #3017 and #3014 are related with this one. Basically regardless of middlewares (proxy, ingress, etc...) when user opens up code-server in browser, and if browser doesn't properly support websocket zlib compression and/or negotiation it fails. (Websocket Error 1006 in Safari) Just my 2 cents about what you guys can do about it:
|
I also would like to leave a comment. On my iPhone with iOS 14.4.2 - fix worked. |
@pvtmert thanks for the suggestions! I'll let @code-asher weigh in if he has any thoughts. But yeah, we're also planning to add e2e testing for Safari to catch some of these things sooner. @vendvahk thanks for the report! Are there any logs ( |
Instead of recreating the pod; try restarting the s6 service after applying the patch and while still in the pod. s6-svc -r /var/run/s6/services/code-server Worked for me. |
Edit 1: Chrome JS Console on iPad shows following:
Edit 2: I did some more investigation with Safari. Other than 404 messages as in #1640, only major diference is:
I think it's easier for me to just update iOS to latest version. |
@grewhit25 How do I restart the service, once I have a terminal inside the pod? Also, not following what you mean by
|
@nodesocket I am using nicholaswilde/code-server helm chart. The container for this chart is supervised by s6 overlay. s6-svc -r /var/run/s6/services/code-server Is a command available while exec'd into an s6 supervised container to manage a service. Did you try running the above command as shown after applying the patch? |
|
We've published Feel free to post on this discussion. |
I can confirm v3.9.3 works for me: |
Version 3.9.3 work for me, iPadOS 14.4.2, install using deb package |
Awesome! So glad to hear that. Thank you @grewhit25 and @springhack ! |
OS/Web Information
code-server --version
: 3.9.2 109d2ceSteps to Reproduce
Expected
Actual
Screenshot
Notes
Seems cause by
permessage-deflate
extension, maybe add a option via command line is better?This issue can be reproduced in VS Code:No
The text was updated successfully, but these errors were encountered: