Skip to content
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

fix: W/a sending images sent as stickers on some platforms (#4611) #4619

Merged
merged 2 commits into from
Aug 28, 2023

Conversation

iequidoo
Copy link
Collaborator

Check if a sticker has at least one fully transparent corner and otherwise change the Sticker type to Image. This would fix both Android and iOS at the same time and prevent similar bug on future platforms that may get this bug like Ubuntu Touch.

@iequidoo iequidoo force-pushed the iequidoo/sticker-to-image branch 2 times, most recently from 694d1b5 to eeff507 Compare August 14, 2023 02:53
@iequidoo iequidoo marked this pull request as ready for review August 14, 2023 03:13
@iequidoo iequidoo requested review from r10s and link2xt August 14, 2023 03:14
@iequidoo iequidoo force-pushed the iequidoo/sticker-to-image branch from 642e899 to 0be1d31 Compare August 15, 2023 21:53
@iequidoo iequidoo requested a review from link2xt August 17, 2023 17:27
@link2xt
Copy link
Collaborator

link2xt commented Aug 25, 2023

Could this change target stable branch instead of master?

@iequidoo iequidoo changed the base branch from master to stable August 25, 2023 14:47
@iequidoo iequidoo force-pushed the iequidoo/sticker-to-image branch from 0be1d31 to 84c4725 Compare August 25, 2023 15:05
@iequidoo iequidoo force-pushed the iequidoo/sticker-to-image branch from 84c4725 to 2148a3b Compare August 27, 2023 14:15
Check if a sticker has at least one fully transparent corner and otherwise change the Sticker type
to Image. This would fix both Android and iOS at the same time and prevent similar bug on future
platforms that may get this bug like Ubuntu Touch.
@iequidoo iequidoo changed the base branch from stable to master August 27, 2023 14:22
…as Exif

We mustn't send images with Exif as it can leak metadata such as location, camera model, etc.
@iequidoo iequidoo force-pushed the iequidoo/sticker-to-image branch from 2de5ce3 to 97455ed Compare August 27, 2023 14:23
@iequidoo iequidoo changed the base branch from master to stable August 27, 2023 14:23
@iequidoo iequidoo requested a review from link2xt August 27, 2023 14:25
@iequidoo iequidoo merged commit 3e07f2c into stable Aug 28, 2023
@iequidoo iequidoo deleted the iequidoo/sticker-to-image branch August 28, 2023 02:16
@r10s
Copy link
Contributor

r10s commented Aug 28, 2023

great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants