This affects standalone mode.

In most of popular shells when you are entering a command and you press Ctrl+C (intr) the shell starts a new prompt below and the old text is visible above. This behavior is present when connecting over ssh to crsh.

This patch leverages jline feature which allows to enable interruption of readline() invocations.

CRaSH is used on multiple servers (we, specifically, use it as standalone), so it is needed to have a different host key per installation. For this, we find it somewhat a requirement to generate the host key per installation, either at boot or when someone first connects through SSH (Apache's SSHD examples are using SimpleGeneratorHostKeyProvider).

This patch adds the option to automatically generate a host key, by demand. A new option (ssh.keygen) in crash.properties was added.

I've made sure this patch is backward compatible. It's also OK to first use it with "ssh.keygen=true" and then disable it. The key is generated with PEMGeneratorHostKeyProvider so it should work with URLKeyPairProvider, once disabled.

I have also tried to make the change set as small as possible.

A re-implementation of the previously cancelled pull request (https://github.com/crashub/crash/pull/66), only now with CRaSHPlugin as a way to add Subsystem to the SSH life cycle. This is my naive approach, though it might be encapsulated better (like the CommandPlugin).

I noticed SpringBootstrap used to have code to obtain Spring beans of type CRaSHPlugin; however, the resulting list wasn't actually used and can understand why the code was removed at commit 064f6f131bc256d33f66d71f911e6477ff1c7ed5.

This seemed like a very useful feature since our app is heavily Spring based. It appeared extending ServiceLoaderDiscovery with the Spring logic made more sense and that did indeed work for our application.

Add exit status to ERROR when the shell process finishes with a response different than ShellResponse.OK

We can then use the exit status in shells scripts to get the command execution status

Here comes the promised guice integration module.

I wrote some integration tests using Arquillian to make sure it will keep working in the future.

I didn't put license header right now, would you mind merging my patch with license headers that does not contain "copyright eXo" in it ? If so, I'll update my pull request to include the license in it.

When using the dashboard command, a list of gc pools is printed to screen before the main table layout is shown.

This is caused because of (jvm.pools | { name -> .. })(). The closure, in this case, returns the name of the pool in question. This causes it to be printed to screen while the UIBuilder.table() is generated.

Explicitly placing "null" at the end of the closure makes Groovy consider this as the return value, thus nothing gets printed to screen before the dashboard is displayed.

• Add jmx info
• Add jmx ls

TODO :

• Correct Pb with jmx info --os NOVAL
• Correct Pb with jmx ls -inexistingOption => Must show help
• No jmx pwd implemented
• invoke command

Added crash.ssh.idle-timeout property to set custom idle timeout of the underlying SSH server Added crash.ssh.auth-timeout property to set custom authentication timeout of the underlying SSH server

The help [-h, --help] option, which is added by default to all commands, prevents one from adding a different "-h" option.

This patch reverses the order in which options are being searched. It makes it first search the option in the subordinate, which makes it possible for -h to take precedence over the help descriptor.

Normally, one does not need to override -h, but for the edge cases when someone does, this should provide the expected behavior.

serverKey resource wasn't being set when ssh.keypath property was used. This made the plugin basically ignore a key that is being set from a property.