Skip to content
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

wezterm-ssh: make SshPty fields public #6610

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

iamazy
Copy link

@iamazy iamazy commented Jan 28, 2025

I'm using wezterm-ssh, but some of its fields are inaccessible from outside. I want to make their visibility public

@iamazy iamazy changed the title wezterm-ssh: make structs/fields public wezterm-ssh: make SshPty fields public Jan 29, 2025
@wez
Copy link
Member

wez commented Feb 8, 2025

I'm not sure about this; it's making something of an API commitment, and it is revealing all of the internal implementation details for this struct.

Can you share more about what you want to do with those fields? Maybe there's a more supportable interface that could be provided instead?

@iamazy
Copy link
Author

iamazy commented Feb 9, 2025

Hi, @wez, Thanks for replying. I am building a ssh session manager that needs to implement some pty interfaces for ssh (see ssh-pty). However, the fields in SshPty in wezterm-ssh are not visible to the outside, so I hope to make them public

@wez
Copy link
Member

wez commented Feb 9, 2025

I understand that you want to make them public, but it is not a supportable change: it will either lock me into a specific set of implementation details, or will break you when I need to make a change in the future. If you can elaborate on specifically what you need from the interface, we can consider how to support that.

@iamazy
Copy link
Author

iamazy commented Feb 10, 2025

If you can elaborate on specifically what you need from the interface, we can consider how to support that.

I just want to obtain the mutable and immutable references of the reader/writer in SshPty

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.

2 participants