React-native-iap: リリース後に多くのデバイスでAndroidがクラッシュする

最近のバージョン2.3.19とカムバックを試していただけませんか？

できません。 それは当面の状況だったからです。 そこで、iapパッケージを変更しました。 + 3kユーザーでクラッシュは発生しません。 でもありがとう。 次のプロジェクトで試してみます。

こんにちは、この問題を再開できますか？ 同じ問題があり、バージョン2.3.21を使用しています

java.lang.RuntimeException：
com.facebook.react.bridge.CallbackImpl.invoke（CallbackImpl.java:28）で
com.facebook.react.bridge.PromiseImpl.resolve（PromiseImpl.java:30）で
com.dooboolab.RNIap.RNIapModule $ 4.run（RNIapModule.java:155）で
com.dooboolab.RNIap.RNIapModule $ 3.onBillingSetupFinished（RNIapModule.java:124）で
com.android.billingclient.api.BillingClientImpl $ BillingServiceConnection.onServiceConnected（BillingClientImpl.java:903）で
android.app.LoadedApk $ ServiceDispatcher.doConnected（LoadedApk.java:1658）で
android.app.LoadedApk $ ServiceDispatcher $ RunConnection.run（LoadedApk.java:1687）で
android.os.Handler.handleCallback（Handler.java:789）で
android.os.Handler.dispatchMessage（Handler.java:98）で
android.os.Looper.loop（Looper.java:164）で
android.app.ActivityThread.main（ActivityThread.java:6938）で
java.lang.reflect.Method.invoke（ネイティブメソッド）で
com.android.internal.os.Zygote $ MethodAndArgsCaller.run（Zygote.java:327）で
com.android.internal.os.ZygoteInit.main（ZygoteInit.java:1374）で

これは私のcomponentDidMountです

           const itemSKus = Platform.select({
            ios: [
                ...
            ],
            android: [
                ...
            ]
        });

        try {
            const message = await RNIap.initConnection();
            //console.log(`message = ${message}`);
            const items = await RNIap.getProducts(itemSKus);
            //console.log(`items = ${items.length}`);
            //console.log(items);
            this.props.storeInAppPurchaseList(items);
        } catch (errorCode) {
            console.log(`error rniap = ${errorCode}`);
        }

@LinusUここで私たちを助けてくれませんか？ リリース時にcallback.run()クラッシュするのはなぜですか？ ensureConnection callbackがnullかどうかを確認する必要がありますか？ うーん。 runtime exceptionです。

うーん、ここでRuntimeExceptionどういう意味ですか、これ以上の情報はありませんか？ @hyochanなぜcallbackがnull思いますか？ 🤔

@LinusUフィードバックをありがとうございます。 今はわかりません。 以下の問題行を見たところです。

RNIapModule.java行124で、

callback.run();

他に何かアイデアはありますか？

@LinusUこれについて何もわからない場合は、以前の場所にロールバックすることを検討します。

さて、ここに問題があります：

https://github.com/facebook/react-native/blob/370bcffba748e895ad8afa825bfef40bff859c95/ReactAndroid/src/main/java/com/facebook/react/bridge/CallbackImpl.java#L27 -L31

何らかの理由で、コールバックを複数回呼び出していますが、簡単に修正できるはずです。

@LinusUあなたがトラックにいるので、PRを教えていただけますか？

そのメソッドにRuntimeExceptionがあるのはなぜですか？

if (!mInvoked) {
   mJSInstance.invokeCallback(mCallbackId, Arguments.fromJavaArgs(args));
   mInvoked = true;
}

このようなものでは十分ではありませんか？

@hyochan申し訳ありませんが、昨日これを見る時間がありませんでした、うまくいけば私はすぐに見ることができます

@ Ilario17おそらく、どこかに論理エラーがあることを示すフラグが立てられます。公平を期すために、そのメソッドを複数回呼び出すべきではありません。

編集：ああ、問題はonBillingSetupFinishedが複数回発生したときに発生します。 ensureConnectionでは、完了したら（つまり、 callback.run()を呼び出すか、渡された約束を拒否するときに）リスナーを削除する必要があります。 誰かが無料のカルマを求めているなら、簡単なPRになるはずです！ そうでなければ私はすぐにそれに到達しようとします

