Projet

Général

Profil

Bug #58972

BadHeaderError: Header values can't contain newlines (got 'inline; filename="Signalement-70-4025-N2 - NDP\nPlusie...

Ajouté par Sentry Io il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
25 novembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/53965/

BadHeaderError: Header values can't contain newlines (got 'inline; filename="Signalement-70-4025-N2 - NDP\nPlusieurs ndp dans toute la rue-2021-11-25_10_07.pdf"')
(3 additional frame(s) were not displayed)
...
  File "django/views/generic/base.py", line 71, in view
    return self.dispatch(request, *args, **kwargs)
  File "django/views/generic/base.py", line 97, in dispatch
    return handler(request, *args, **kwargs)
  File "wcs/compat.py", line 83, in get
    response[name] = value
  File "django/http/response.py", line 137, in __setitem__
    value = self._convert_to_charset(value, 'latin-1', mime_encode=True)
  File "django/http/response.py", line 119, in _convert_to_charset
    raise BadHeaderError("Header values can't contain newlines (got %r)" % value)

Fichiers

0001-misc-remove-invalid-characters-from-content-disposit.patch (3,02 ko) 0001-misc-remove-invalid-characters-from-content-disposit.patch Frédéric Péters, 30 novembre 2021 14:47

Révisions associées

Révision 5f70e8b6 (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

misc: remove invalid characters from content-disposition header (#58972)

Historique

#1

Mis à jour par Lauréline Guérin il y a plus de 2 ans

  • Projet changé de Suivi des traces à w.c.s.
#2

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Assigné à mis à Frédéric Péters
#3

Mis à jour par Frédéric Péters il y a plus de 2 ans

Voilà pour retirer \n \r et " du nom (les remplacer par _).

Il y a un test mais il ne correspond pas à la situation de la trace, je ne suis pas arrivé à trouver le chemin par où ça passait. Dans la trace le \n est dans le orig_filename alors que dans le code qui amène à la ligne en erreur c'est base_filename qui est utilisé et qui est déjà nettoyé. Visiblement il y a une situation où c'est mal nettoyé mais j'ai arrêté de chercher (surtout que AttachmentSubstitutionProxy retourne pour une propriété "filename" le contenu de "orig_filename" et ça commençait à faire des grep trop longs pour trouver une piste).

#4

Mis à jour par Lauréline Guérin il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée
#5

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 5f70e8b6e15eff2e7d47d0402d107834594177b4
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Nov 30 14:08:04 2021 +0100

    misc: remove invalid characters from content-disposition header (#58972)
#6

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#7

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF