Skip to content
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

Paypal payments not working - INVALID_EXPERIENCE_PROFILE_ID #393

Closed
icougil opened this issue Feb 18, 2018 · 4 comments
Closed

Paypal payments not working - INVALID_EXPERIENCE_PROFILE_ID #393

icougil opened this issue Feb 18, 2018 · 4 comments
Assignees
Labels

Comments

@icougil
Copy link

icougil commented Feb 18, 2018

Hi.

We've found a new issue with Paypal. We have just updated the real client ID and client secret (only in the system configuration, we didn't setup anything in any other place) and we found that now it is not possible to pay with Paypal (even we checked that everything was working perfectly in the "demo mode" with the Paypal demo user/pwd).

This is what we are seeing in the log:

ERROR com.paypal.base.HttpConnection - Response code: 400 Error response: {"name":"INVALID_EXPERIENCE_PROFILE_ID","message":"The requested experience profile ID was not found","information_link":"https://developer.paypal.com/docs/api/payments/#errors","debug_id":"XXXXXX"

What we are doing wrong? What we can do?

Best,

@syjer syjer self-assigned this Feb 18, 2018
@syjer syjer added the bug label Feb 18, 2018
@syjer
Copy link
Member

syjer commented Feb 18, 2018

hi @cougil , actually you found a bug :). For performance reason I've cached the profile id (see https://github.com/alfio-event/alf.io/blob/master/src/main/java/alfio/manager/PaypalManager.java#L59 ).

But I've not taken in account the change of the client id for a given event/locale pair (as the key of the map is generated using https://github.com/alfio-event/alf.io/blob/master/src/main/java/alfio/manager/PaypalManager.java#L72 ) . As a simple workaround, you can restart the instance so the cache is empty.

TODO:

  • generate the key of the cache using the client_id too, it should be enough
  • switch to caffeine for the caching (?)

@icougil
Copy link
Author

icougil commented Feb 18, 2018

Done, thank for your super fast reply!
It is working right now after the restart ;-)

@icougil icougil closed this as completed Feb 18, 2018
@syjer
Copy link
Member

syjer commented Feb 18, 2018

@cougil I'll reopen the issue so I can track my commit with it:)

@syjer
Copy link
Member

syjer commented Feb 19, 2018

should be fixed now, @cougil thanks for finding the bug 👍

@syjer syjer closed this as completed Feb 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants