Skip to content

Conversation

@philprime
Copy link
Member

This PR is still work in progress and only here for comparing cleaning up merge conflicts

@philprime philprime self-assigned this Oct 31, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 31, 2025

Fails
🚫 Please consider adding a changelog entry for the next release.
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Features

- Add alternative masking strategies ([#6619](https://github.com/getsentry/sentry-cocoa/pull/6619))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against c73f441

@codecov
Copy link

codecov bot commented Oct 31, 2025

⚠️ JUnit XML file not found

The CLI was unable to find any JUnit XML files to upload.
For more help, visit our troubleshooting guide.

@philprime philprime force-pushed the philprime/masking-dev-docs branch 2 times, most recently from 83b0f8c to c887b99 Compare October 31, 2025 13:36
@philprime philprime force-pushed the philprime/masking-dev-docs branch from c887b99 to c73f441 Compare October 31, 2025 13:44
@philprime
Copy link
Member Author

philprime commented Nov 5, 2025

Also consider these approaches:

  • Defensive-programming approach, masking everything unless there is a 100% certainty an area can be unmasked safely (safest but most complicated approach)
  • Rebuilding a wireframe using colored rectangles based on the view hierarchy, without calling draw methods (should also result in performance improvement)
  • Draw the view into a PDF context, then use PDF manipulation to remove all images (probably not possible because the rendered view is an image by itself)

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.

2 participants