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

Why microphone permission was asked #68

Open
Jean-BaptisteC opened this issue Jun 23, 2024 · 2 comments · May be fixed by #70
Open

Why microphone permission was asked #68

Jean-BaptisteC opened this issue Jun 23, 2024 · 2 comments · May be fixed by #70

Comments

@Jean-BaptisteC
Copy link
Contributor

When the user wants to take a sequence for the first time, the app asks to get access to the microphone.
IMO this permission is not necessary to take a sequence.
Keep in mind, that when you publish an app on the store, Google asks you to tell why you need this permission (Users can also ask the same question).
I have seen this permission was not declared in the Android manifest, this permission is asked by the camera library?

@OtterWays
Copy link
Collaborator

Yes, this permission is linked to the camera permission. In fact, this won't prevent the publication on the Play Store since the mechanism is automatic. However, it is possible to disable the permission request to avoid disturbing users. Note that the permission will still be visible in the app's information, but its default value will be "denied".

However, the permission is not requested on iOS.

@Jean-BaptisteC
Copy link
Contributor Author

Thanks for detail
Maybe we can hide permission in manifest with tools:node="remove" like this?

https://github.com/Exodus-Privacy/exodus-android-app/blob/c38f092316105ccc520a199bf8356d6ccebd5fbb/app/src/main/AndroidManifest.xml#L9

@OtterWays OtterWays linked a pull request Jun 23, 2024 that will close this issue
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

Successfully merging a pull request may close this issue.

2 participants