search

yoomoney / yookassa-android-sdk

This library allows implementing payment acceptance into mobile apps on Android. It works as an extension to the YooMoney API.
https://yookassa.ru/developers/payment-forms/android-sdk
MIT License
40 stars 22 forks source link

Memory leak in fragment when use Checkout.setResultCallback #13

Closed chim0380 closed 5 years ago

chim0380 commented 6 years ago

В вашем SDK есть метод Checkout.setResultCallback , он не принимает параметр null и нет методов по отписке от callback, и из-за этого происходит утечка памяти в том фрагменте приложения, в котором используется Checkout

Checkout.detach() так же не отписывает от callback

Утечка легко обнаруживается при помощи LeakCanary

Если не использовать setResultCallback , то утечки нет.

Было бы здорового, если бы в следующей версии, можно было бы отписываться от Callback.

drukarev commented 6 years ago

Спасибо, исправим в ближайшей версии.

drukarev commented 5 years ago

Fixed in 1.3.0.5