Skip to content

Commit

Permalink
set a (very large) alt text length limit
Browse files Browse the repository at this point in the history
The motivation is to have *some* size limit on every string in post
records, to maximize interoperation. For example, we currently have a
CBOR library rejecting some records because of too-long strings.

We don't want to limit the ability of folks to be very descriptive in
alt text, specifically, so chose what seems to be a very large limit. If
this is not large enough, based on feedback, we can bump it even higher.
As context this is the largest string length limit in all of our
lexicons.
  • Loading branch information
bnewbold committed Dec 28, 2023
1 parent 6b69822 commit 4d6ec9c
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions lexicons/app/bsky/embed/images.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,11 @@
"accept": ["image/*"],
"maxSize": 1000000
},
"alt": { "type": "string" },
"alt": {
"type": "string",
"maxGraphemes": 5000,
"maxLength": 50000
},
"aspectRatio": { "type": "ref", "ref": "#aspectRatio" }
}
},
Expand Down Expand Up @@ -53,7 +57,11 @@
"properties": {
"thumb": { "type": "string" },
"fullsize": { "type": "string" },
"alt": { "type": "string" },
"alt": {
"type": "string",
"maxGraphemes": 5000,
"maxLength": 50000
},
"aspectRatio": { "type": "ref", "ref": "#aspectRatio" }
}
}
Expand Down

0 comments on commit 4d6ec9c

Please sign in to comment.