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

Permissions #22

Open
IzzySoft opened this issue Jul 3, 2017 · 3 comments
Open

Permissions #22

IzzySoft opened this issue Jul 3, 2017 · 3 comments

Comments

@IzzySoft
Copy link

IzzySoft commented Jul 3, 2017

Just curious: for what does an AdBlocker need the READ_PHONE_STATE permission? This makes the app suspicious to especially privacy-aware people (I got that response when recommending your app).

If it's for the "usual purpose" (i.e. to "background" on incoming calls, which the "official developer documentation" mentions it for): I don't see how that should make sense. An AdBlocker should work in the background anyway, and configuration/settings pages make no sound and play no video. So there's nothing to interfere with phone calls. And even if there were, there are other solutions available.

TL;DR: Would you consider removing that permission requirement?

@ggsava
Copy link
Owner

ggsava commented Jul 3, 2017

Thanks @IzzySoft. I thought I already removed that, I will take a look at it again. I think it might be from something I tested in a previous version.. but I believe its not necessary anymore.

@IzzySoft
Copy link
Author

IzzySoft commented Jul 3, 2017

Thanks for your fast reaction, @ggsava – I will report that back to the reporting user, uh... "on the double" 😆 Looking forward then to the next version without that permission 👍

@IzzySoft
Copy link
Author

Almost a year later: Will there be a release anytime soon to remove that? Last release was quite a while ago, and according to the repo there has been some more development since (though the last change was 3 months back).

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

No branches or pull requests

2 participants