-
Notifications
You must be signed in to change notification settings - Fork 3
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
No where to pass AppPayload for android purchases. #1
Comments
the problem with developer payload is that it is simply not supported in iOS and I will to make a library which can be used on both platforms the same. as this is not supported in iOS, I decided against it. |
Ah... |
Sure you can verify the purchase even without payload. if you cant use the server side verification, the plugin does a local validation of the purchase data integrity using your public key. even though this is not as safe as server side verification (as the application may be "cracked") but it makes sure that the data passed to your code is at least signed with the right private key (which only Google has). |
I understand your point (though not sure I agree :) ). In any case, I was looking forward at your master branch, and was wondering if you could add to the buy() response the full original payload (including full purchase data json and data signature). That could just be in a originalPayload field in the Purchase (js code) and will be clearly labled platform dependant. |
@yarontt sorry for late response, Im totally out of time now a days. regarding the original payload, I think there is one problem here, and that is, that there is no such thing in iOS! once the payment is processed on iOS, you can only get the purchases from app receipt and there are all the purchases listed together. I can see your point, but you have to consider that my goal in this plugin was to help developer deal with ONE api instead of two complete sets of apis. you are looking at this as stand point of an android developer, and the other side of story can also happen when a iOS developer asks for some iOS specific features. if you are really looking into having all the details that each platform provides, then you are searching in the wrong place and better use two separate plugins. |
I'm actually using your plugin because it does support iOS. I'm writing both code paths as I go along. I think the SKPaymentTransaction (a json of it) might suite what I mentioned as an original payload since that's what the iOS callback gets. I think the main thing is this: Your plugin is x-plat, however on the server side there's no x-plat code. On Android it's possible to validate server side without calling any Google backend. Providing the exact original payload returned to the app, allows developers to be more flexible with the server side logic. Considering the fact that there IS an original payload in iOS (SKPaymentTransaction), I would urge you to reconsider my request. |
@yarontt sorry for my late reply, I'm currently in a situation that remains me scarce time for putting on side projects, I'm not sure when can I put some time on these changes. |
Hi, For most apps (at least in their early days), no one will put a lot of effort in hacking them. In any case, I don't want to invest time in server-to-server validation, and I think adding the original payload to your API doesn't break x-platform in any way. Regards |
sorry, since this campaign, I decided to stop any further development to this plugin: https://www.indiegogo.com/projects/phonegap-cordova-in-app-purchase-ios-and-android |
got it. |
sorry I'm not going to be involved in that project. I dont believe they will also adopt my changes. anyway I hope also much success for that project. |
Hi,
The plugin is missing a way to pass App Payloads in the Android billing flow.
I saw you had partial support for that (in the .java files) that was deleted. Is there a plan to support this?
I'm willing to help out with this.
The text was updated successfully, but these errors were encountered: