Skip to content

added handling for X11 Forwarding #8

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 2 commits into from
Sep 26, 2023
Merged

Conversation

bepeit01
Copy link

This changes enables X11 Forwarding.
The routines ForwardToRemoteX11 and ForwardToRemote could possibly be merged to on, but i couldn't not test again forwarding, therefore i did not touch it.

@vdombrovski
Copy link
Collaborator

Thank you for the contribution @bepeit01 ! Are we the upstream now? 😂

@vdombrovski vdombrovski merged commit 0caa216 into BSONetwork:master Sep 26, 2023
@bepeit01
Copy link
Author

No, you are more advanced than upstream ;)
I need the feature from this PR: moul#469 which is open since july and i didn't want to create a PR to a PR...

@vdombrovski
Copy link
Collaborator

Yeah about that, I'm not really sure that moul is that active / motivated by the project anymore, there is definitely interest around the project however in the other forks, I know that https://github.com/systemmonkey42/sshportal, and https://github.com/alterway/sshportal are really active and are cherry-picking some of our improvements too, which is great.

Quick note, we're merging our own improvements (which are not always bug fixes, more like modifications) into this fork, and while we're trying to keep them generic, some of them might not suit your needs.

@bepeit01
Copy link
Author

Same here... We have a different fork at work where we add our improvements/adaptions as we need it. And i use my fork to add the generic once back to the community,
Actually we have chosen your fork as basis for out internal one because you were one of the most active once and a lot of bugs were already fixed.

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