@LinusUつまり、 onBillingSetupFinishedことですか、 billingClientStateListenerを削除する必要がありますか？ 誰かが私たちの一部になるためにこれを試してみませんか？

迅速で汚い解決策が必要な場合は、その関数にローカル変数didCallCallback = falseを追加します。 次に、 if (didCallCallback) { didCallCallback = true; callback.run() }背後にあるcallback.run()への呼び出しを保護します。

よりクリーンな解決策は、最初の呼び出しの後でリスナーの登録を完全に解除することです...

Log.d(TAG, "billing client ready");
callback.run();
//deregister the listener here?

これを修正するために2.4.0-beta2をリリースしました。 してみてください。

@hyochanこの新しいバージョンを試してみます

beta2には回帰の問題があったため、2.4.0-beta3を試してください。

更新後の@hyochanエラーはまだあります:(

@ Ilario17 2.4.0-beta5どうですか？ 私の側ではこれを再現できないので、あなたのテストが必要です:(

@LinusUが言ったように@hyochan理由だけではなく、リスナーを削除しますか？

@ Ilario17それが正しい解決策かどうか興味がありました。 また、 onBillingServiceDisconnectedを通知する必要があると思います。 あなたはそれを削除してあなたの側でこれをテストし、それが機能するかどうかを確認しようとするかもしれません。

これが関連しているかどうかはわかりませんが、2.4.0-beta5でこのクラッシュが発生しています。 エミュレーターで再現するのは簡単です：
IAPがロードされる場所にコンポーネントをマウントし、マウントを解除してから再度マウントすると、ブームがクラッシュします。
編集：このコミットにダウングレードしましたが、クラッシュしなくなりましたhttps://github.com/dooboolab/react-native-iap/commit/2db337ac770a93508507ec046f31085ddbb346fb

Attempt to invoke virtual method 'void com.android.billingclient.api.BillingClient.querySkuDetailsAsync(com.android.billingclient.api.SkuDetailsParams, com.android.billingclient.api.SkuDetailsResponseListener)' on a null object reference
run
    RNIapModule.java:223
ensureConnection
    RNIapModule.java:113
getItemsByType
    RNIapModule.java:218
invoke
    Method.java
invoke
    JavaMethodWrapper.java:372
invoke
    JavaModuleWrapper.java:160
run
    NativeRunnable.java
handleCallback
    Handler.java:789
dispatchMessage
    Handler.java:98
dispatchMessage
    MessageQueueThreadHandler.java:29
loop
    Looper.java:164
run
    MessageQueueThreadImpl.java:192
run
    Thread.java:764

2番目のマウント（2.4.0-beta5）でクラッシュを引き起こすコードは次のとおりです。

componentDidMount() {
    this.loadIAPProducts()
}

componentWillUnmount() {
    RNIap.endConnection();
}

loadIAPProducts = async () => {
    const itemSkus = ['XXXX_id']

    const result = await RNIap.initConnection();
    if(result) {
      try {
        const products = await RNIap.getProducts(itemSkus);
        if (products) {
          this.setState({ iapProducts: products })
        }
      } catch (err) {
        //console.log(err); // standardized err.code and err.message available
      }

      this.restoreIAPPurchases()
    }
  }

  restoreIAPPurchases = async () => {
    try {
      const purchases = await RNIap.getAvailablePurchases()
      if(purchases) {
        purchases.forEach(purchase => {
          if (purchase.productId === 'XXXX_id') {
            try {
              RNIap.consumePurchase(purchase.purchaseToken);
            } catch(err) {

            }
          }
        })
      }
    } catch (err) {
      //console.log(err)
    }
  }

@ rom1kまあこれはAndroidエミュレータでは機能しません。 実際のデバイスでお試しください。

@hyochanバージョン2.4.0-

Androidの2.4.0-beta3の場合、getProductsを呼び出すだけで、「コードで購入に失敗しました：0（OK）」というエラーが発生します。 ベータ4と5は、実際にクラッシュを引き起こします。 2.3.5にダウングレードしましたが、正常に動作しますが、2.4.0ベータ版のiOSリスナーのバグ修正が必要です。

`` `
componentDidMount（）{
this.getProductDetails（）;
}

getProductDetails = async () => {
    try {
        await RNIap.initConnection();
        const details = await RNIap.getProducts(products);
        this.setState({
            title: details[0].title,
            description: details[0].description,
            price: details[0].localizedPrice,
            loading: false
        });
    } catch (err) {
        console.log(err.message)
    }
};

beta5とカムバックを試していただけますか？ そのバージョンに間違いがあったので、すぐに修正しました

@ hyochan @ LinusUバージョン2.4.0-beta5ではエラーがまだ発生しています。

この議論に関連する情報がいくつかあります。

2.4.0-beta5で、次のエラーが発生します

Attempt to invoke virtual method void com.android.billingclient.api.BillingClient.querySkuDetailsAsync(...) on a null object reference

RNIapModule.java 223行目を指しています。これは、 mBillingClientが予期せずnullであることを意味します。

再現するには、次RNIap.getProducts()にcomponentDidMountとRNIap.endConnection()のcomponentWillUnmount 。 コンポーネントをアンマウントして、再度マウントします。 上記のエラーが発生するはずです。

変数clientReadyは、 onBillingServiceDisconnected()コールバックでfalseに設定することにより、課金クライアントのセットアップが完了したかどうかを追跡するために使用されています。 ただし、 mBillingClient自体は、 endConnection()が呼び出されるとすぐにnullに設定されます。 これは不一致です。 mBillingClientがnullの場合でも、 clientReadyがtrueに設定されたままになる可能性があります。 少なくとも私の間違いについては、これが起こっていることだと思います。 何かがonBillingServiceDisconnected()呼び出しを妨げているようです。

期待どおりにアンマウント時にlogcatで"billing client disconnected"を取得していないことを確認できるため、 onBillingServiceDisconnected()は実際に呼び出されていません。

修正の可能性： endConnection()メソッド内にclientReady = falseます（たとえば、179行目以降）。 私はこれをデバイスでテストしましたが、動作しているようです。 アプリ内購入コンポーネントを複数回再マウントしても、エラーやクラッシュは発生しないようです。 また、 mBillingClientをnullに設定する必要があるかどうかを考えてみてください。 私はここでの経験がないので言うことはできませんが、少し奇妙に思えます。 また、このコールバックで請求クライアントを無効にするGithub上の他のプロジェクトは見当たりません。

@ mitchellmcm27キャッチしてくれてありがとう。 これは非常に有望であり、私はあなたに同意します。 最小限の再現なしで問題を解決することは私にとって困難でした。 また、一緒に問題を解決したいと思います。

この問題を強化することを考えている場合は、 PRを教えてください。

宜しくお願いします。

私のアプリの最新バージョンで2.4.0-beta6をリリースしましたが、これまでに数千のアップグレードがありました。 まだクラッシュはありません👍

いいね。 修正していただきありがとうございます！ とりあえずこの号を閉じます。

私はすでに同じスタックトレースでいくつかのクラッシュがありますが、以前よりはるかに少ないです。

java.lang.RuntimeException: 
  at com.facebook.react.bridge.CallbackImpl.invoke (CallbackImpl.java:28)
  at com.facebook.react.bridge.PromiseImpl.resolve (PromiseImpl.java:30)
  at com.dooboolab.RNIap.RNIapModule$4.run (RNIapModule.java:155)
  at com.dooboolab.RNIap.RNIapModule$3.onBillingSetupFinished (RNIapModule.java:124)
  at com.android.billingclient.api.BillingClientImpl$BillingServiceConnection.onServiceConnected (BillingClientImpl.java:903)
  at android.app.LoadedApk$ServiceDispatcher.doConnected (LoadedApk.java:1658)
  at android.app.LoadedApk$ServiceDispatcher$RunConnection.run (LoadedApk.java:1687)
  at android.os.Handler.handleCallback (Handler.java:789)
  at android.os.Handler.dispatchMessage (Handler.java:98)
  at android.os.Looper.loop (Looper.java:164)
  at android.app.ActivityThread.main (ActivityThread.java:6944)
  at java.lang.reflect.Method.invoke (Native Method)
  at com.android.internal.os.Zygote$MethodAndArgsCaller.run (Zygote.java:327)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1374)

また、2.4.0-beta6でも同じスタックトレースを取得しています

このバグは再開する必要があると思います。

24時間以内に120がクラッシュします。

最近の変更が独自の問題を引き起こしていないかどうかはわかりません。たとえば、 mBillingClientがnullでないが、まだ準備ができていない場合、 ensureConnection関数は単にmBillingClient上書きします。新しいクライアントで

安定したバージョンにロールバックできるように、どのバージョンでこのエラーが表示され始めたかを知りたいのですが。

私も問題を再開すべきだと思います。 元の問題はコールバックに関係しており、その修正により「nullオブジェクト参照」エラーが発生しました。 後者は修正された可能性がありますが（ @LinusUにはさらに問題がある可能性があることに同意し

beta6のコールバックに関連するクラッシュもいくつか発生しました。

これを再開しました。 誰かがこれにPRを与えることができることを願っています。

これがいつ導入されたか知っていますか？ ロールバックできれば幸いです。

クラッシュはbeta6とAndroidの2.3.23に存在することを確認できます。 この[1]コメントに従って、かなり簡単に再作成できます。

編集：家にいるので、beta6のAndroidネイティブコードにデバッグしています。 これが役立つ場合、これが起こっていることです（少なくとも私のアプリでは、getAvailablePurchasesを呼び出します）：

1. 「inapp」のgetAvailableItemsByTypeは、mBillingClientをまだ設定せずに呼び出されます。
2. 請求クライアントがセットアップを取得し、リスナーのonBillingSetupFinishedが呼び出され、コールバック（Promise）が消費（解決）されます。 リスナーはまだ存在しています。
3. 「subs」のgetAvailableItemsByTypeは、mBillingClientセットアップで呼び出されます。 新しいコールバック（Promise）が消費されます（解決されます）。
4. Playストアサービスを強制終了します-BillingClientImplのonServiceDisconnectedが呼び出され、パート1のリスナーがそれを処理します。 新しいサービスが起動し、パート1のリスナーがonBillingSetupFinishedを処理し、ステップ1のコールバック（Promise）を再び呼び出そうとします。

リスナーが不要になったら、リスナーを削除してみます。

[1]
https://github.com/googlesamples/android-play-billing/issues/45#issuecomment -324466519

＃379をマージして、 beta8リリースしました。 ぜひお試しください。

これは最新バージョンで修正されていますか？ 最新にアップグレードしても安全ですか？

@hyochanこの問題はまだ3.0.0-rc-2に表示されます。

java.lang.RuntimeException
com.dooboolab.RNIap.RNIapModule$3.onBillingSetupFinished

java.lang.RuntimeException: 
  at com.facebook.react.bridge.CallbackImpl.invoke (CallbackImpl.java:28)
  at com.facebook.react.bridge.PromiseImpl.resolve (PromiseImpl.java:30)
  at com.dooboolab.RNIap.RNIapModule$3.onBillingSetupFinished (RNIapModule.java:129)
  at com.android.billingclient.api.BillingClientImpl$BillingServiceConnection$1.run (BillingClientImpl.java:1518)
  at android.os.Handler.handleCallback (Handler.java:739)
  at android.os.Handler.dispatchMessage (Handler.java:95)
  at android.os.Looper.loop (Looper.java:148)
  at android.app.ActivityThread.main (ActivityThread.java:7325)
  at java.lang.reflect.Method.invoke (Native Method)
  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run (ZygoteInit.java:1230)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1120)

@fokoz 3.0.0-rc-4を試していただけませんか？ それは完全に機能します（うまくいけば）。

@hyochan try catchを使用すると、このランタイムエラーは発生しませんが、rc-4で解決できると思います。
ありがとうございました ！