SKPaymentQueue addTransactionObserver asking for App Store password on startup after in-app purchase

16,894

Solution 1

I had the same problem.make sure that you call

[[SKPaymentQueue defaultQueue] finishTransaction:transaction]; 

for all three states of the transactions: SKPaymentTransactionStatePurchased, SKPaymentTransactionStateRestored, SKPaymentTransactionStateFailed.

Solution 2

DO NOT DELETE THE ANSWER HERE. It was this particular Stackoverflow question that misled me and messed me up for days.

I'm putting this here because there are a lot of really bad answers that provide WRONG information on how to resolve the problem.

DO NOT:

  • Delete the sandbox test user. This makes it impossible to resolve the problem and you will have to contact Apple developer support to resolve manually.
  • If you delete the sandbox test user, when you are subsequently repeatedly prompted to log in as that user and complete the transaction, you can't, hence the name Endless Loop problem. Nor will you be able to add the deleted test user again; the developer portal says the user id has already been used.
  • Delete the App or re-install iOS or any other such nonsense. It has no effect, doesn't solve the problem and wastes a lot of time.

DO:

  • Call Finish for ALL transactions.
  • If one is interrupted for some reason, simply complete on a subsequent run of the App. The app will be repeatedly sent the payment queue notice until you call finish on it:

[[SKPaymentQueue defaultQueue] finishTransaction:transaction];

That's it, Finish all transactions! Else you will be sent to the hell of the Endless Loop of sign in requests every single time your App launches on that device.

Solution 3

I had the same problem of having the login prompt coming up at the call:

[[SKPaymentQueue defaultQueue] addTransactionObserver:observer];

It would also come up every now and then even when I wasn't using my app (on the home screen or in other apps), which was really annoying. Looking around, there seem to be so many suggested answers to this issue but I finally found a solution from a combination of what I've gathered.

Note: Before step 1, I had removed the test sandbox account in iTunes Connect. I'm not sure if that would affect the solution.

