hassanakbar4 / mailarchive-tickets

0 stars 0 forks source link

Access to attatchments is inconsistent #227

Closed hassanakbar4 closed 3 years ago

hassanakbar4 commented 6 years ago

component_MailArchive: User Interface resolution_fixed type_enhancement | by jason@zx2c4.com


(Sent to the django-project@ietf.org address):

Hello,

I was reading an email over on ietf.org/mail-archive when I saw in the
headers listed an "Archived-at:" listing what appeared to be a more
official link over on the mailarchive.ietf.org domain, so I went to
have a look. I then noticed two discrepancies with the way your email
software is processing attachments, I think related to their mime
types. Usually I'd shrug something like this off, but I saw the link
at the bottom of the page entitled "report a bug by email", so I
figure you're actually interested in hearing about these sorts of
minutiae.

Observation 1)

[a] https://www.ietf.org/mail-archive/web/cfrg/current/msg09495.html
[b] https://www.ietf.org/mail-archive/web/cfrg/current/msg09496.html

Notice that [a] shows the attachment as text in-line, while [b] shows
the attachment as an actual attachment link.

Observation 2)

[c] https://mailarchive.ietf.org/arch/msg/cfrg/JSe2K7jDQ_fwkI5wgqlAOONW0N4
[d] https://mailarchive.ietf.org/arch/msg/cfrg/fS2v14HTIvmAItqOkZQnZ2haGmo

Notice that [c] is entirely stripped of its attachment, while [d]
shows the attachment as an actual attachment link.

I'd suggest that the attachment in [a][c] be handled in the same way
as the attachment in [b][d] -- namely, showing it as an actual
attachment.

Thanks,
Jason

Issue migrated from trac:2456 at 2021-09-22 16:52:31 +0500

hassanakbar4 commented 6 years ago

@hassanakbar4 changed status from new to closed

hassanakbar4 commented 6 years ago

@hassanakbar4 changed resolution from ` tofixed`

hassanakbar4 commented 6 years ago

@hassanakbar4 commented


Fixed in [812/mailarch]:

Refactor attachment handling to use copy of /etc/mime.types and serve as needed rather than creating files. Fixes #2456