-
Notifications
You must be signed in to change notification settings - Fork 375
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MSC2702: Specifying semantics for Content-Disposition on media #2702
Conversation
Implementation: t2bot/matrix-media-repo@da0be74 |
Co-authored-by: Andrew Morgan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This review contains 3 comments but 2 of them are old ones. I will try to remove them after submitting this.
I think this is ready to go as a "nice to have" item for Matrix 1.9/the future: @mscbot fcp merge |
Team member @mscbot has proposed to merge this. The next step is review by the rest of the tagged people: Concerns:
Once at least 75% of reviewers approve (and there are no outstanding concerns), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up! See this document for information about what commands tagged team members can give me. |
Co-authored-by: David Baker <[email protected]>
* `audio/x-wav` | ||
* `audio/x-pn-wav` | ||
* `audio/flac` | ||
* `audio/x-flac` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the sake of anyone (like me) who's surprised that audio/opus
isn't on the list, the audio/opus
mimetype seems to be specifically for RTP payloads. Recorded files (such as what we would use in voice messages) would be served with the audio/ogg
mimetype.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, flac has its own container format for some reason. Opus just lives in ogg/webm/mp4 (it's the hermit crab of codecs).
🔔 This is now entering its final comment period, as per the review above. 🔔 |
The final comment period, with a disposition to merge, as per the review above, is now complete. |
* Proposal to introduce sane Content-Disposition semantics * Update proposals/2702-media-content-disposition.md Co-authored-by: Andrew Morgan <[email protected]> * Update for the year 2023 * Clarify thumbnails receiving Content-Disposition header * Clarify layering * Update proposals/2702-media-content-disposition.md Co-authored-by: David Baker <[email protected]> * Fix wording * Apply suggestions from code review Co-authored-by: Richard van der Hoff <[email protected]> * Clarify wording * Add clarifying words * Adjust security section --------- Co-authored-by: Andrew Morgan <[email protected]> Co-authored-by: David Baker <[email protected]> Co-authored-by: Richard van der Hoff <[email protected]>
Spec PR: matrix-org/matrix-spec#1758 |
Rendered
Implementations:
t2bot/matrix-media-repo@da0be74- legacyFCP tickyboxes