Closed hyochan closed 1 week ago
Risk Level 2 - /home/runner/work/react-native-iap/react-native-iap/android/src/amazon/java/com/dooboolab/rniap/RNIapAmazonListener.kt
The receiptToMap
function is a useful addition for mapping receipt data to a WritableMap
. However, ensure that all fields are correctly mapped and consider adding null checks where necessary to avoid potential NullPointerException
.
Risk Level 3 - /home/runner/work/react-native-iap/react-native-iap/android/src/amazon/java/com/dooboolab/rniap/RNIapAmazonModule.kt
initConnection
method's promise.safeReject
should ensure that the error message is clear and actionable.deepLinkToSubscriptions
method should handle potential exceptions when starting activities to avoid crashes.Risk Level 3 - /home/runner/work/react-native-iap/react-native-iap/android/src/play/java/com/dooboolab/rniap/RNIapModule.kt
ensureConnection
method's nested promise handling is complex and could benefit from additional error handling and logging.buyItemByType
method should ensure that all parameters are validated before proceeding with the purchase flow to avoid runtime errors.🛠️ The receiptToMap
function is a useful addition but needs null checks to avoid potential NullPointerException
.
⚠️ The initConnection
method should have clear and actionable error messages, and the deepLinkToSubscriptions
method needs exception handling to prevent crashes.
🔍 The ensureConnection
method's nested promise handling is complex and requires better error handling and logging, while the buyItemByType
method should validate all parameters to avoid runtime errors.
https://developer.android.com/google/play/billing/migrate-gpblv7
Related: #2755