Hide app content in app launcher (EXPOSUREAPP-2612) #1390
Conversation
|
Works on a Huawei CLT-L09 (Android 8.1.0): Works on a Pixel 3a Emulator (Android 11): Doesn't work on a rooted Pixel 3a (Android 11): So is the flag here automatically ignored if the device is rooted? Seems weird 🤔 Does it check the unlocked bootloader? |
see https://developer.android.com/reference/android/view/Display#FLAG_SECURE Seems to me, that on rooted devices, the app launcher can choose itself what to show, and as the app sandboxing is circumvented the encrypted data can be accessed nevertheless. Hence several posts mentioned that the flag is not reliable on rooted devices. |
d4rken
left a comment
There was a problem hiding this comment.
lgtm, if possible confirm behavior on an unrooted Pixel 3a (API30), but not a blocker, current behavior is already better than before.
|
|
|
@d4rken tested in a non-rooted Pixel 3A. Same behavior as on the rooted one. |
|
I think screenshots should not be disabled. I don't think an app should tell the user what he/she can do with their device. If they want to take a screenshot, they can drop their phone on a copier 🤷, disabling it just makes it cumbersome. If someone wants to take screenshots, let them. I also don't see a huge risk with the app content showing in the launcher, if the device is locked, no one can see it, if the device is unlocked, someone can just open the app and view it. While I agree that you may show someone else your device and could inadvertently reveal the corona app's screen content, I don't think it's an important enough "feature" to warrant disabling screenshots. Final decision probably lies with the POs here then. |
a671065 to
b91ae41
Compare
b91ae41 to
efa96e3
Compare
BMItr
left a comment
There was a problem hiding this comment.
already tested, with known result. approved
|
Kudos, SonarCloud Quality Gate passed!
|
Description
How to test
Run the app and validate the above-mentioned criteria on any screen you can navigate to:
Screenshots
