-
-
Notifications
You must be signed in to change notification settings - Fork 924
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
KITTY_LISTEN_ON and tcp sockets #2569
Comments
What's so poor about creating wrapper scripts or aliases? |
It would be so great if it did, with
Of course those are one way to solve a problem. But there are virtually no drawbacks in binding to a random unix socket, like e.g. neovim does. Secondly, awesome things like https://github.com/chmln/nvim-ctrl become impossible when users have to apply these workarounds at scale. |
Sorry what? Why should nvim-ctrl be impossible? Remote control is a security risk, so it is never going to be on by default. Given that, there is absolutely no point in listening to a random socket. Since all messages on said socket will be ignored by default. If you want to write an equivalent of nvim-ctrl, simply have it look for KITTY_LISTEN_ON in the environment and if not found to write to /dev/tty. This is what kitty @ itself does. And note that environment variables are not passed through ssh, not to mention that over ssh you typically cant access either unix or tcp sockets, unless they are explicitly forwarded. |
Thanks @kovidgoyal 🎉 |
It would be really nice if kitty created unix sockets by default in
/tmp
. I'm aware of #1330 but creating wrapper scripts and aliases are really poor workarounds. So I am currently trying to work withKITTY_LISTEN_ON
but for whatever reason it seems to be ignored. See attached screenshots below:With the env var:
![image](https://user-images.githubusercontent.com/11352152/79676856-6a8fdc80-81b8-11ea-978c-18f273740e6c.png)
Works as an option:
![image](https://user-images.githubusercontent.com/11352152/79676861-7f6c7000-81b8-11ea-82c0-ced72e43c727.png)
kitty config:
The text was updated successfully, but these errors were encountered: