zcash-hackworks
/
bitcoin
mirror of https://github.com/zcash-hackworks/bitcoin.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Qt/RPCConsole: Truncate filtered commands to just the command name, rather than skip it entirely in history

This commit is contained in:
Luke Dashjr 2016-10-04 15:43:10 +00:00
parent d80a00660f
commit 1755c04576
1 changed files with 14 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/qt/rpcconsole.cpp
View File

@ -74,14 +74,14 @@ const QStringList historyFilter = QStringList()
<< "walletpassphrasechange"
<< "encryptwallet";
bool command_may_contain_sensitive_data(const QString cmd)
QString command_filter_sensitive_data(const QString cmd)
{
Q_FOREACH(QString unallowedCmd, historyFilter) {
if (cmd.trimmed().startsWith(unallowedCmd)) {
return true;
return unallowedCmd;
}
}
return false;
return cmd;
}
}
@ -779,10 +779,8 @@ void RPCConsole::on_lineEdit_returnPressed()
message(CMD_REQUEST, cmd);
Q_EMIT cmdRequest(cmd);
bool storeHistory = !command_may_contain_sensitive_data(cmd);
cmd = command_filter_sensitive_data(cmd);
if (storeHistory)
{
// Remove command, if already in history
history.removeOne(cmd);
// Append command to history
@ -792,7 +790,7 @@ void RPCConsole::on_lineEdit_returnPressed()
history.removeFirst();
// Set pointer to end of history
historyPtr = history.size();
}
// Scroll console view to end
scrollToEnd();
}