braintree / braintree_ios

Braintree SDK for iOS
https://developer.paypal.com/braintree/docs/start/hello-client/ios/v5
MIT License
560 stars 294 forks source link

libKountDataCollector.a messes with isIdleTimerDisabled? #777

Closed JetForMe closed 2 years ago

JetForMe commented 2 years ago

Integration Details (please complete the following information):

Describe the bug Our app has a live streaming component, and we have code in there to disable the idle timer when a livestream is playing. Unfortunately, something is re-enabling it.

Searching through the entire source base of our app and our dependencies, there are no unexpected references to isIdleTimerDisabled. However, searching the binaries produces the following results:

$ ag --search-binary isIdleTimerDisabled …project-btahrkmjjfdolqatefsxegnpulhn/SourcePackages 
Binary file …project-btahrkmjjfdolqatefsxegnpulhn/SourcePackages/checkouts/braintree_ios/Sources/BraintreeDataCollector/Kount/libKountDataCollector.a matches.
Binary file …project-btahrkmjjfdolqatefsxegnpulhn/SourcePackages/checkouts/braintree_ios/Frameworks/XCFrameworks/KountDataCollector.xcframework/ios-arm64_x86_64-simulator/libKountDataCollector.a matches.
Binary file …project-btahrkmjjfdolqatefsxegnpulhn/SourcePackages/checkouts/braintree_ios/Frameworks/XCFrameworks/KountDataCollector.xcframework/ios-arm64_armv7/libKountDataCollector.a matches.

Expected behavior It doesn't seem to me that anything in Braintree needs to touch isIdleTimerDisabled. But I'm not even sure it's actually touching it. I figured you guys could take a look at your code and see.

Thanks!

sarahkoop commented 2 years ago

Hi @JetForMe - Thanks for using the Braintree iOS SDK. We are looking into this and will post an update here when we have more information.

sarahkoop commented 2 years ago

@JetForMe - Are you including the BraintreeDataCollector framework and using a Kount Custom Integration for device data collection?

The Kount library is no longer receiving updates, and integrations should migrate to using PayPalDataCollector instead. See our developer docs for how to use PayPalDataCollector.

JetForMe commented 2 years ago

Thanks, I'll look into making sure we don't use it. This was just a shot in the dark, based on a crude search of our sources and dependencies. It seems we've found another possible culprit, as well. Thanks!