Uploaded image for project: 'MINA SSHD'
  1. MINA SSHD
  2. SSHD-117

Ignore "keepalive@jcraft.com" in addition to "keepalive@openssh.com"

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Trivial
    • Resolution: Fixed
    • 0.5.0
    • 0.6.0
    • None
    • Any environment, JSch based client.

    Description

      It would be nice to also ignore jcraft keepalive messages.
      I think it is silly that this is how keepalive is done, but if MINA SSHD ignores one it may as well ignore the other.

      sshd-core/src/main/java/org/apache/sshd/server/session/ServerSession.java

      • if (req.equals("keepalive@openssh.com")) {
        + if (req.equals("keepalive@openssh.com") || req.equals("keepalive@jcraft.com")) {

      Personally I'd also like to see:

      • log.error("Unknown global request: {}", req);
        + log.warn("Unknown global request: {}", req);

      Attachments

        1. SSHD-117.patch
          1 kB
          Bill Kuker

        Activity

          People

            spearce Shawn Pearce
            bkuker1 Bill Kuker
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: