Missing call to success() or failure() in setDeviceVerification()?

[bjorn-ove]

To me it seems there should be an else here that calls one of the completion callbacks. Or does the API not guarantee one of the callbacks is called?

matrix-ios-sdk/MatrixSDK/Crypto/MXCryptoV2.swift

Lines 398 to 429 in 05e5e85

	if (userId == machine.userId) {
	if (machine.crossSigningStatus().hasSelfSigning) {
	// If we can cross sign, upload a new signature for that device
	Task {
	do {
	try await machine.verifyDevice(userId: userId, deviceId: deviceId)
	log.debug("Successfully marked device as verified")
	await MainActor.run {
	success?()
	}
	} catch {
	log.error("Failed marking device as verified", context: error)
	await MainActor.run {
	failure?(error)
	}
	}
	}
	} else {
	// It's a good time to request secrets
	Task {
	do {
	try await machine.queryMissingSecretsFromOtherSessions()
	await MainActor.run {
	success?()
	}
	} catch {
	log.error("Failed to query missing secrets", context: error)
	failure?(error)
	}
	}
	}
	}