Mirrors/overte
3
0
Fork 0
mirror of https://github.com/overte-org/overte.git synced 2025-04-09 15:22:27 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Add comments indicating intent.

Browse source
This commit is contained in:
Kasen IO 2020-06-09 15:07:39 -04:00
parent 49f72f0ca6
commit f6214d46d0
1 changed files with 21 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
libraries/entities/src/EntityTree.cpp
View file

@ -1451,6 +1451,9 @@ void EntityTree::addCertifiedEntityOnServer(EntityItemPointer entity) {
if (!existingEntityItemID.isNull()) {
qCDebug(entities) << "Certificate ID" << certID << "already exists on entity with ID"
<< existingEntityItemID << ". No action will be taken to remove it.";
// FIXME: All certificate checking needs to be moved to its own files,
// then the deletion settings need to have a toggle for domain owners
// and a setting to change the verification service provider.
// withWriteLock([&] {
// deleteEntity(existingEntityItemID, true);
// });
@ -1528,6 +1531,9 @@ void EntityTree::startDynamicDomainVerificationOnServer(float minimumAgeToRemove
}
qCDebug(entities) << "Entity's cert's domain ID" << jsonObject["domain_id"].toString()
<< "doesn't match the current Domain ID" << thisDomainID << ". No action will be taken to remove it: " << entityID;
// FIXME: All certificate checking needs to be moved to its own files,
// then the deletion settings need to have a toggle for domain owners
// and a setting to change the verification service provider.
// withWriteLock([&] {
// deleteEntity(entityID, true);
// });
@ -1556,6 +1562,9 @@ void EntityTree::startChallengeOwnershipTimer(const EntityItemID& entityItemID)
});
connect(_challengeOwnershipTimeoutTimer, &QTimer::timeout, this, [=]() {
qCDebug(entities) << "Ownership challenge timed out for entity " << entityItemID << ". No action will be taken to remove it.";
// FIXME: All certificate checking needs to be moved to its own files,
// then the deletion settings need to have a toggle for domain owners
// and a setting to change the verification service provider.
// withWriteLock([&] {
// deleteEntity(entityItemID, true);
// });
@ -1651,6 +1660,9 @@ void EntityTree::sendChallengeOwnershipPacket(const QString& certID, const QStri
if (text == "") {
qCDebug(entities) << "CRITICAL ERROR: Couldn't compute nonce. No action will be taken to remove this entity.";
// FIXME: All certificate checking needs to be moved to its own files,
// then the deletion settings need to have a toggle for domain owners
// and a setting to change the verification service provider.
// withWriteLock([&] {
// deleteEntity(entityItemID, true);
// });
@ -1725,11 +1737,17 @@ void EntityTree::validatePop(const QString& certID, const EntityItemID& entityIt
if (networkReply->error() == QNetworkReply::NoError) {
if (!jsonObject["invalid_reason"].toString().isEmpty()) {
qCDebug(entities) << "invalid_reason not empty, no action will be taken to delete entity" << entityItemID;
// FIXME: All certificate checking needs to be moved to its own files,
// then the deletion settings need to have a toggle for domain owners
// and a setting to change the verification service provider.
// withWriteLock([&] {
// deleteEntity(entityItemID, true);
// });
} else if (jsonObject["transfer_status"].toArray().first().toString() == "failed") {
qCDebug(entities) << "'transfer_status' is 'failed', no action will be taken to delete entity" << entityItemID;
// FIXME: All certificate checking needs to be moved to its own files,
// then the deletion settings need to have a toggle for domain owners
// and a setting to change the verification service provider.
// withWriteLock([&] {
// deleteEntity(entityItemID, true);
// });
@ -1744,6 +1762,9 @@ void EntityTree::validatePop(const QString& certID, const EntityItemID& entityIt
} else {
qCDebug(entities) << "Call to" << networkReply->url() << "failed with error" << networkReply->error() << "; no action will be taken to delete entity" << entityItemID
<< "More info:" << jsonObject;
// FIXME: All certificate checking needs to be moved to its own files,
// then the deletion settings need to have a toggle for domain owners
// and a setting to change the verification service provider.
// withWriteLock([&] {
// deleteEntity(entityItemID, true);
// });