Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Steps to reproduce:
1) Set your UPS password to something with a & in it
2) Checkout as a customer and try to calculate shipping
3) Observe Error
Error: Hard 10001: The XML document is not well formed
Comment | File | Size | Author |
---|---|---|---|
#4 | ups.patch | 1 KB | TR |
| |||
#2 | ubercart-fix-xml-request-2698723-1-7.patch | 1.01 KB | Derimagia |
|
Comments
Comment #2
Derimagia CreditAttribution: Derimagia at Mindgrub Technologies commentedPatch included
Comment #3
Derimagia CreditAttribution: Derimagia at Mindgrub Technologies commentedComment #4
TR CreditAttribution: TR commentedThis issue is a subset of the larger problem raised in #2459739: No escaping of XML characters in uc_ups, namely there has never been any xml-encoding of values inserted into the UPS (or USPS) requests.
I have been working on this for the Drupal 8 version of Ubercart, and will eventually backport the fix. Attached is a patch which is a backport of just the UPS AccessRequest, which is what you're having trouble with. Can you test out this patch and let me know if it fixes the problem you're currently having?
Comment #5
Derimagia CreditAttribution: Derimagia at Mindgrub Technologies commentedSorry for the late response - Sadly I was just helping out with the project and I just told them to change their password to something else in the end. I will test this next time it's possible for me too.