Skip to content

Support unix-based X11 server connections as well as TCP-based ones #456

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

Merged
merged 1 commit into from
Mar 31, 2025

Conversation

karcherm
Copy link
Contributor

Typically, on unix systems, local X11 is connected through the socket /tmp/.X11-unix/X0, as indicated by the missing host name part in DISPLAY=:0 or DISPLAY=:0.0. WSLg (the X11 implementation provided with WSL in Microsoft Windows) does not listen on the TCP socket localhost:6000, but on the Unix socket only.

So this pull request enables SSH X11 forwarding to connect to the unix domain socket if the X server host name is set to the empty string.

@WebFreak001
Copy link
Owner

you should add some mention about this in the user facing config documentation: https://github.com/WebFreak001/code-debug/blob/eb9f5e4dd7dc64f6e6e8a40115d47d30e522525c/package.json

Typically, on unix systems, local X11 is connected through the socket `/tmp/.X11-unix/X0`, as indicated by the missing host name part in `DISPLAY=:0` or `DISPLAY=:0.0`. WSLg (the X11 implementation provided with WSL in Microsoft Windows) does not listen on the TCP socket localhost:6000, but on the Unix socket only.

So this commit enables SSH X11 forwarding to connect to the unix domain socket if the X server host name is set to the empty string.
@WebFreak001 WebFreak001 merged commit 5bbe034 into WebFreak001:master Mar 31, 2025
4 checks passed
@GitMensch
Copy link
Collaborator

Oh, is that possibly a step for the next release? Note: the last one is over a year old and we have a bunch of bugfixes and features, see Changelog (this one still needs to be added) from the diff v0.27.0...master (note: #448 is mostly ready to get in as well).

So @WebFreak001: any chance for a release (which is still done and then published to the stores by yourself), soon?

@WebFreak001
Copy link
Owner

the store publishing should be part of GH actions, no?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants