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

Biometric prompt, Jailbreak / Root detection and screenshot prevention #8645

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

enahum
Copy link
Contributor

@enahum enahum commented Mar 6, 2025

Summary

Added a security manager class to control the new security features introduced on a per-server level in addition to the app level managed by EMM providers.

This features are:

  • Prompt for biometric authentication
  • Jailbreak / root detection
  • Screen capture prevention

These features will work on a per-server basis only if not set with a EMM provider.

In order to prevent screenshots on iOS (the way it works is detailed in the library PR) we are adding a nativeID to views on screens and components where shielded means that it should be protected and skip.shielded means that we won't be protecting this view (for screens like the ones in the login flow)

Related PR's:
Server: mattermost/mattermost#30411
Library: mattermost/react-native-emm#18

Ticket Link

https://mattermost.atlassian.net/browse/MM-63362
https://mattermost.atlassian.net/browse/MM-63363
https://mattermost.atlassian.net/browse/MM-63364

Checklist

  • Added or updated unit tests (required for all new features)
  • Includes text changes and localization file updates
  • Have tested against the 5 core themes to ensure consistency between them.
  • Have run E2E tests by adding label E2E iOS tests for PR.

Device Information

This PR was tested on:
Android 14 & 15 emulators
iPhone 16 Pro and iPad 13 Pro iOS 18

Screenshots

Will add if requested

Release Note

Added the ability to configure per server (enterprise and premium) prompting for biometric authentication, preventing the use of the app if the device is jailbroken / rooted and prevent screen captures

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester release-note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants