Merge branch 'mkirk/cds-feedback-2'

pull/1/head
Michael Kirk 7 years ago
commit 0598733fc6

@ -416,8 +416,6 @@ class CDSFeedbackOperation: OWSOperation {
enum FeedbackResult: String { enum FeedbackResult: String {
case ok case ok
case mismatch case mismatch
case serverError = "server-error"
case clientError = "client-error"
case attestationError = "attestation-error" case attestationError = "attestation-error"
case unexpectedError = "unexpected-error" case unexpectedError = "unexpected-error"
} }
@ -464,10 +462,9 @@ class CDSFeedbackOperation: OWSOperation {
if let error = cdsOperation.failingError { if let error = cdsOperation.failingError {
switch error { switch error {
case ContactDiscoveryError.serverError: case ContactDiscoveryError.serverError, ContactDiscoveryError.clientError:
self.makeRequest(result: .serverError) // Server already has this information, no need to report.
case ContactDiscoveryError.clientError: self.reportSuccess()
self.makeRequest(result: .clientError)
case ContactDiscoveryError.attestationError: case ContactDiscoveryError.attestationError:
self.makeRequest(result: .attestationError) self.makeRequest(result: .attestationError)
default: default:

Loading…
Cancel
Save