Since the introduction of quoting of header content in https://review.openstack.org/568698, the 'x-image-meta-location' header
has been broken, because urllib.quote() is really intended to be applied
to only the path section of a URL, but in this case, it gets applied to
the entire URL, and catches the colon that separates the scheme from the
remainder of the URL.
This change adds the colon to the list of characters that should not get
quoted. Since a colon can be directly represented in ASCII, this should
not invalidate the previous change.
Reviewed: https:/ /review. openstack. org/597730 /git.openstack. org/cgit/ openstack/ python- glanceclient/ commit/ ?id=1156346dc24 3dc46bcc7c78a64 454ff4bae7ddc5
Committed: https:/
Submitter: Zuul
Branch: master
commit 1156346dc243dc4 6bcc7c78a64454f f4bae7ddc5
Author: imacdonn <email address hidden>
Date: Thu Nov 1 21:36:11 2018 +0000
Don't quote colon in HTTP headers
Since the introduction of quoting of header content in /review. openstack. org/568698, the 'x-image- meta-location' header
https:/
has been broken, because urllib.quote() is really intended to be applied
to only the path section of a URL, but in this case, it gets applied to
the entire URL, and catches the colon that separates the scheme from the
remainder of the URL.
This change adds the colon to the list of characters that should not get
quoted. Since a colon can be directly represented in ASCII, this should
not invalidate the previous change.
Change-Id: I76a1c9a361b6c9 f6eb95ae766b8c3 bcf2267703a
Closes-Bug: #1788942