Make the payments endpoint configurable

[asdesigned]

Recently a data issue with Paysafe meant that the api.netbanx endpoint wasn't working properly, but the api.paysafe one was. Currently the SDK only allows LIVE or TEST environment parameters, both use the netbanx endpoint. Would be much nicer to allow an endpoint to be configurable if required

[francoisneron]

Hi @asdesigned, actually both endpoint should be changed to api.paysafe.com or api.test.paysafe.com, but we still support the old URL. The endpoint should be hard-coded in the code since it's static value and should never change.

The data issue you are talking about was it really on api.netbanx in PROD?
Do you have timestamp, request, response, logs about the issue?
Did you notify our [email protected] about this?

paysafe_sdk_csharp/Paysafe/PaysafeApiClient.cs

Lines 108 to 121 in bb06ba5

	protected void environment(Environment newEnvironment)
	{
	this.apiEnvironment = newEnvironment;
	if (this.apiEnvironment == Environment.TEST)
	{
	this.apiEndPoint = "https://api.test.netbanx.com";
	}
	else
	{
	this.apiEndPoint = "https://api.netbanx.com";
	}
	}

[asdesigned]

Hi, agreed api.paysafe.com would be a better alternative. Yes the api.netbanx.com was having data replication issues that meant that the customer vault was replicating single use tokens quickly enough so a a retry method had to be put in the code to create a single use token and then keep retrying until a card could be created for a customer.

…

On 13 June 2018 at 23:58, François Néron ***@***.***> wrote: Hi @asdesigned <https://github.com/asdesigned>, actually both endpoint should be changed to api.paysafe.com or api.test.paysafe.com, but we still support the old URL. The endpoint should be hard-coded in the code since it's static value and should never change. The data issue you are talking about was it really on api.netbanx in PROD? Do you have timestamp, request, response, logs about the issue? Did you notify our ***@***.*** about this? https://github.com/paysafegroup/paysafe_sdk_csharp/blob/ bb06ba5/Paysafe/ PaysafeApiClient.cs#L108-L121 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#11 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AkpU9MqgY47me5JqOZ_0tvkrzeKJHeIaks5t8ZkjgaJpZM4UmUmX> .

-- Kind Regards Andy