feat: strip metadata from profile pictures (WPB-11170)

[yamilmedina]

WPB-11170 [Android] Strip Exif data from profile pictures before upload

---

PR Submission Checklist for internal contributors

• The PR Title

  • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
  • contains a reference JIRA issue number like SQPIT-764
  • answers the question: If merged, this PR will: ... ³

• The PR Description

  • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

---

What's new in this PR?

Issues

Make sure that we don't upload Exif metadata to profile pictures.

Solutions

• Introduce ExifHandler, so we centralized in there, Exif manipulation of images
• Removes unnecessary metadata (keeping Orientation)
• Paying tech debt, old logic performed in the UI, now is moved to the ViewModel and remove unused code.

Dependencies (Optional)

wireapp/kalium#3089

Needs releases with:

• GitHub link to other pull request

Testing

Test Coverage (Optional)

• I have added automated test to this contribution

Attachments (Optional)

Original image

Screen.Recording.2024-11-04.at.17.34.00.mov

Uploading avatar:

Screen.Recording.2024-11-04.at.17.34.37.mov

Result uploaded image:

Screen.Recording.2024-11-04.at.17.39.22.mov

---

PR Post Submission Checklist for internal contributors (Optional)

• Wire's Github Workflow has automatically linked the PR to a JIRA issue

---

PR Post Merge Checklist for internal contributors

• If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

---

References

1. https://sparkbox.com/foundry/semantic_commit_messages
2. https://github.com/wireapp/.github#usage
3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 86.80203% with 26 lines in your changes missing coverage. Please review.

Project coverage is 45.99%. Comparing base (e3c1dee) to head (ef361ff).
Report is 2 commits behind head on develop.

Files with missing lines	Patch %	Lines
...c/main/kotlin/com/wire/android/util/ExifHandler.kt	91.66%	14 Missing ⚠️
...src/main/kotlin/com/wire/android/util/ImageUtil.kt	52.63%	7 Missing and 2 partials ⚠️
.../src/main/kotlin/com/wire/android/util/FileUtil.kt	0.00%	3 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3590      +/-   ##
===========================================
+ Coverage    45.33%   45.99%   +0.65%     
===========================================
  Files          472      472              
  Lines        15794    15948     +154     
  Branches      2632     2630       -2     
===========================================
+ Hits          7160     7335     +175     
+ Misses        7884     7855      -29     
- Partials       750      758       +8     

Files with missing lines	Coverage Δ
.../userprofile/avatarpicker/AvatarPickerViewModel.kt	89.87% <100.00%> (+0.83%)	⬆️
...id/ui/userprofile/self/SelfUserProfileViewModel.kt	56.71% <ø> (+3.65%)	⬆️
.../src/main/kotlin/com/wire/android/util/FileUtil.kt	18.66% <0.00%> (+0.28%)	⬆️
...src/main/kotlin/com/wire/android/util/ImageUtil.kt	33.33% <52.63%> (+31.51%)	⬆️
...c/main/kotlin/com/wire/android/util/ExifHandler.kt	91.66% <91.66%> (ø)

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e3c1dee...ef361ff. Read the comment docs.

[github-actions]

Built wire-android-staging-compat-pr-3590.apk is available for download

[github-actions]

Built wire-android-dev-debug-pr-3590.apk is available for download

[github-actions]

Built wire-android-staging-compat-pr-3590.apk is available for download

[github-actions]

Built wire-android-dev-debug-pr-3590.apk is available for download

[Garzas]

Looking good, just one comment about logs 😄

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[github-actions]

Built wire-android-staging-compat-pr-3590.apk is available for download

[github-actions]

Built wire-android-dev-debug-pr-3590.apk is available for download