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

inexplicable microphone capture warning on backgrounding app, when it really shouldn't be going anywhere near the microphone #2336

Closed
ara4n opened this issue Mar 14, 2019 · 8 comments

Comments

@ara4n
Copy link
Member

ara4n commented Mar 14, 2019

on iPhone X and iOS 12.1.4 and latest TF. vid at https://matrix.to/#/!tAMgPDeOawUJMkFXVZ%3Amatrix.org/%241552601254152567Qiaxn%3Amatrix.org

@SBiOSoftWhare
Copy link
Contributor

SBiOSoftWhare commented Mar 20, 2019

This issue appears since 0.8.2 TestFlight release, first build with Xcode 10 and iOS 12 as base SDK.
It seems interesting to check after updating Jitsi and WebRTC frameworks (#1483) if the problem still occurs.

Currently we did not find a way to reproduce it systematically.
But we found steps to sometimes reproduce this issue:

  • Make a 1:1 video call with someone who will not answer.
  • Wait for the call to end itself.
  • Background the app.

Then you could see the red record status bar appearing.

@ara4n
Copy link
Member Author

ara4n commented Mar 25, 2019

to be clear: this happens even if you haven't been doing VoIP in the app.

@oliviabarrick
Copy link

Also seeing this. Resolved by revoking microphone permissions but still spooky.

@spantaleev
Copy link
Contributor

I can reproduce something similar by:

  • going to any room
  • triggering the "Send photo or video" button
  • closing the "take or choose a photo" window
  • (not returning to the home screen, but staying on the room screen)
  • pressing the Home button

Doing this, a red bar will appear for a second or so.

Every time you background using the Home button, you would see the red bar.
.. until you move away from the room screen (pressing back to return to the riot-ios home screen).

@SBiOSoftWhare
Copy link
Contributor

Thanks for the report @spantaleev. Above case has been identified and fixed by #2382.

For information camera picker asking to access camera and microphone when it appears. But due to a memory issue the picker was still alive when closing it and continued to ask access to the microphone.
It's no longer the case with this fix. The camera picker is now removed from memory and access to the microphone and camera are stopped when you close it by pressing the close button or selecting a media.

@ara4n
Copy link
Member Author

ara4n commented May 21, 2019

@SBiOSoftWhare it’s still not fixed on latest testflight. if you background the app whilst the photo picker is in the foreground, you see the red mic icon... and then everytime thereafter you bg the app you see the red icon too.

can we just kill the custom photo picker?

@aaronraimist
Copy link
Contributor

matrix-org/matrix-ios-kit#557 and #2457 is my WIP to do that. It currently works with images from the photo library but doesn't work for images from the camera yet.

The code was mostly already there, I guess it was used on iOS 8 and earlier. Feel free to look at that if useful.

@manuroe
Copy link
Member

manuroe commented Aug 8, 2019

Fixed by using native camera view (#2620)

@manuroe manuroe closed this as completed Aug 8, 2019
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

6 participants