-
Notifications
You must be signed in to change notification settings - Fork 399
MSC2448: Using BlurHash as a Placeholder for Matrix Media #2448
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
Open
anoadragon453
wants to merge
52
commits into
old_master
Choose a base branch
from
anoa/blurhash
base: old_master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
8ba6071
Add blurhash MSC
anoadragon453 f71535a
some cleanup
anoadragon453 8adf2b2
Add blurhash to img tag, make things optional, /upload mod
anoadragon453 56703fc
unstable prefixes, no encoding needed
anoadragon453 793107d
Move m.r.m msgtype into the m.r.m section
anoadragon453 777c30c
hm
anoadragon453 b80822e
data-mx-blurhash
anoadragon453 b240d92
Must HTML encode, link to blurhash algo description
anoadragon453 1afad01
New endpoint, not new key on existing endpoint
anoadragon453 60773e4
No HTML encoding warning
anoadragon453 df5b6d7
Straight mxc->blurhash endpoint
anoadragon453 571ce2a
Unstable feature flag
anoadragon453 e885eae
blurhash in "info", download->blurhash
anoadragon453 4b83c51
Merge branch 'anoa/blurhash' of github.com:matrix-org/matrix-doc into…
anoadragon453 b893c21
Get an image from a blurhash, get a blurhash from an MXC url
anoadragon453 6e8eb59
Merge branch 'anoa/blurhash' of github.com:matrix-org/matrix-doc into…
anoadragon453 3695ecf
No GET body
anoadragon453 842c2a0
Update proposals/2448-blurhash-for-media.md
anoadragon453 3994010
Explicitly state blurhashes are not exempt from encryption
anoadragon453 616bc81
/upload -> xyz.amorgan.blurhash instead
anoadragon453 e0a7442
Add blurhashes to URL previews
anoadragon453 385be8a
Clarify that 'blurhash' should go in content.info
anoadragon453 708b756
formatting
anoadragon453 b761b06
Add blurhash to m.sticker events
anoadragon453 40d71ff
Add m.room.member
anoadragon453 7f13184
Add m.room.avatar
anoadragon453 1300a6e
Update unstable prefixes section for new event types
anoadragon453 2a02d2c
Specify endpoints that will need to be modified to support avatar blu…
anoadragon453 7ea82b4
Remove the controversial /_matrix/media/r0/blurhash endpoint
anoadragon453 f93d708
Format the Unstable prefixes section a little more nicely
anoadragon453 ed9ed5e
grammar
anoadragon453 48c4d55
Add a quick note about encrypted media and server-side blurhash calc
anoadragon453 fba60db
Rename with a more generic title
anoadragon453 c837c83
Riot -> Element
anoadragon453 b3f1915
Slight title adjustment
anoadragon453 5b8c191
Add note about negligible DoS potential of large blurhashes
anoadragon453 63d4966
Add note about invalid BlurHashes
anoadragon453 676571f
Move blurhash to m.room.avatar's content.info dict
anoadragon453 a302197
Merge branch 'master' of github.com:matrix-org/matrix-doc into anoa/b…
anoadragon453 594bcee
Apply suggestions from code review
anoadragon453 64116ae
Clarify that clients must calculate blurhashes for encrypted media
anoadragon453 9bd0ba6
Fix inconsistency of blurhashes MUST vs SHOULD be omitted
anoadragon453 e7e0fb7
blurhashes can be inserted into more than just m.room.message events
anoadragon453 1d954f0
Remove the need for an unstable_features entry
anoadragon453 9e981ba
Make blurhash generation on the server opt-in
anoadragon453 75a4fa6
Clarify why we place blurhash under the matrix:image:blurhash OG prop
anoadragon453 934b6d7
Mention that the generate_blurhash query param needs an unstable prefix
anoadragon453 974d368
lint proposal markdown
anoadragon453 5668397
Apply suggestions from TravisR
anoadragon453 234877c
Clear client recommendations for blurhashes vs. thumbnails
anoadragon453 abf5283
r0 -> v3; update links to new spec website
anoadragon453 754fa7a
Add and update request/repsonse examples
anoadragon453 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
blurhash in "info", download->blurhash
- Loading branch information
commit e885eae98a759a773b4816219eba6b1f968d0e2b
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.