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

[BUG] Security flags for screen recording #3468

Closed
JuancaG05 opened this issue Dec 9, 2021 · 3 comments · Fixed by #3560
Closed

[BUG] Security flags for screen recording #3468

JuancaG05 opened this issue Dec 9, 2021 · 3 comments · Fixed by #3560

Comments

@JuancaG05
Copy link
Collaborator

JuancaG05 commented Dec 9, 2021

Actual behaviour

Currently, if we set the passcode or the pattern as lock for the app, the screen is black from that screen on, in programs such as Vysor, or if we try to record the screen in some way.

Expected behaviour

The passcode or pattern unlock screen should appear in black when recording, but not the rest of the screens.

Steps to reproduce

  1. Set passcode or pattern
  2. Close the app
  3. Open the app while recording it or watching it with Vysor
  4. Unlock the app
  5. From that moment, every screen appear in black

The code responsible for this is located in MainApp.kt:

                if (!enabledLogging) {
                    // To enable biometric you need to enable passCode or pattern, so no need to add check to if
                    if (passCodeEnabled || patternCodeEnabled) {
                        activity.window.addFlags(WindowManager.LayoutParams.FLAG_SECURE)
                    } else {
                        activity.window.clearFlags(WindowManager.LayoutParams.FLAG_SECURE)
                    }
                } // else, let it go, or taking screenshots & testing will not be possible

We should check if this makes sense and if so, how to fix this unexpected behaviour.

Environment data

ownCloud app version: 2.19

@jesmrec
Copy link
Collaborator

jesmrec commented Dec 10, 2021

Howto

Debug:

  • All clear

Release:

  • Passcode / Pattern hidden
  • Login hidden

@jesmrec jesmrec added this to the 2.20-current milestone Dec 10, 2021
@michaelstingl
Copy link
Contributor

but not the rest of the screens.

Maybe other content is also worth protecting it? Branding/MDM option for full-protection vs. login-protection?

@jesmrec
Copy link
Collaborator

jesmrec commented Jan 13, 2022

How could this be useful for external users/customers? our idea is helping ourselves with that... any hidden use case?

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

Successfully merging a pull request may close this issue.

3 participants