-
Notifications
You must be signed in to change notification settings - Fork 293
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add diagnosticsQueue to BackendConfiguration #3794
Conversation
Sources/Networking/Backend.swift
Outdated
@@ -174,6 +175,14 @@ extension Backend { | |||
return operationQueue | |||
} | |||
|
|||
static func createDiagnosticsQueue() -> OperationQueue { | |||
let operationQueue = OperationQueue() | |||
operationQueue.name = "Diagnostics Queue" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we name it RC Diagnostics queue
? So people know it's from RevenueCat.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the other queue doesn't have RC either, adding it
/// Adds the `operation` to the diagnostics `OperationQueue` potentially adding a random delay. | ||
func addDiagnosticsOperation( | ||
_ operation: NetworkOperation, | ||
delay: Delay |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FFTI. Should this have a default value for Delay
? I guess it doesn't really matter, depending on how we use it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that makes sense, I will add a Long
**This is an automatic release.** ### Bugfixes * Prevent Template 4 from wrapping Lifetime (#3789) via Josh Holtz (@joshdholtz) * Add enum entry for external purchases store (#3779) via Mark Villacampa (@MarkVillacampa) ### Dependency Updates * Bump fastlane from 2.219.0 to 2.220.0 (#3783) via dependabot[bot] (@dependabot[bot]) ### Other Changes * Add option to intercept touch events in `PaywallViewController` (#3801) via Toni Rico (@tonidero) * Create DiagnosticsPostOperation (#3795) via Cesar de la Vega (@vegaro) * Add diagnosticsQueue to BackendConfiguration (#3794) via Cesar de la Vega (@vegaro) * Add origin to HTTPResponseType (#3793) via Cesar de la Vega (@vegaro) * Add name property to HTTPRequestPath (#3790) via Cesar de la Vega (@vegaro) * Add name to VerificationResult (#3792) via Cesar de la Vega (@vegaro) * Add HTTPRequest.DiagnosticsPath (#3791) via Cesar de la Vega (@vegaro) * Add async `syncAttributesAndOfferingsIfNeeded()` (#3785) via Josh Holtz (@joshdholtz) * iOS append events to JSONL file and get diagnostics events (#3781) via Cesar de la Vega (@vegaro) * Fix offerings integration test (#3782) via Josh Holtz (@joshdholtz)
Create a different OperationQueue to be used when sending Diagnostics