It's risky (and unnecessary) to apply check_plain() filtering to the data components of the transaction response hash, as the module currently does. The SHA version of the hash now includes customer-entered data, and some of this potentially contains characters that would be encoded by check_plain(). The module should instead use the raw string values returned by Authorize.Net, as these presumably were used to generate Authorize's version of the hash value.
Patch to follow.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | use_raw_hash_data-3030919-2.patch | 3.85 KB | jerry |
Comments
Comment #2
jerry commentedPatch attached.
Comment #4
jerry commented