nat
1월 2, 2025, 5:04오후
16
간단히 업데이트드립니다 -
위에서 언급한 우리가 사용하는 ext_mime_db는 IANA 미디어 유형 을 따릅니다. 안타깝게도 이는 mp4가 application/mp4로 설정되는 것이 올바른 것임을 의미합니다
다만, S3 업로더 구현을 더 자세히 살펴보니 이미지 이외의 거의 모든 업로드에 Content-Disposition 헤더 "attachment"가 추가되고 있음을 확인했습니다. 이는 본래 svg 파일에만 추가될 의도였던 것으로 보입니다. "attachment"를 사용하면 콘텐츠가 새 탭에서 열리는 대신 다운로드됩니다. 영상의 경우 application/video와 attachment가 함께 사용되면서 이러한 문제가 발생하고 있습니다.
최소 이 헤더는 제거할 수 있을 것 같습니다.
main ← use-inline-no-download
merged 02:32AM - 07 Jan 25 UTC
Back then in https://github.com/discourse/discourse/commit/31e31ef44973dc4daaee2… f010d71588ea5873b53, we added the [Content-Disposition](https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Disposition) headers so that SVGs get downloaded instead of get run in the browser. Inadvertently, this also causes other attachments like pdfs and videos to be downloaded instead of heeding the "Open in new tab" option that users choose in the browser.
### Problem video with `Content-Disposition: attachment`:
https://github.com/user-attachments/assets/f05d388f-4454-4615-9d5a-7a0a66fe3b52
When the header is removed, the default value is `"inline"`, this allows the browser to perform as requested. This also applies to other file types like pdfs, allowing users to "Open in new tab" and view them in the browser instead of always downloading them.
Existing tests (https://github.com/discourse/discourse/pull/10205) already do check that SVGs remain downloaded. Some existing tests written for PDFs have been modified to cater for SVGs instead, when there was a bug in defining the filenames per https://github.com/discourse/discourse/pull/10108
3개의 좋아요
