WiredUK / SagePay.IntegrationKit

The source code for the SagePay Integration Kit for .Net
10 stars 11 forks source link

DataObject missing VAA property - null reference exception in GetDirectPaymentResult #3

Open Andycharalambous opened 4 years ago

Andycharalambous commented 4 years ago

Example response from live sagepay vspdirect endpoint (oddly enough, the test endpoint doesn't include this value in the response, so we never encountered this problem before):

VPSProtocol=3.00&Status=3DAUTH&StatusDetail=2007 : Please redirect your customer to the ACSURL, passing the MD and PaReq.&3DSecureStatus=OK&MD=&ACSURL=&VAA=A&PAReq=

System.NullReferenceException: Object reference not set to an instance of an object. at SagePay.IntegrationKit.Standard.SagePayIntegration.GetDirectPaymentResult(String resultQueryString)

Happy to create a pull request for this.

WiredUK commented 4 years ago

Please do! I've not used SagePay for a few years now, so I can't really test it.

Andycharalambous commented 4 years ago

Great - ok will do. Have hacked around it for now as I had a deadline to meet, but would love to patch this.