redstrate/astra
1
Fork 0
mirror of https://github.com/redstrate/Astra.git synced 2025-04-23 21:07:45 +00:00
Code Activity

Log warnings when keychain fails

Browse source
This commit is contained in:
Joshua Goins 2023-12-20 21:23:38 -05:00
parent 4112db4af0
commit d9a578d857
3 changed files with 16 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
launcher/include/account.h
View file

@ -106,7 +106,7 @@ private:
/*
* Sets a value in the keychain. This function is asynchronous.
*/
void setKeychainValue(const QString &key, const QString &value);
QCoro::Task<> setKeychainValue(const QString &key, const QString &value);
/*
* Retrieves a value from the keychain. This function is synchronous.

14
launcher/src/account.cpp
View file

@ -269,7 +269,7 @@ void Account::fetchAvatar()
}
}
void Account::setKeychainValue(const QString &key, const QString &value)
QCoro::Task<> Account::setKeychainValue(const QString &key, const QString &value)
{
auto job = new QKeychain::WritePasswordJob(QStringLiteral("Astra"), this);
job->setTextData(value);
@ -280,6 +280,14 @@ void Account::setKeychainValue(const QString &key, const QString &value)
#endif
job->setInsecureFallback(m_launcher.isSteamDeck()); // The Steam Deck does not have secrets provider in Game Mode
job->start();
co_await qCoro(job, &QKeychain::WritePasswordJob::finished);
if (job->error() != QKeychain::NoError) {
qWarning(ASTRA_LOG) << "Error when writing" << key << job->errorString();
}
co_return;
}
QCoro::Task<QString> Account::getKeychainValue(const QString &key)
@ -295,6 +303,10 @@ QCoro::Task<QString> Account::getKeychainValue(const QString &key)
co_await qCoro(job, &QKeychain::ReadPasswordJob::finished);
if (job->error() != QKeychain::NoError) {
qWarning(ASTRA_LOG) << "Error when reading" << key << job->errorString();
}
co_return job->textData();
}

3
launcher/src/launchercore.cpp
View file

@ -209,8 +209,9 @@ bool LauncherCore::isSteam() const
bool LauncherCore::isSteamDeck() const
{
return true;
if (m_steamApi != nullptr) {
return m_steamApi->isDeck() || qEnvironmentVariable("SteamDeck") == QStringLiteral("1");
return true;
} else {
return false;
}