This code is actually only ever used for handling messageTypes.IMAGES, which doesn't have any video. We accidentally included it, but it's never used.
Details
Details
To confirm that the fetch path would never hit videos, I ran this query on my production keyserver and got no results:
SELECT m.*, u.type AS upload_type FROM messages m LEFT JOIN uploads u ON u.container = m.id WHERE m.type = 14 AND u.type != 'photo' LIMIT 5;
I also consulted @atul, and we read through the code to make sure the upload path doesn't use messageTypes.IMAGES if there are any videos. Also Flow.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable