You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The documentation does lack some details about how the notification is actually verified. I think the SDK is probably the main source of information for this.
The text was updated successfully, but these errors were encountered:
Without tests, the main functionality is available.
Where signatures are validated in notification handlers is a little unclear in Omnipay across the drivers. It cannot be done in message instantiation, as the constructor does not have access to the gateway parameters (these are set using initialize()after instantiation. It could be done at that point, however, throwing an exception here means there is no object to inspect and log when debugging.
The approach I have taken, is to assert the signature validity at the point the transaction result is fetched - all the methods in the NotificationInterface. Until that point, the object will be instantiated, you can extract the raw and parsed data, and can manually do a check isSignatureValid().
The notification message comes with a signature in the header, for example:
This should be used to validate the notification has not been tampered with.
Some details:
The documentation does lack some details about how the notification is actually verified. I think the SDK is probably the main source of information for this.
The text was updated successfully, but these errors were encountered: