Possible to send > 32 images in a single message by quickly tapping the capture button

awaitlink commented 1 year ago

[x] I have searched open and closed issues for duplicates
[x] I am submitting a bug report for existing functionality that does not work as intended
[x] This isn't a feature request or a discussion topic

Bug description

It's possible to send > 32 images in a single message.

Notes on how this message is received:

Signal Android drops excess attachments (i.e. after the 32nd).
Signal Desktop drops the whole message: https://github.com/signalapp/Signal-Desktop/issues/6491

Steps to reproduce

Go to the in-app camera (chat list > camera icon).
Tap the capture button fairly quickly.

Actual result: Because capturing isn't instant, while Signal tries to catch up it ends up capturing more photos than 32, and only then showing the warning. It's possible to send > 32 images in a single message (the most I got was 39, but more may be possible, depending on the device and other conditions).

Expected result: Since the limit is there, it shouldn't be possible to send > 32 images in a single message.

Device info

Device: iPhone XR iOS version: 16.6 Public Beta (20G5047d) Signal version: 6.30.0.4

Link to debug log

Example message sent like this: 1687551066754 (with 35 images).

https://debuglogs.org/ios/6.30.0/90dcd979de7055cdbf7c0a916418601d53aea840d6d4fee0c50be1a21ea09b12.zip

stale[bot] commented 10 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] commented 9 months ago

This issue has been closed due to inactivity.

signalapp / Signal-iOS