liwanliangblog

1. 创作背景

使用deepin-terminal的时候,我发现一些小的问题。

在论坛的帖子(https://bbs.deepin.org/zh/post/224502)也总结反馈了这些问题

  • 终端标签变色
  • 远程管理标签标题跟右键选项显示异常
  • 截图发送到微信之后,再切换到终端时,双击复制无法将内容置入剪切板

经过我虚拟机全新安装deepin20.2.3后,几轮测试下来发现

终端标签变色的问题,应该只是个特性,发生在如果标签1有工作正在进行,或者有更新,比如sudo apt update或者ssh 连接超时被close时,而此时你正在标签2工作,那么这个变色会提醒你,标签1的工作结束了,或者有更新

截图发送到微信,然后切换到终端,双击复制,概率性无法复制的问题,应该是个稳定的bug。这个bug我也反馈给官方了,不清楚是否后续会解决。

今天来说的是远程管理

2. 远程管理的bug描述

deepin-terminal提供的远程管理,一般会要求我们输入名称,地址 ,帐号,密码这些。正常情况下,如果这些都是正常的,远程连接不会存在问题。连接上以后,标签标题会变成远程主机的用户@主机名,而终端右键也出现“上传文件”和“下载文件“的选项。

正常的情况我们就不说了,说说不正常的情况

2.1 点击远程连接时,突然不想连接了,或者密码错误了,ctrl+c终止连接失败

这种情况发生在,比如管理员重新设置了远程连接的密码,而你的远程连接并未更新,那么你使用这个连接远程主机时,必然会提示你输入密码,3此输入不正确,那么就无法登陆。如果你输入1次,发现不正确,直接ctrl+c了,那么此时标签的标题已经被设置为远程主机的主机名@主机地址了,而右键选项也多出了“上传文件”和“下载文件”的选项。

在未成功登陆远程主机的情况下,这种显示的变化,显然是个BUG。如何解决,且等下文分析

2.2 输错了服务器的IP,因为网络路由不正确,连接失败

如果你不小心直接输错了远程主机的IP地址,而且点击登陆时发现根本没法登陆,报一些比如No route to host的信息,告诉你没有到远程主机的路由。而此时标签的标题已经被设置为远程主机的主机名@主机地址了,而右键选项也多出了“上传文件”和“下载文件”的选项。

在未成功登陆远程主机的情况下,这种显示的变化,显然是个BUG。如何解决,且等下文分析

2.3 修改了ssh配置文件,比如修改了LogLevel导致退出远程主机后显示异常

如果你是一个不喜欢被繁杂的信息打扰的人,修改了/etc/ssh/ssh_config里面,把LogLevel设置为quit,那么当你使用deepi-terminal的远程管理登陆远程主机时,标签标题和右键选项均可以正常显示,但是你工作结束,通过exit命令,logout命令或者直接ctrl+d登出时,你会发现你的标签和右键选项还在。

在成功退出远程主机后,终端标签标题跟右键选项没有变化,这显然是个BUG。如何解决,且等下文分析

3. 如何解决

具体的解决,肯定是要修改代码的啦

3.1 先分析一下为何如此

分析deepin-terminal的源码得知,当你使用远程管理登陆远程节点时,deepi-terminal实际上载入和执行了一段标准的expect脚本,该脚本是一个免交互的脚本。deepin-terminal会把远程管理面板里面的信息同步到这段expect里面,替换到设置的“宏”,然后执行它们。

同时deepin-terminal启动了一个定时器触发,0.1s的时间,判断这个远程管理是否存在expect的进程信息,如果有就把当前标签的连接,设置为远程标签的格式,还有其他信息,比如编码,和右键选项。

试想,如果expect执行的ssh超时还没有到,标签已经变化,而ssh连接失败后,变化的标签会变回去吗?

答案是不一定的。这就是上面的2.1和2.2描述的bug原因。

至于2.3的描述,就要看deepin-terminal退出远程时,是如何把标签重新设置为本地的。它是通过截获收发数据的字符串,判断其是否包含"Connected to "和"closed."或者"Permission denied"来判断终端断开了远程连接。

inline void TermWidget::onTermWidgetReceivedData(QString value)
{
    /******** Modify by ut000610 daizhengwen 2020-05-25: quit download****************/
    if (value.contains("Transfer incomplete")) {
        QKeyEvent keyPress(QEvent::KeyPress, Qt::Key_C, Qt::ControlModifier);
        QApplication::sendEvent(focusWidget(), &keyPress);
    }    
    if (value.endsWith("\b \b #")) {                     // 结束的时候有乱码的话,将它清除
        QKeyEvent keyPress(QEvent::KeyPress, Qt::Key_U, Qt::ControlModifier);
        QApplication::sendEvent(focusWidget(), &keyPress);
    }    
    /********************* Modify by ut000610 daizhengwen End ************************/
    // 退出远程后,设置成false
    if ((value.contains("Connection to") && value.contains(" closed.")) || value.contains("Permission denied"))
        QTimer::singleShot(100, this, &TermWidget::onExitRemoteServer);
}

而我修改了LogLevel为quit,这个配置在我登出远程节点时,只显示Logout或者中文“登出”,”注销“等字样,并不存在”Connection to"的字样,因此终端就认定这个操作不是退出远程,自然它的标签标题跟右键菜单是不会变化的了。

3.2 再看看如何处理

直接看修改结果吧,对于2.1和2.2的bug描述,直接修改源码文件:deepin-terminal/src/assets/other/ssh_login.sh

修改为:

# Spawn and expect
if { $authentication == "no" } {
eval spawn $ssh_cmd $ssh_opt -t $remote_command exec \\\$SHELL -l
} else {
eval spawn $ssh_cmd $ssh_opt -i $private_key -t $remote_command exec \\\$SHELL -l
}


#fix bug:修复在登陆过程中直接ctlr+c终止登陆时,终端标签和右键选项变成远程的现象.
trap {
    send_user "Connection to closed.\n"
    exit
} SIGINT

if { [string length $password] } {
    expect {
        timeout {send_user "ssh connection time out, please operate manually\n"}
        -nocase "yes/no" {send "yes\r"; exp_continue}
        -nocase -re "password:|enter passphrase for key" { send "$encoded_password\r" }
        "No route to host" { send "Connection to closed.\n";exp_continue} #修改远程地址错误导致的终端标签和右键选项变成远程
        "Connection to closed." { exit }
    }
}

interact

大概意思就是通过expect捕获2.1和2.2描述的bug或者crtl+c信号,向终端抛出"Connection to closed."字样,让终端捕获这个字样后,把当前标签设置为本地标签即可,就不会显示为远程的样式了

而对于2.3的bug描述,还是修改/etc/ssh/ssh_config的LogLevel参数吧,修改为LogLevel=ERROR即可,另外一种动作比较大就是修改源码,把"登出",“注销”,"logout"等字样包含在对终端退出远程的判断,不过这个成本比较大了,意义也不是很大。暂且不做了。

4. 总结

花了2天时间摸索和测试吧,总的来说,达到了验证,修改和修复这些bug的目的。估计没有多少人会触发这样的bug,但是至少可以学习deepin-terminal的工作原理不是,赚到了。

分类:

技术点:

相关文章: