Does the AppView not currently enforce image blob size limits or accept types? #2845
Replies: 3 comments
-
|
We're currently adding image processing in snarfed/granary#844 to get aspect ratios, and we'll probably use our own higher limit for images, maybe 5MB, since the appview still doesn't seem to enforce the lexicon's 1M limit. |
Beta Was this translation helpful? Give feedback.
-
|
Catching up on some old reports! This one is quite interesting. I think the short answer is that the bsky appview implementation is not being strict enough here; these posts should be rejected. cc: @devinivy @ericvolp12 |
Beta Was this translation helpful? Give feedback.
-
|
The plot thickens. Evidently the Bluesky CDN has an (undocumented?) limit of 10MB or so for images? We get occasional reports of images getting corrupted when they're over that size, eg snarfed/bridgy-fed#2107, snarfed/bridgy-fed#1925. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Hi all! I recently implemented (well, finished) lexicon schema validation for stored/emitted records and XRPC method input, output, and params, and I've been seeing a trickle of surprising failures on data that the AppView is evidently ok with. One big one: it seems like image blobs are often allowed with types and sizes that don't satisfy
maxSizeoraccept. Is this known?A couple specific examples:
size1858016, which is larger thanmaxSize: 1000000onapp.bsky.embed.images#image, but the AppView allowed it and it's visible on the post.avatar(getBlob call) withmime_type: image/webp, which doesn't satisfyaccept: [image/png, image/jpeg]onapp.bsky.actor.profile, but the AppView allowed it and it's visible on the profile.Totally ok if this is expected! I'm just trying to understand how to reconcile it with any lexicon schema validation I do on my end.
Thanks in advance!
(re: snarfed/bridgy-fed#1348, snarfed/bridgy-fed#1349 (comment))
Beta Was this translation helpful? Give feedback.
All reactions