-
Notifications
You must be signed in to change notification settings - Fork 795
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
iOS 16 (am not sure) restoring purchases hangs? #682
Comments
Yes, there seems to be an issue with iOS 16 https://developer.apple.com/forums/thread/713533 Now I'm not sure if that's a bug on their side that they are going to fix or if some parameters have changed, leading to SwiftyStoreKit not recognizing the data the App Store sends |
I have tried the SwiftyStoreKit demo app with Xcode 14.1 Beta 1, but the callback is not called after restoring. Does this problem occur only in the Sandbox environment? |
FYI, it’s also in Production environment. |
I can confirm that happens, not on all the devices. |
I have apps that rely on SwiftyStoreKit and running directly on device and also through Testflight I can purchase and restore with no problems (testing on iPhone 11 Pro, iOS 16.0). Could you please share more details about your test environment? |
After I built my apps (no changes in the code) with |
Does it work on Testflight? |
It's not the Xcode version, my builds are not submitted with Xcode 14 |
Nope. (TestFlight == production right?) (as suggested on main repo site - I migrated to StoreKit 2 all of my apps) |
Which version of SwiftyStoreKit are you using? I'm using cocoapods: pod 'SwiftyStoreKit', '0.15.0' |
Always latest/updated as Swift package. |
I was planning to release an updated version built with Xcode 14 but I guess I'll wait a while then |
I can confirm that this is happening on iOS 16 even when build with Xcode 13. |
https://zenn.dev/ch3cooh/scraps/5a907126170634 (use Google Translate to translate the page) has some more insight on this. The most important bit is that one: I confirmed that the callback of paymentQueueController.restorePurchases was not returned.
I've kind of given up on the whole SwiftyStoreKit thing but now that this person has been able to estimate a cause there may actually be some hope |
@CH3COOH sorry for the tag, just found your GH - what's your status now? Did you figure out anything else regarding SwiftyStoreKit or did you drop it entirely and go fully StoreKit 2? |
I investigated the success or failure of the combination of Xcode and Testflight/AppStore. I was able to restore if I used a combination of Xcode 14 and App Store. I am installing SwiftyStoreKit with Carthage.
|
This bug is that SwiftyStoreKit/Sources/SwiftyStoreKit/PaymentQueueController.swift Lines 265 to 271 in f3265c7
However, we were unable to discover why the callback was not called in the case of the combination of Xcode 14 and Testflight. I would like to fix SwiftyStoreKit, but I have no knowledge of StoreKit 1 and cannot fix it. If I can't fix it after investigating for a while, I would like to move to StoreKit 2. |
I still haven't been able to reproduce this error with my apps, I just tested submitting new builds to TestFlight integrating SwiftyStoreKit from Carthage and SPM and both work fine for me too on my iPhone 11 Pro. Maybe this problem could be related to the App Store region that you're using? My Apple ID account is Brazilian, with BRL as currency. I'll submit one build to TestFlight review and share a link here so you can test if it is hanging on your devices |
@CH3COOH thank you for the insights - we have moved to SK2 now. Testing is a real pain but hopefully everything is going to work out now. Good luck to you too! @ArthurAlvarez It's hard for me to estimate but I'd say I don't see this problem with all devices if I consider purchases and support requests. You can do the link-sharing and I can tell you whether it worked or not if that helps you. I will probably say bye to SSK but not because I think it's at fault - it's because I think the issue is very likely with SK1, it seems like Apple tried to touch something they shouldn't have touched for iOS 16. Otherwise it doesn't make sense why iOS 16 behaves differently than iOS 15. What's odd is that I can hardly find any other reports. I'd guess more than 90% of the apps on the App Store are on SK1 or on a library that depends on it. |
This is what I don't understand. It doesn't seem to affect everyone for some reason. Do you have bitcode enabled in your apps? |
@Reiszecke Testing SK2 with Xcode 14 is quite easy using StoreKit configuration file. Check it on google. FYI, I also disabled Bitcode on my apps as suggested by Xcode 14. |
Here is the Testflight link: In my device (iPhone 11 Pro - iOS 16.0) this build is restoring OK. Please check on your devices so we can try to determine if the problem is build-related. Can someone also provide a Testflight link of some app that is failing to restore? |
As per the screenshot feedback I just sent through TestFlight, I’m just getting a spinning green Buy button |
Green button spun for 10 seconds (probably SK TF Sandbox delay) I have first tried restoring because why not, did not work. Then I have purchased, purchase went through, was recognized. I have uninstalled, restored again, worked again. iPhone SE 3rd gen iOS 16. What's your dev machine setup? OS, Xcode version and bitcode enabled/disabled? |
Please wait a while for it to fetch the product. I don't know why but it is slow on TestFlight environment.
Macbook with M1 Pro processor, macOS Monterey, Xcode 14.0 (14A309), bitcode disabled |
Can everyone here comment their bitcode configurations? This may actually be the issue here. Checked with another app so so far we have
@sabiland I'm not sure if I understood what apps of yours are bitcode and which ones aren't - can you elaborate the missing info?
|
So we don't have any development on this... Are you all moving away from SwiftyStoreKit or this problem just happens in Sandbox? |
If I understood correctly you didn't have any issues in your non-bitcode app. I think that's the key, you don't need to do anything on your end I guess. SSK is pretty much on hold anyway. I moved away from SSK to SK2 now because I couldn't afford messing around with build settings etc. and releasing a potentially breaking update so I have released whatever worked with TestFlight (automatic Sandbox) accounts and released that. |
Just for curiosity, are you using a library that wraps SK2 or implementing it directly? |
Was thinking about using tikhop/Mercato#4 but the issue sounded too risky, so it's native SK2 without any library. RevenueCat is also fairly simple to add however if you browse through their forums, they have also messed up something in their code more than once so far so that's another risk I would only take for a temporary hotfix. In other words unless we see any issues with native SK2 it's probably the best decision to bite the bullet and move over to SK2. Testing is still an absolute nightmare tho. About adding the code; SK2 really is much simpler than SK1 - the implementation code amount required is fairly similar to SSK. I think Apple could've made this A LOT simpler still by simply providing a singleton that has getters for each IAP identifier but I doubt there will ever be a SK3 so it probably won't get better than this, ever |
I refactored 3 my apps to Mercato, no problems so far. Though I only use 1 non-consumable IAP. |
Yeah I'm simply afraid of Mercato breaking in future just like SSK broke for some reason. Users are (understandably) very sensitive when it comes to purchase errors, they are quick to call you a scam etc. so the risk of using a lib with 41 stars was too high for me this time. |
I'm also facing this issue running on Xcode 14. Restore hangs |
It seems SK2 is quite easy for manually to implement and as I checked Mercato code, it’s also quite straightforward. And another thing (I did not know about this until recently), IAP can so easily be tested locally with StoreKit configuration file. |
This Xcode testing also works with SK1, I have tested my app both with SSK and using this Mercato Library and one thing that I found is that when SSK restores a purchase, instead of only validating the existing transaction, it creates a new transaction with the same ID of the original one, so if you restore a bunch of times you will end up with a lot of transactions on the debug menu. Maybe this behavior could be a hint to the problem |
Guys I finally migrated to StoreKit2 and it was more easy than expected. Also I had important issues with Testflight versions and the damn sandbox server. So I finally figure out that after doing a phone reset and using a single apple id (don't ever change it if you can) everything worked as expected. Hope it helps |
I would like to add that I published this using Version 14.0.1 (14A400). Also our CI Server has an outdated version and I think that was an issue too. |
@mercadodiego as far as i understand the issue was not in code but in the environment? |
I think Apple fixed this bug in iOS 16.0.3 . |
The combination of Xcode 14.0, iOS 16.0.3 and Testflight did not fix this problem... |
Yep, that's why I had to migrate to StoreKit2 |
@CH3COOH after I migrated to StoreKit2 I found issues with the environment as I described above. A factory reset seems to resolve the issue I was having only on my phone. |
I have confirmed that it works with Xcode 14.1 RC2, iOS 16.1 and a combination of TestFlight and SwiftyStoreKit, and my app was able to restore. I would appreciate it if you could also check if you are having this issue. |
Confirmed. |
Is there any info on restoring one-time purchases on macOS 13? |
Restoring purchases hangs
I am not sure if this is AppStore issue, but today I cannot successfully call restoring purchases method. Completion handler is never called - hangs.
Anyone else with this issue?
My third app still built with Xcode 13 is ok. Both my other apps built with Xcode 14 have the same issues - hangs. I guess the problem could not be with Bitcode setting No (it is not recommended anymore having Bitcode Yes on Pods)?
EDIT: More specifically, this one hangs
restoreCompletedTransactions
The text was updated successfully, but these errors were encountered: