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

Multiple terminal support #9789

Open
andyrobreid opened this issue Feb 13, 2021 · 6 comments
Open

Multiple terminal support #9789

andyrobreid opened this issue Feb 13, 2021 · 6 comments
Assignees

Comments

@andyrobreid
Copy link

andyrobreid commented Feb 13, 2021

Problem

Multiple terminal support, currently I'm only aware that you can configure what terminal is executed via c.LabApp.terminado_settings={'shell_command':[<command>]} jupyter_notebook_config.py setting.

If I wish to use bash instead of PowerShell I have to change the config and restart Jupyter lab.

Proposed Solution

Ability to configure multiple terminals, having configurable properties shell_command, name, icon_path (icon request)

Additional context

Here is a discussion regarding the request.

@meeseeksmachine
Copy link
Contributor

This issue has been mentioned on Jupyter Community Forum. There might be relevant details there:

https://discourse.jupyter.org/t/multiple-terminal-shells-in-jupyterlab/3261/5

@dhirschfeld
Copy link
Member

Related: #4034

@dhirschfeld
Copy link
Member

Also, jupyter/notebook#3262

@andyrobreid
Copy link
Author

Also, jupyter/notebook#3262

Didn't think to look through the notebook repo issues. Thanks

@cameron-toy
Copy link
Contributor

cameron-toy commented May 6, 2021

Would the client or server be choosing the terminal application? It seems like it could be a security issue if the client can choose an arbitrary program to use as the terminal.

@dhirschfeld
Copy link
Member

I fail to see how it could be a security issue if the user can simply open a terminal and run any arbitrary program they want in any case?

If somehow it is still a concern (maybe terminals are disallowed for that reason) it could be mitigated by a c.ServerApp.terminals.allow_list configuration

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

No branches or pull requests

4 participants