diff options
author | Dylan Baker <dylan@pnwbakers.com> | 2018-08-11 15:27:53 -0700 |
---|---|---|
committer | Patrick Totzke <patricktotzke@gmail.com> | 2018-08-13 09:25:34 +0100 |
commit | 536f39b0586e1a117f7d7580ae3609865b76d4f9 (patch) | |
tree | 72f10b04cbea0bfa775d163bcbf4b463d004c9d2 /extra | |
parent | 696bcbefb4e95036259eab2fc876377c97bb9738 (diff) |
Don't try to add Content-Disposition header to email that already has one
Currently in db.utils.extract_body we try to add a Content-Disposition
header, regardless of whether an email already has one. This isn't
actually legal, and individual mime message may only have one C-D. The
correction is to replace the header with the modified header if it
already exists, and to add a new one only if the message doesn't have
one.
Note that I haven't actually seen a message that hits the path that
would need "add_header", only "replace_header". I have however included
it because it would be a behavioral change to not handle that case.
Fixes #1297
Diffstat (limited to 'extra')
0 files changed, 0 insertions, 0 deletions