To solve the problem this is what I did:

  1. Run your app from Xcode.
  2. Wait for the prompt to come up. Type in the password for the account it wants and tap OK.
  3. Press the Home button on the device.
  4. Kill the app from Xcode.
  5. Delete the app from the device.
  6. Log out of iTunes & App Store in the Settings app.
  7. Turn off the device and then turn it back on.
  8. Purchase something from the App Store. When it prompts you, log in with a production Apple ID account. (I'm assuming you should be able to just log in with a production account in iTunes & App Store under the Settings app but this is how I did it).
  9. Go back to Xcode and run your app again. (This should be a new install, as you deleted the app before.)
  10. Wait for the login prompt to come up.
  11. Tap Cancel. A dialog saying "Sign In Required. Tap Continue and sign in to check for downloads. [Environment: Sandbox]" should come up. This was a key difference from before. I never had this dialog come up when I pressed Cancel when it was asking me for the password.
  12. Tap Continue.
  13. Enter the password for the account.

That's it. From then on the login prompt stopped coming up whenever I ran my app and also stopped coming up at random times.

Hope this helps!

Solution 4

There is a problem called the "endless loop". It was big issue back in the early days of auto renewables when, for about a week, the servers did not limit renewals to 5. A device that gets a transaction and doesn't call finishTransaction will get that transaction delivered to the device about once a week until that particular test user logs in and calls finishTransaction. If you switch to airplane mode you can 'clear' those transactions for another week - but they come back.

Solution 5

I suspect that this is a correct behaviour. When you set a delegate SKPaymentQueue try to check if there are some transactions to finalize. There may be no not finished transactions but the fact of checking requires to login in iTunes. And I think you can do nothing with it.

It generally has some sense, but it is pretty annoying for users who have set up asking for a password on each transaction (some child protection for instance). So the only way to struggle with it is to set delegate explicitly when you are about to request iTunes. For example you can add some button like "Restore my purchases". Not very beautiful but definitely less annoying.

Share:
16,894
montuno
Author by

montuno

Updated on June 08, 2022

Comments

  • montuno
    montuno almost 2 years

    My app is using in-app purchases, and most of my users can purchase just fine without any problems. For these folks, my app downloads the content after the purchase succeeds and they are happy.

    However, for a growing number of my users, once they complete a successful in-app purchase they are being asked for their App Store password every time the app starts up after that. I believe this is happening on the call to:

    [[SKPaymentQueue defaultQueue] addTransactionObserver:observer];
    

    which I am calling on startup in accordance with step 6 in Apple's in-app purchase guide:

    My guess is that, for some reason, Apple's in-app purchase servers aren't registering that the transaction finished successfully - even though I call

    [[SKPaymentQueue defaultQueue] finishTransaction:transaction];
    

    when the transaction is completed and my content has been successfully downloaded.

    2 questions:

    1. Is anyone else seeing this?

    2. Does anyone have a suggested fix?

    BOUNTY EDIT:

    Its a transaction which was made with a different Apple-ID. Thats why it cannot be finished unless you type in the right credentials into the dialog. The Question should be either:

    1. How can I prevent such dead transactions (transaction has not been finished, user has no network, meanwhile changes App-ID)?
    2. How can you prune the SkPaymentQueue?
  • F.X.
    F.X. almost 12 years
    I actually did the same thing with an app I helped develop. Probably the easiest solution.
  • bobobobo
    bobobobo over 10 years
    Yeah but I'm not even getting any transactions. I even inserted delegate for each of the 5 SKPaymentTransactionObserver delegate methods. The iTunes store asks me to sign in every time I start my app now..
  • pipipi
    pipipi almost 10 years
    Troubled by the same issue, this helped me so much!
  • Mark A. Durham
    Mark A. Durham over 8 years
    Worked for me. Steps 11 through 13 are the critical steps that solve the problem. I am not certain which or how many of steps 1 through 10 are necessary to trigger it. For the record, I did not remove the test sandbox account (Step 0) and I did not purchase anything from the App Store (Step 8). I did log in with a production account in Settings instead though (as suggested in Step 8).
  • iVader
    iVader about 8 years
    Situation: you had registered some [email protected] and tested IAP with it and not finished some transaction. Then you registered [email protected]. So, if you open app (be wanting to test it with [email protected]), you will see dialog with input credentials for [email protected]! Even if you delete [email protected] user from ITC - this dialog will appeared again, until you input correct credentials for your old user ([email protected]). After this dialog willn't appear! It is working for me
  • Cliff Ribaudo
    Cliff Ribaudo almost 8 years
    No it does not! Bad advice. Do not do this.
  • Cliff Ribaudo
    Cliff Ribaudo almost 8 years
    This is BAD ADVICE. NEVER, EVER delete the sandbox test user. The simple solution is to call [[SKPaymentQueue defaultQueue] finishTransaction:transaction]; If you deleted the test user you will be unable to do that and go into the "Endless Loop" IAP hell that can only be resolved via a TSI
  • Maciek Czarnik
    Maciek Czarnik almost 8 years
    Thats not true @CliffRibaudo 😉
  • Cliff Ribaudo
    Cliff Ribaudo almost 8 years
    Yes it IS true. NEVER, EVER, EVER, delete the sandbox test user in an effort to try to resolve incomplete IAPs. Read the numerous posts on "Endless Loop" problem on Apple Developer forums. You should actually delete this answer is it will create problems for people who follow it.
  • Maciek Czarnik
    Maciek Czarnik almost 8 years
    @CliffRibaudo you're underestimating my knowledge and the fact, that sometimes IAP sandbox acts weird. I know about -finishTransaction: (obviously) and in most cases it works, but what you see in documentation is NEVER, EVER, EVER 100% how it looks like in reality. Actually you should have known that as an experienced (I believe) developer
  • Cliff Ribaudo
    Cliff Ribaudo almost 8 years
    Not really sure why people are down voting this?! Clearly the people down voting are either ignorant of the facts or simply vindictive. As a long time iOS Developer and someone who was recently seriously messed up by one of the answers here to "Delete Test Users" I assure you, DON'T DO IT.
  • Infaz
    Infaz almost 8 years
    I just did 1 to 10,and then in 11th step i tapped on cancel,thats it.It really worked :)
  • Jonny
    Jonny over 7 years
    That sounds very similar to a problem I had that existed way before auto renewables existed. Unfortunately those popups were tied to MY personal main account, so I had them show up like every once a week or more and I had to press cancel like 30-50 times before they gave up. After many years I was able to fix it by going back to a past test app I had been testing IAP implementations with. Creating that app again with the same bundle identifier, I was able to catch some really old unfinished transactions, finish them off, and the problems I had a few years were then gone.
  • Vitalii
    Vitalii over 7 years
    I guess people downvote because it is not stated clear enough that you suggest finishing all transactions as a temporary solution while testing/debugging. Some other people might use your suggestion and their app will go into production with .deferred or .purchasing states being cleared right away (which is not good for app in Production).
  • Cliff Ribaudo
    Cliff Ribaudo over 7 years
    What are you talking about?! I don't suggest finishing transactions is a TEMPORARY solution. One must always finish the transaction regardless of if in testing or production.... so if YOU down voted perhaps you should reconsider that or show me exactly where it indicated it was temporary.
  • Vitalii
    Vitalii over 7 years
    Cliff, I didn't downvote. I upvoted 'cause your answer helped me. The answer is bold, that's why I made a guess on your question "why ppl downvoting?"... You are definitely right that the app should be designed (and well tested) in such a way that all transactions do end up being finished after all, otherwise things go bad. My guess is that some people might have read your suggestion literally - they could try finishing everything right away in the paymentQueue(_:updatedTransactions:) and were disappointed, because they were finishing transactions in "purchasing" and "deferred" states.
  • Cliff Ribaudo
    Cliff Ribaudo over 7 years
    Ok, I understand now ;) Some people did down vote it for some reason. Not sure why as it is actually the most correct answer. A lot of developers first reaction when they run into trouble is to delete the sandbox user and that will be trouble.
  • siburb
    siburb over 6 years
    I know this is an old answer, but it is helpful - particularly the "Do Not" section. @CliffRibaudo, I think the misunderstanding is people confusing where you say "all transactions" for all states. Transactions have to be "finished" on the queue for the "Purchased", "Restored" AND "Failed" states. Attempting to finish at any other states will throw an exception.
  • Michael Ozeryansky
    Michael Ozeryansky over 6 years
    #11 is the key!
  • pkamb
    pkamb over 4 years
    The docs for addTransationObserver: state "This may require that the user authenticate." - so this is seemingly correct/expected behavior if you call that method.