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

Conflict between prod and sandbox api #229

Open
lukaszbajorski-filmbooking opened this issue Oct 17, 2024 · 6 comments
Open

Conflict between prod and sandbox api #229

lukaszbajorski-filmbooking opened this issue Oct 17, 2024 · 6 comments

Comments

@lukaszbajorski-filmbooking

Hello folks,

In our use case we need to be able to switch between prod and sandbox mangopay endpoints dynamically, depending on the data in the running system. Unfortunately, it seems there's an underlying conflict making it impossible. Perhaps a state that the SDK keeps that prevents changing Client+ClientPassword+BaseUrl combo on the fly?

Any help or direction will be appreciated.

Cheers,
Łukasz

@lukaszbajorski-filmbooking
Copy link
Author

lukaszbajorski-filmbooking commented Oct 17, 2024

Hello again,

We've downloaded the source code of the SDK, did some searching and found the issue.

The cause of the problem is usage of singleton RestSharpDto instead of map-per-url.

I can push the code or you can make the changes yourselves, it's not a biggie:

image

Cheers,
Ł.

PS - as a side issue, the HTTP response code returned with the erroneous behaviour was 0 and the message null which caused a NPE in the SDK:

image

@lukaszbajorski-filmbooking
Copy link
Author

lukaszbajorski-filmbooking commented Oct 18, 2024

PR with the changes: #230

@lukaszbajorski-filmbooking
Copy link
Author

Hello @iulian03, I've just noticed there's a new 3.19.3 version of the SDK but the error above stops us from migrating to it. Could you include the fix we provided?

@iulian03
Copy link
Contributor

Hi @lukaszbajorski-filmbooking ,
We will take a look at this. Thank you for the PR!

@lukaszbajorski-filmbooking
Copy link
Author

Thanks @iulian03!

@lukaszbajorski-filmbooking
Copy link
Author

Hello @iulian03,

Any luck with planning this in?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants