Thomas Weißschuh <[email protected]> wrote:
> +++ b/lib/PublicInbox/Mbox.pm
> @@ -58,10 +58,10 @@ sub res_hdr ($$) {
>       my @hdr = ('Content-Type');
>       if ($ctx->{ibx}->{obfuscate}) {
>               # obfuscation is stupid, but maybe scrapers are, too...
> -             push @hdr, 'application/mbox';
> +             push @hdr, 'application/mbox; charset=UTF-8';
>               $fn .= '.mbox';
>       } else {
> -             push @hdr, 'text/plain';
> +             push @hdr, 'text/plain; charset=UTF-8';
>               $fn .= '.txt';

Applied and pushed patches 1 + 2, thanks.  This (3/3) seems
incorrect for non-UTF-8-compatible messages.

I should have a better approach for this in the next day or so.
The correct approach would be to use the Content-Type from the
$eml object, but the $eml object isn't likely in memory when
res_hdr() is called.

I was actually doing some surgery with the WwwStream /
GzipFilter async response components earlier, soI'll probably
get reading charset supported, soon.

Thanks for bringing this up.
--
unsubscribe: one-click, see List-Unsubscribe header
archive: https://public-inbox.org/meta/

Reply via email to