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.

Sign up for GitHub

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

Jump to bottom

onScreenCapturedWithDeniedPermission may be incorrect #5

Closed
Lewin671 opened this issue Oct 10, 2021 · 1 comment
Closed

onScreenCapturedWithDeniedPermission may be incorrect #5

Lewin671 opened this issue Oct 10, 2021 · 1 comment

Comments

@Lewin671
Copy link

Lewin671 commented Oct 10, 2021 •
edited

Hi, I have seen all the source codes of this project, and I notice that onScreenCapturedWithDeniedPermission may cause some unexpected results.

For example, if the external disk is put something new, then you will detect a new file and invoke onContentChanged, but it's actually not the screen capture.
@akexorcist
Copy link
Owner

This seem removing onScreenCapturedWithDeniedPermission might be the solution for this case. Because we cannot check the actual path of file without permission.

@akexorcist akexorcist mentioned this issue Nov 9, 2021
Closed
akexorcist added a commit that referenced this issue Nov 9, 2021
@akexorcist
Remove onScreenCapturedWithDeniedPermission
b99a55b 
Because it called when non screenshot content change with denied permission (#4 #5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@akexorcist @Lewin671