Produits Entr'ouvert » w.c.s.

Lors de la décomposition de la query-string d'une URL pour en préparer la signature, on fait un « parse_qsl » :

# extrait de wcs/api_utils.py
def sign_url_auto_orig(url):
    try:
        signature_key, orig = get_secret_and_orig(url)
    except MissingSecret:
        return url
    parsed = urllib.parse.urlparse(url)
    querystring = urllib.parse.parse_qsl(parsed.query)
    querystring.append(('orig', orig))
    querystring = urllib.parse.urlencode(querystring)
    url = urllib.parse.urlunparse(parsed[:4] + (querystring,) + parsed[5:6])
    return sign_url(url, signature_key)

Or parse_qsl a une option keep_blank_values à False par défaut : https://docs.python.org/3/library/urllib.parse.html#urllib.parse.parse_qsl

Ce qui fait que si on a un « ?vide=&foo=bar » on a juste « ?foo=bar » dans la requête reconstituée. Ça devient un bogue quand un système tiers a besoin de l'argument vide.

Je pense qu'on pourrait sans dommage ajouter un keep_blank_values=True sur le/les parse_qs(l) (et peut-être à d'autres endroits où on fait l'analyse de querystring).