diff --git a/src/firsttimewizard.cpp b/src/firsttimewizard.cpp index 0e6df79..73cbbf4 100644 --- a/src/firsttimewizard.cpp +++ b/src/firsttimewizard.cpp @@ -623,21 +623,49 @@ bool NewSeedPage::validatePage() { dialog.exec(); - if ((verifyseed.verify->toPlainText() == seed) && (verifyseed.verifyBirthday->toPlainText() == birthday)) - { - char* resp = litelib_execute("save", ""); - QString reply = litelib_process_response(resp); + QString reply = ""; + if ((verifyseed.verify->toPlainText() == seed) && (verifyseed.verifyBirthday->toPlainText() == birthday)) { + try { + char* resp = litelib_execute("save", ""); + reply = litelib_process_response(resp); + } catch (const std::exception& e) { + qDebug() << __func__ << ": caught an exception, ignoring: " << e.what(); + } - auto parsed = json::parse(reply.toStdString().c_str(), nullptr, false); - if (parsed.is_discarded() || parsed.is_null() || parsed.find("result") == parsed.end()) { + qDebug() << __func__ << ": reply=" << reply; - QMessageBox::warning(this, tr("Failed to save wallet"), - tr("Couldn't save the wallet") + "\n" + reply, - QMessageBox::Ok); - return false; - } else { - return true; - } + // TODO: this is duplicated code that should be refactored + // into a dedicated function + if (reply.toUpper().trimmed() != "OK") { + qDebug() << "Lite server " << parent->server << " is down, getting a random one"; + parent->server = Settings::getRandomServer(); + qDebug() << __func__ << ": new server is " << parent->server; + + // make a new connection to the new server + char* resp = litelib_initialize_new(parent->dangerous,parent->server.toStdString().c_str()); + reply = litelib_process_response(resp); + + // retry with the new server + try { + resp = litelib_execute("save", ""); + reply = litelib_process_response(resp); + } catch (const std::exception& e) { + qDebug() << __func__ << ": caught an exception with new server, something is fucky: " << e.what(); + } + } + + qDebug() << __func__ << ": reply=" << reply; + + auto parsed = json::parse(reply.toStdString().c_str(), nullptr, false); + if (parsed.is_discarded() || parsed.is_null() || parsed.find("result") == parsed.end()) { + + QMessageBox::warning(this, tr("Failed to save wallet"), + tr("Couldn't save the wallet") + "\n" + reply, + QMessageBox::Ok); + return false; + } else { + return true; + } }else{ qDebug()<<"Wrong Seed"; QFile file(dirwalletencfirst); @@ -645,9 +673,7 @@ bool NewSeedPage::validatePage() { file.remove(); file1.remove(); - QMessageBox::warning(this, tr("Wrong Seed"), - tr("Please try again") + "\n" , - QMessageBox::Ok); + QMessageBox::warning(this, tr("Wrong Seed"), tr("Please try again") + "\n" , QMessageBox::Ok); form.birthday->setVisible(true); form.txtSeed->setVisible(true); return false; @@ -702,22 +728,67 @@ bool RestoreSeedPage::validatePage() { qint64 number = number_str.toUInt(); // 3. Attempt to restore wallet with the seed phrase { - char* resp = litelib_initialize_new_from_phrase(parent->dangerous, parent->server.toStdString().c_str(), - seed.toStdString().c_str(), birthday, number); - QString reply = litelib_process_response(resp); + QString reply = ""; + try { + char* resp = litelib_initialize_new_from_phrase(parent->dangerous, parent->server.toStdString().c_str(), + seed.toStdString().c_str(), birthday, number); + reply = litelib_process_response(resp); + } catch (const std::exception& e) { + qDebug() << __func__ << ": caught an exception, ignoring: " << e.what(); + } + + qDebug() << __func__ << ": reply=" << reply; + + if (reply.toUpper().trimmed() != "OK") { + qDebug() << "Lite server " << parent->server << " is down, getting a random one"; + parent->server = Settings::getRandomServer(); + qDebug() << __func__ << ": new server is " << parent->server; + + // retry with the new server + char* resp = litelib_initialize_new(parent->dangerous,parent->server.toStdString().c_str()); + reply = litelib_process_response(resp); + } + if (reply.toUpper().trimmed() != "OK") { QMessageBox::warning(this, tr("Failed to restore wallet"), tr("Couldn't restore the wallet") + "\n" + reply, QMessageBox::Ok); return false; - } + } } // 4. Finally attempt to save the wallet { - char* resp = litelib_execute("save", ""); - QString reply = litelib_process_response(resp); + QString reply = ""; + try { + char* resp = litelib_execute("save", ""); + reply = litelib_process_response(resp); + } catch (const std::exception& e) { + qDebug() << __func__ << ": caught an exception, ignoring: " << e.what(); + } + + // TODO: this is duplicated code that should be refactored + // into a dedicated function + if (reply.toUpper().trimmed() != "OK") { + qDebug() << "Lite server " << parent->server << " is down, getting a random one"; + parent->server = Settings::getRandomServer(); + qDebug() << __func__ << ": new server is " << parent->server; + + // make a new connection to the new server + char* resp = litelib_initialize_new(parent->dangerous,parent->server.toStdString().c_str()); + reply = litelib_process_response(resp); + + // retry with the new server + try { + resp = litelib_execute("save", ""); + reply = litelib_process_response(resp); + } catch (const std::exception& e) { + qDebug() << __func__ << ": caught an exception with new server, something is fucky: " << e.what(); + } + } + + qDebug() << __func__ << ": reply=" << reply; auto parsed = json::parse(reply.toStdString().c_str(), nullptr, false); if (parsed.is_discarded() || parsed.is_null() || parsed.find("result") == parsed.end()) { @@ -728,6 +799,6 @@ bool RestoreSeedPage::validatePage() { return false; } else { return true; - } + } } }