Corrected:\n- bug on ipv6\n- bug on saving settings
This commit is contained in:
parent
7d59b24df0
commit
571da3d8a3
@ -52,7 +52,7 @@ void MainWindow::download()
|
|||||||
{
|
{
|
||||||
info(DEBUGMACRO, "Adding user to server (user@server");
|
info(DEBUGMACRO, "Adding user to server (user@server");
|
||||||
this->downloading.user = this->connexion.user;
|
this->downloading.user = this->connexion.user;
|
||||||
server = this->connexion.user + "@" + this->downloading.server;
|
server = this->connexion.user + "@" + "[" + this->downloading.server + "]";
|
||||||
env.insert("RSYNC_PASSWORD", this->downloading.password); // Add an environment variable
|
env.insert("RSYNC_PASSWORD", this->downloading.password); // Add an environment variable
|
||||||
this->downloading.process->setProcessEnvironment(env);
|
this->downloading.process->setProcessEnvironment(env);
|
||||||
}
|
}
|
||||||
@ -102,6 +102,7 @@ void MainWindow::readRsyncOutput()
|
|||||||
{
|
{
|
||||||
list.clear();
|
list.clear();
|
||||||
line = QString::fromUtf8(this->downloading.process->readLine());
|
line = QString::fromUtf8(this->downloading.process->readLine());
|
||||||
|
info(DEBUGMACRO, "downloading progress : " + line);
|
||||||
if (line.isEmpty())
|
if (line.isEmpty())
|
||||||
{
|
{
|
||||||
break;
|
break;
|
||||||
|
@ -370,13 +370,14 @@ void MainWindow::populateList(QString server, uint port)
|
|||||||
QGuiApplication::setOverrideCursor(Qt::WaitCursor);
|
QGuiApplication::setOverrideCursor(Qt::WaitCursor);
|
||||||
|
|
||||||
// verify if server is in history
|
// verify if server is in history
|
||||||
this->settings.beginGroup("connexion/server");
|
this->settings.beginGroup("connexion/Servers");
|
||||||
if (this->settings.contains(server))
|
if (this->settings.contains(server))
|
||||||
{
|
{
|
||||||
|
this->settings.beginGroup(server);
|
||||||
info(DEBUGMACRO, "Server configuration exists in settings");
|
info(DEBUGMACRO, "Server configuration exists in settings");
|
||||||
|
|
||||||
// server is in history => setting port value
|
// server is in history => setting port value
|
||||||
port = this->settings.value(server).toUInt();
|
port = this->settings.value("port").toUInt();
|
||||||
this->connexion.ipversion = this->settings.value("ipversion").toUInt();
|
this->connexion.ipversion = this->settings.value("ipversion").toUInt();
|
||||||
ui->portEdit->setText(QString::number(port));
|
ui->portEdit->setText(QString::number(port));
|
||||||
this->connexion.port = port;
|
this->connexion.port = port;
|
||||||
@ -384,8 +385,10 @@ void MainWindow::populateList(QString server, uint port)
|
|||||||
ok = true;
|
ok = true;
|
||||||
//display list of services
|
//display list of services
|
||||||
//listServices(); // TODO clear in listServices
|
//listServices(); // TODO clear in listServices
|
||||||
|
this->settings.endGroup();
|
||||||
}
|
}
|
||||||
this->settings.endGroup();
|
this->settings.endGroup();
|
||||||
|
this->settings.endGroup();
|
||||||
}
|
}
|
||||||
if (port != this->connexion.port)
|
if (port != this->connexion.port)
|
||||||
{
|
{
|
||||||
@ -401,7 +404,7 @@ void MainWindow::populateList(QString server, uint port)
|
|||||||
if (this->connexion.ipversion == 0)
|
if (this->connexion.ipversion == 0)
|
||||||
{
|
{
|
||||||
this->connexion.ipversion = validateServer(server);
|
this->connexion.ipversion = validateServer(server);
|
||||||
info(DEBUGMACRO, QString::number(this->connexion.ipversion));
|
info(DEBUGMACRO, "IP version : " + QString::number(this->connexion.ipversion));
|
||||||
}
|
}
|
||||||
|
|
||||||
if (this->connexion.ipversion != 0)
|
if (this->connexion.ipversion != 0)
|
||||||
@ -411,17 +414,27 @@ void MainWindow::populateList(QString server, uint port)
|
|||||||
info(DEBUGMACRO,"Saving server configuration in settings");
|
info(DEBUGMACRO,"Saving server configuration in settings");
|
||||||
|
|
||||||
// storing serverURL and port in settings
|
// storing serverURL and port in settings
|
||||||
this->settings.setValue(server, port);
|
this->settings.beginGroup("connnexion");
|
||||||
this->settings.setValue(server, this->connexion.ipversion);
|
this->settings.beginGroup("Servers");
|
||||||
|
this->settings.beginGroup(server);
|
||||||
|
this->settings.setValue("port", port);
|
||||||
|
this->settings.setValue("ipversion", this->connexion.ipversion);
|
||||||
|
this->settings.endGroup();
|
||||||
|
this->settings.endGroup();
|
||||||
|
this->settings.endGroup();
|
||||||
|
|
||||||
this->settings.sync();
|
this->settings.sync();
|
||||||
this->downloading.server = server;
|
this->downloading.server = server;
|
||||||
|
this->downloading.port = port;
|
||||||
this->downloading.ipversion = this->connexion.ipversion;
|
this->downloading.ipversion = this->connexion.ipversion;
|
||||||
|
|
||||||
// storing in history of combobox
|
// storing in history of combobox
|
||||||
ui->khistorycombobox->addToHistory(server);
|
ui->khistorycombobox->addToHistory(server);
|
||||||
|
|
||||||
// load and display rsync services of the rsync server
|
// load and display rsync services of the rsync server
|
||||||
|
}else
|
||||||
|
{
|
||||||
|
info(DEBUGMACRO, "ERROR : Unknown IP version");
|
||||||
}
|
}
|
||||||
}else
|
}else
|
||||||
{
|
{
|
||||||
@ -429,7 +442,8 @@ void MainWindow::populateList(QString server, uint port)
|
|||||||
}
|
}
|
||||||
listServices();
|
listServices();
|
||||||
|
|
||||||
this->settings.beginGroup("Hidden/" + server);
|
this->settings.beginGroup("Hidden");
|
||||||
|
this->settings.beginGroup(server);
|
||||||
hidden = this->settings.allKeys();
|
hidden = this->settings.allKeys();
|
||||||
if (hidden.count() > 0)
|
if (hidden.count() > 0)
|
||||||
{
|
{
|
||||||
@ -444,6 +458,7 @@ void MainWindow::populateList(QString server, uint port)
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
this->settings.endGroup();
|
this->settings.endGroup();
|
||||||
|
this->settings.endGroup();
|
||||||
// clearing listview
|
// clearing listview
|
||||||
ui->treeWidget->clear();
|
ui->treeWidget->clear();
|
||||||
QGuiApplication::restoreOverrideCursor(); //setting cursor to default
|
QGuiApplication::restoreOverrideCursor(); //setting cursor to default
|
||||||
@ -509,7 +524,7 @@ bool MainWindow::testServerPresence(QString service, bool askPassword)
|
|||||||
//list services of the rsync server
|
//list services of the rsync server
|
||||||
void MainWindow::listServices()
|
void MainWindow::listServices()
|
||||||
{
|
{
|
||||||
QString cmd;
|
QString cmd = "/usr/bin/rsync";
|
||||||
QStringList param;
|
QStringList param;
|
||||||
QString line;
|
QString line;
|
||||||
QString errorRsync;
|
QString errorRsync;
|
||||||
@ -524,7 +539,6 @@ void MainWindow::listServices()
|
|||||||
// clearing listwidget
|
// clearing listwidget
|
||||||
ui->listWidget->clear();
|
ui->listWidget->clear();
|
||||||
|
|
||||||
cmd = "/usr/bin/rsync";
|
|
||||||
if (this->connexion.ipversion == 0)
|
if (this->connexion.ipversion == 0)
|
||||||
{
|
{
|
||||||
this->connexion.ipversion = whatIpVersion(this->connexion.server);
|
this->connexion.ipversion = whatIpVersion(this->connexion.server);
|
||||||
@ -538,7 +552,12 @@ void MainWindow::listServices()
|
|||||||
{
|
{
|
||||||
param << "-" + QString::number(this->connexion.ipversion).trimmed();
|
param << "-" + QString::number(this->connexion.ipversion).trimmed();
|
||||||
server = "[" + this->connexion.server + "]";
|
server = "[" + this->connexion.server + "]";
|
||||||
|
}else
|
||||||
|
{
|
||||||
|
error(tr("Error : Unknown IP version"));
|
||||||
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
param << "--contimeout=20" << "--port=" + QString::number(this->connexion.port) << server + "::";
|
param << "--contimeout=20" << "--port=" + QString::number(this->connexion.port) << server + "::";
|
||||||
myProcess = new QProcess(this);
|
myProcess = new QProcess(this);
|
||||||
|
|
||||||
@ -583,7 +602,7 @@ void MainWindow::listServices()
|
|||||||
// connect to rsync server to get list of files
|
// connect to rsync server to get list of files
|
||||||
bool MainWindow::scanDir(Connexion * connexion, QTreeWidgetItem *parent, QString path)
|
bool MainWindow::scanDir(Connexion * connexion, QTreeWidgetItem *parent, QString path)
|
||||||
{
|
{
|
||||||
QString cmd;
|
QString cmd = "rsync";
|
||||||
QStringList param;
|
QStringList param;
|
||||||
QStringList sizeA;
|
QStringList sizeA;
|
||||||
QString line;
|
QString line;
|
||||||
@ -593,6 +612,7 @@ bool MainWindow::scanDir(Connexion * connexion, QTreeWidgetItem *parent, QString
|
|||||||
QString fileType;
|
QString fileType;
|
||||||
QString date;
|
QString date;
|
||||||
QString dirName = "";
|
QString dirName = "";
|
||||||
|
QString user;
|
||||||
QProcess * myProcess;
|
QProcess * myProcess;
|
||||||
QProcessEnvironment env = QProcessEnvironment::systemEnvironment();
|
QProcessEnvironment env = QProcessEnvironment::systemEnvironment();
|
||||||
bool isDir = false;
|
bool isDir = false;
|
||||||
@ -615,24 +635,26 @@ bool MainWindow::scanDir(Connexion * connexion, QTreeWidgetItem *parent, QString
|
|||||||
}
|
}
|
||||||
if (nChild == 0)
|
if (nChild == 0)
|
||||||
{
|
{
|
||||||
if (!connexion->user.isEmpty())
|
|
||||||
{
|
|
||||||
server.prepend(connexion->user + "@");
|
|
||||||
env.insert("RSYNC_PASSWORD", connexion->password); // Add an environment variable
|
|
||||||
}else
|
|
||||||
{
|
|
||||||
server.prepend("anonymous@");
|
|
||||||
env.insert("RSYNC_PASSWORD", "anonymous"); // Add an environment variable
|
|
||||||
}
|
|
||||||
myProcess->setProcessEnvironment(env);
|
myProcess->setProcessEnvironment(env);
|
||||||
|
|
||||||
info(DEBUGMACRO, "ipversion => " + QString::number(connexion->ipversion));
|
info(DEBUGMACRO, "ipversion => " + QString::number(connexion->ipversion));
|
||||||
cmd = "rsync";
|
|
||||||
if(this->connexion.ipversion == 4 || this->connexion.ipversion == 6)
|
if(this->connexion.ipversion == 4 || this->connexion.ipversion == 6)
|
||||||
{
|
{
|
||||||
param << "-" + QString::number(connexion->ipversion).trimmed();
|
param << "-" + QString::number(connexion->ipversion).trimmed();
|
||||||
}
|
}
|
||||||
param << "--contimeout=20" << "--port=" + QString::number(connexion->port) << "[" + server + "]::" + path;
|
|
||||||
|
|
||||||
|
if (!connexion->user.isEmpty())
|
||||||
|
{
|
||||||
|
user = connexion->user + "@";
|
||||||
|
env.insert("RSYNC_PASSWORD", connexion->password); // Add an environment variable
|
||||||
|
}else
|
||||||
|
{
|
||||||
|
user = "anonymous@";
|
||||||
|
env.insert("RSYNC_PASSWORD", "anonymous"); // Add an environment variable
|
||||||
|
}
|
||||||
|
param << "--contimeout=20" << "--port=" + QString::number(connexion->port) << user + "[" + server + "]::" + path;
|
||||||
|
|
||||||
info(DEBUGMACRO, cmd + " " + param.join(" "));
|
info(DEBUGMACRO, cmd + " " + param.join(" "));
|
||||||
|
|
||||||
@ -761,9 +783,10 @@ uint MainWindow::validateServer(QString server)
|
|||||||
info(DEBUGMACRO, "Digging server name");
|
info(DEBUGMACRO, "Digging server name");
|
||||||
|
|
||||||
cmd = "dig";
|
cmd = "dig";
|
||||||
param << "-t A"; // IP V4 query
|
|
||||||
param << "-t AAAA"; // IP V6 query
|
|
||||||
param << server.trimmed() ;
|
param << server.trimmed() ;
|
||||||
|
param << "A"; // IP V4 query
|
||||||
|
param << server.trimmed() ;
|
||||||
|
param << "AAAA"; // IP V6 query
|
||||||
|
|
||||||
info(DEBUGMACRO, cmd + " " + param.join(" "));
|
info(DEBUGMACRO, cmd + " " + param.join(" "));
|
||||||
|
|
||||||
@ -874,7 +897,8 @@ void MainWindow::addTreeItem(QString name, QString fileSize, QString fullsize, Q
|
|||||||
}else
|
}else
|
||||||
{
|
{
|
||||||
// item is a file
|
// item is a file
|
||||||
this->settings.beginGroup("Downloaded/");
|
this->settings.beginGroup("Downloaded");
|
||||||
|
//TODO add to settings
|
||||||
this->settings.endGroup();
|
this->settings.endGroup();
|
||||||
treeItem->setText(1,type);
|
treeItem->setText(1,type);
|
||||||
}
|
}
|
||||||
@ -1040,7 +1064,7 @@ void MainWindow::on_treeWidget_itemClicked(QTreeWidgetItem *item, bool downloadD
|
|||||||
}else
|
}else
|
||||||
{
|
{
|
||||||
this->downloading.savePath = this->settings.value(str).toString();
|
this->downloading.savePath = this->settings.value(str).toString();
|
||||||
info(DEBUGMACRO, "saving folder" + this->downloading.savePath);
|
info(DEBUGMACRO, "saving folder : " + this->downloading.savePath);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Item is a file
|
// Item is a file
|
||||||
@ -1237,7 +1261,7 @@ void MainWindow::downloadFinished(int exitCode, QProcess::ExitStatus exitStatus)
|
|||||||
pos = path.lastIndexOf(" => ");
|
pos = path.lastIndexOf(" => ");
|
||||||
this->downloading.server = path.midRef(pos+4).toString();
|
this->downloading.server = path.midRef(pos+4).toString();
|
||||||
path.resize(pos);
|
path.resize(pos);
|
||||||
this->downloading.port = this->settings.value("connexion/server/" + this->downloading.server).toInt();
|
this->downloading.port = this->settings.value("connexion/Servers/" + this->downloading.server + "/port").toInt();
|
||||||
this->downloading.path = path;
|
this->downloading.path = path;
|
||||||
|
|
||||||
//getUserPassword(true);
|
//getUserPassword(true);
|
||||||
@ -1331,7 +1355,7 @@ void MainWindow::loadSettings()
|
|||||||
// loading connexion settings
|
// loading connexion settings
|
||||||
// loading servers history
|
// loading servers history
|
||||||
info(DEBUGMACRO, "Restoring server combobox history");
|
info(DEBUGMACRO, "Restoring server combobox history");
|
||||||
this->settings.beginGroup("connexion/server");
|
this->settings.beginGroup("connexion/Servers");
|
||||||
QStringList servers = this->settings.allKeys();
|
QStringList servers = this->settings.allKeys();
|
||||||
this->settings.endGroup();
|
this->settings.endGroup();
|
||||||
ui->khistorycombobox->insertItems(1,this->settings.value("kHistoryComboBox").toStringList());
|
ui->khistorycombobox->insertItems(1,this->settings.value("kHistoryComboBox").toStringList());
|
||||||
@ -1498,7 +1522,7 @@ void MainWindow::loadDownloadList()
|
|||||||
path.resize(pos);
|
path.resize(pos);
|
||||||
pos = path.lastIndexOf(" => ");
|
pos = path.lastIndexOf(" => ");
|
||||||
this->downloading.server = path.midRef(pos+4).toString();
|
this->downloading.server = path.midRef(pos+4).toString();
|
||||||
this->downloading.port = this->settings.value("connexion/server/" + this->downloading.server).toInt();
|
this->downloading.port = this->settings.value("connexion/Servers/" + this->downloading.server+ "/port").toInt();
|
||||||
path.resize(pos);
|
path.resize(pos);
|
||||||
this->downloading.path = path;
|
this->downloading.path = path;
|
||||||
str = "Folder/" + this->downloading.server + "/" + this->downloading.service;
|
str = "Folder/" + this->downloading.server + "/" + this->downloading.service;
|
||||||
|
Loading…
Reference in New Issue
Block a user