Fixes for #467

Decode string in Content-disposition header parameter such as "filename" properly.

as long as it is consistent with all other places I don't care much ...
somehow getting this behavior into the encoder would be nice so it can be
easily overwritten

On Thu, Nov 20, 2014 at 10:48 PM, Benjamin Fleischer <
notifications@github.com> wrote:

> In spec/mail/attachments_list_spec.rb:
>
> >        else
> > -        expected = "01 Quien Te Dij\212at. Pitbull.mp3"
> > +        expected = "01 Quien Te Dijat. Pitbull.mp3"
>
> Do we want to change this behavior? here and in
> spec/mail/fields/content_type_field_spec.rb It looks like a regression to
> me 4d9b61a
> <https://github.com/mikel/mail/commit/4d9b61a291fa2cbcd691c01b596614d25065ea8e>
>
> instead of str.encode(Encoding::UTF_8, :invalid => :replace, :replace =>
> "") replacing with '' maybe we should use the default U+FFFD �
> <http://graphemica.com/%EF%BF%BD>?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mikel/mail/pull/819/files#r20700872>.
>