Skip to content

Conversation

@BrayanDSO
Copy link
Member

Fixes

Approach

In the commits

How Has This Been Tested?

Emulator 31, with the deck provided at #16319 (comment)

Screen_recording_20260115_101314.mp4

Learning (optional, can help others)

https://developer.android.com/reference/android/Manifest.permission#MODIFY_AUDIO_SETTINGS

Checklist

Please, go through these checks before submitting the PR.

  • You have a descriptive commit message with a short title (first line, max 50 chars).
  • You have commented your code, particularly in hard-to-understand areas
  • You have performed a self-review of your own code
  • UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
  • UI Changes: You have tested your change using the Google Accessibility Scanner

@BrayanDSO BrayanDSO changed the title Feat/webview record fix(study-screen): NotReadableError on JS getUserMedia while trying to record audio in the WebView Jan 15, 2026
@david-allison
Copy link
Member

I have a security concern here: I believe this lets a deck author record an end-user's audio without requesting permission from the user

The implementation is fine in the happy path, but at LEAST can this require user consent via a scary-ish setting.

@david-allison david-allison added the Needs Author Reply Waiting for a reply from the original author label Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Author Reply Waiting for a reply from the original author Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NotReadableError on getUserMedia while trying to record audio in card's webview with javascript

2 participants