Skip to content

kitten ssh breaks '^ ' mapping in ZSH after returning from session #6780

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

Closed
mehalter opened this issue Nov 2, 2023 · 5 comments
Closed

kitten ssh breaks '^ ' mapping in ZSH after returning from session #6780

mehalter opened this issue Nov 2, 2023 · 5 comments
Labels

Comments

@mehalter
Copy link

mehalter commented Nov 2, 2023

Describe the bug

If I have something bound to ctrl-space in ZSH ('^ ') and I use kitten ssh to ssh to a host it breaks my original mapping.

This does not even have to be a complete, successful ssh connection. If you use kitten ssh to access a machine with a password prompt (not password-less login) and quit with <C-c> several times, the mapping is still broken. This rules out that conflicting ZSH bindings could be coming into play.

I have tested this across many terminals and cannot replicate the issue. I also cannot replicate the issue when I just use ssh and do not use kitten ssh

To Reproduce
Steps to reproduce the behavior:

  1. Start a clean kitty session with no config: kitty --config=NONE
  2. Start a clean zsh session with no config: zsh -f
  3. Set a ctrl+space binding: bindkey -s '^ ' TEST
  4. Test that it works by pressing ctrl+space
  5. SSH to a remote machine with kitten ssh
    • this doesn't have to complete, you can ctrl+c back to the prompt from the password prompt
  6. Close the session and return to your machine
  7. Press ctrl+space and see that "TEST" is not printed
  8. run bindkey and see that the binding is still set

Environment details
Fully and completely clean environment with v0.30.1 to rule out all possible conflicts

@mehalter
Copy link
Author

mehalter commented Nov 3, 2023

Thanks for the quick fix! 😍

@mehalter
Copy link
Author

mehalter commented Nov 4, 2023

@kovidgoyal this problem is not fully resolved. This should be reopened. If you do ctrl-c in the middle of making an ssh connection then the mapping still gets messed up. It does seem to be resolved in the case where a full ssh connection is made and then closed.

@kovidgoyal
Copy link
Owner

bc1da55

@mehalter
Copy link
Author

mehalter commented Nov 5, 2023

Thanks! I'll test in the next day or two! 🤞 Very excited for this to be resolved after many months! Thanks so much for quick fixes after reporting with a clear issue.

@mehalter
Copy link
Author

mehalter commented Nov 7, 2023

I can confirm that this appears to be fully fixed. Thanks again!

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

No branches or pull requests

2 participants