Skip to content

Commit

Permalink
Avoid forced unwrapping in Analytics Client.
Browse files Browse the repository at this point in the history
  • Loading branch information
sshropshire committed Feb 1, 2023
1 parent 619c17b commit fcd1a10
Showing 1 changed file with 13 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -118,11 +118,12 @@ internal class AnalyticsClient @VisibleForTesting constructor(
if (events.isNotEmpty()) {
val metadata = deviceInspector.getDeviceMetadata(context, sessionId, integration)
val analyticsRequest = serializeEvents(authorization, events, metadata)
val analyticsUrl = configuration!!.analyticsUrl
httpClient.post(
analyticsUrl, analyticsRequest.toString(), configuration, authorization
)
analyticsEventDao.deleteEvents(events)
configuration?.analyticsUrl?.let { analyticsUrl ->
httpClient.post(
analyticsUrl, analyticsRequest.toString(), configuration, authorization
)
analyticsEventDao.deleteEvents(events)
}
}
ListenableWorker.Result.success()
} catch (e: Exception) {
Expand Down Expand Up @@ -168,11 +169,14 @@ internal class AnalyticsClient @VisibleForTesting constructor(
authorization: Authorization?, events: List<AnalyticsEvent>, metadata: DeviceMetadata
): JSONObject {
val requestObject = JSONObject()
if (authorization is ClientToken) {
requestObject.put(AUTHORIZATION_FINGERPRINT_KEY, authorization.bearer)
} else {
requestObject.put(TOKENIZATION_KEY, authorization!!.bearer)
authorization?.let {
if (it is ClientToken) {
requestObject.put(AUTHORIZATION_FINGERPRINT_KEY, it.bearer)
} else {
requestObject.put(TOKENIZATION_KEY, it.bearer)
}
}

requestObject.put(META_KEY, metadata.toJSON())
val eventObjects = JSONArray()
var eventObject: JSONObject
Expand Down

0 comments on commit fcd1a10

Please sign in to comment.