最近线上监控 SFTP 连接频繁爆表，通过重启某个系统，连接数迅速下降，系统就能恢复正常，初步判断是应用程序连接未关闭的问题导致的。

栈长通过 IDE 全局搜索排查，SFTP 连接使用了 jsch 包，确实有一些功能点使用了SFTP 连接而未关闭的情况，或者不在 finally 语句块中正常关闭。

整改上线后，SFTP 还是爆表……

事后运维心态都要崩了，运维主动写了个 SFTP 连接监控，当连接超过 5 分钟空闲时就主动断开。

但这只是临时的处理，真正的原因肯定还是应用程序没有正常关闭导致的，于是再认真排查下程序，终于找出了元凶。。

下面是示例代码：

Session session = null;
ChannelSftp channel = null;
try{for(...){...// 创建会话JSch jsch = new JSch();jsch.getSession(host, username);session = jsch.getSession(username, host, port);session.setPassword(password);session.connect();// 创建sftp连接channel = session.openChannel("sftp");channel.connect();...}
} catch(...){...
} finally{if (null != channel && channel.isConnected()) {channel.disconnect();}if (null != session && session.isConnected()) {session.disconnect();}
}

大家都看出问题了吗？

这程序我检查了 2 遍，排查了 3 天才解决了这个问题。

写出这代码，同事真是个人才啊！！！

乍一看，连接确实是关闭了啊，也确实是在 finally 语句块关闭，为什么还会有问题？

 原因就是在该死的 for 循环中创建连接的，虽然在 finally 中进行了关闭，但是连接变量在循环中进行重建和替换，所以关闭的永远只是最后一个连接。

而且，这还是个下载 Excel 明细的功能，数据很多的时候，一个操作就能导致连接瞬间爆表。

解决方案肯定是要把创建连接的部分拿到 for 循环前面去，连接创建一次就好了，可以反复使用。

另外，知道 JDK 7+ 中的 try-with-resources 语法的朋友可能会问，可以省略 finally 语句块吧，可以直接在 try(...) 中定义，它会自动关闭。

Really？建议还是仔细阅读下 try-with-resources 这篇文章吧，没看过的可以关注公众号Java技术栈进行搜索阅读。

这个 jsch 连接包还真不行，我们来看它的源码吧，不然又是一个坑你没商量的坑。

com.jcraft.jsch.Session：

public class Session impLements Runnable{
// http://ietf .org/internet-drafts/draft- ietf- secsh-ass ignednumbers-01. txt
static final int SSH_ MSG_ DISCONNECT= 1;
static final int SSH_ MSG_ IGNORE= 2;
static final int SSH_ MSG_ UNIMPLEMENTED= 3;
static final int SSH_ MSG_ DEBUG= 4;
static final int SSH_ MSG_ SERVICE REQUEST= 5;
static final int SSH_ MSG_ SERVICE_ ACCEPT= 6;
static final int SSH_ MSG_ KEXINIT= 20;

 com.jcraft.jsch.Channel：

pubLic abstract class Channel implements RunnabLe{static final int SSH_ MSG_ CHANNEL_ OPEN_ CONF IRMATION= 91 ;static final int SSH_ MSG_ CHANNEL_ OPEN_ FAIL URE= 92 ;static finaL int SSH_ MSG_ CHANNEL_ WINDOW_ ADJUST= 93;static finaL int SSH_ OPEN_ ADMINISTRATIVEL Y_ PROHIBITED= 1;static final int SSH_ OPEN_ CONNECT_ FAIL ED= 2;static final int SSH_ OPEN_ UNKNOWN_ CHANNEL_ TYPE= 3;static final int SSH_ OPEN_ RESOURCE_ SHORTAGE= 4;}

这两个类只实现了 Runnable 接口，没有实现 java.lang.AutoCloseable 接口，所以，它并不符合 try-with-resources 自动关闭的原则。

至此，线上 SFTP 连接爆表的问题终于解决了，可以安心睡个好觉了，同时，我也感觉我们的同事太牛逼了，又让我涨知识了。

 更多的java学习资料和趣事可以添加我的java峮：1080621881 。