Skip to content

Allow Shell input to be overridden #612

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 1 commit into from
Jan 13, 2023

Conversation

marvin-enthus
Copy link
Contributor

Hi,

can we set Shell.input() to protected, so it can be overriden by other projects?

In https://github.com/fonimus/ssh-shell-spring-boot is a class ExtendedShell.java which needs to override and also access the input()-Method to inject input received via the SSH session properly.

Thanks,
Chris

- the ssh-shell-spring-boot extensions needs to extend to override the Shell.input()
  method to handle the input provided via the SSH session properly
@pivotal-cla
Copy link

@marvin-enthus Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-cla
Copy link

@marvin-enthus Thank you for signing the Contributor License Agreement!

@jvalkeal
Copy link
Contributor

I think it should be fine.

We'be been cleaning up Shell class quite a bit as it used to contain way too much logic and probably will go away at some point. Might be good idea to start thinking what users might need trying to modify anything Shell class has to get better api's and extension points.

@jvalkeal jvalkeal added this to the 3.0.0 milestone Jan 13, 2023
@jvalkeal jvalkeal changed the title Allow Shell.input() to be overridden in subclasses Allow Shell input to be overridden Jan 13, 2023
@marvin-enthus
Copy link
Contributor Author

Hi @jvalkeal Thanks' for your feedback. Just my cents regarding the api / extension: I'm also thinking of a 'web interface' to spring shell. It might have similar requirements like the spring-ssh extension: session management (as the web UI could be opened serveral times by different users), permission management and a different way to inject the input as it comes from the web console. Quick'n'dirty way would be using a web-based SSH terminal client 🤣

@jvalkeal jvalkeal merged commit 448c507 into spring-projects:main Jan 13, 2023
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.

3 participants