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

new branch of tty.js #178

Open
ghostplant opened this issue May 21, 2018 · 3 comments
Open

new branch of tty.js #178

ghostplant opened this issue May 21, 2018 · 3 comments

Comments

@ghostplant
Copy link

ghostplant commented May 21, 2018

I think the licence of tty.js is good (MIT), which is better than GPLv3 that shellinabox uses, but there are many bugs in tty.js.

So I just put the branch link here since the super branch is no longer maintained for years.
https://github.com/ghostplant/webshell.
Welcome to post bugs and do continuous maintain of tty.js.

@risacher
Copy link

risacher commented Jun 1, 2018

I forked tty.js some time ago, mainly to replace the underlying components with updated versions: express 4, socket.io 1.x and later 2.x, and term.js with xterm.js from sourcelair. I also replaced the default shell with a script that ssh’s to localhost for security reasons. It’s https://github.com/risacher/ttyx/

@ghostplant
Copy link
Author

@risacher
I think it is also a good choice. This version is suitable for micro service without install OpenSSH inside docker image and starts both of services.

In terms of security, this version supports https as well so it is secure if using ca-certificate file correctly, and as for http, I bet none version of web shell are actually secure, whatever using ssh inside or not.

@risacher
Copy link

risacher commented Jun 4, 2018

I haven't tried to use ttyx as https directly, because I use it behind a reverse proxy and I terminate the TLS connection at the proxy.

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

No branches or pull requests

2 participants