mixpanel / mixpanel-swift

Official iOS (Swift) Tracking Library for Mixpanel Analytics
https://mixpanel.com
Apache License 2.0
434 stars 234 forks source link

Temporarily remove the radio listener #388

Closed zihejia closed 4 years ago

zihejia commented 4 years ago

We currently listen to the radio changes during an app session for customers, i.e. LTE, GPRS, Edge, etc.(https://developer.apple.com/documentation/coretelephony/cttelephonynetworkinfo/radio_access_technology_constants) We record this to $radio, a default event property collected by Mixpanel. However this will lead to noticeable crashes since iOS 13. https://github.com/mixpanel/mixpanel-swift/issues/367

This might seem to be an issue from Apple and there is an existing open radar: https://openradar.appspot.com/46873673. Given the relative high occurrences of crashes, we now temporarily remove the radio listener, this will only impact the scenario where the radio actually changes during an app session. $radio is still going to be recorded at the start of each app session and the value won't be changed for the entire app session.