Project

General

Profile

Bug #58972

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

Added by Sentry IO 9 days ago. Updated 2 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
25 Nov 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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)

Files

Associated revisions

Revision 5f70e8b6 (diff)
Added by Frédéric Péters (absent jusqu’au 12/12) 3 days ago

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

History

#1

Updated by Lauréline Guerin 9 days ago

  • Project changed from Suivi des traces to w.c.s.
#2

Updated by Frédéric Péters (absent jusqu’au 12/12) 4 days ago

  • Assignee set to Frédéric Péters (absent jusqu’au 12/12)
#3

Updated by Frédéric Péters (absent jusqu’au 12/12) 4 days ago

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

Updated by Lauréline Guerin 3 days ago

  • Status changed from Solution proposée to Solution validée
#5

Updated by Frédéric Péters (absent jusqu’au 12/12) 3 days ago

  • Status changed from Solution validée to 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

Updated by Frédéric Péters (absent jusqu’au 12/12) 2 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF