-
Notifications
You must be signed in to change notification settings - Fork 27.9k
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
Add audio entitlements supporting live-share #103509
Comments
@mengesb: thanks for your help on this! I'm actively looking at it from the Live Share side but am having a bit of trouble with MacOS notarization at the moment. The Live Share Audio extension currently uses its own copy of Electron to provide audio support. On first activation of the extension there is a dependency downloader that retrieves & unzips Electron to Live Share's global storage location. When someone attempts to connect to a call the Live Share Audio extension spawns a child process that starts the calling app using our copy of Electron. What I'm attempting to do at the moment is to add the necessary entitlements to Live Share's copy of Electron. I've pasted the error I'm seeing below and I've highlighted the binary path that points to Live Share's assets. What is unclear to me is whether adding the entitlements to this will be sufficient or is it VS Code's copy of Electron (responsible path below), that spawns our child process, that will require the audio entitlements? Prompting policy for hardened runtime; service: kTCCServiceMicrophone requires entitlement com.apple.security.device.audio-input but it is missing for RESP:{ID: com.microsoft.VSCode, PID[23936], auid: 501, euid: 501, responsible path: '/Applications/Visual Studio Code.app/Contents/MacOS/Electron', binary path: '/Applications/Visual Studio Code.app/Contents/MacOS/Electron'}, ACC:{ID: com.github.Electron.helper, PID[24070], auid: 501, euid: 501, binary path: '/Users/XXX/Library/Application Support/Code/User/globalStorage/ms-vsliveshare.vsliveshare-audio/electron-4.2.7/dist/Electron.app/Contents/Frameworks/Electron Helper.app/Contents/MacOS/Electron Helper'}, REQ:{ID: com.apple.tccd, PID[22383], auid: 0, euid: 0, binary path: '/System/Library/PrivateFrameworks/TCC.framework/Versions/A/Resources/tccd'} |
@mengesb , @deepak1556 : I was able to add entitlements, sign & notarize Live Share's copy of Electron as well as the calling libraries. Unfortunately I'm still getting the same error above. I think these entitlements will need to added to VS Code for audio to work as expected. |
Potentially both |
It might be that it requires both I'm hopeful more progress will be made! Please remain in pursuit of a resolution |
Any additional progress? |
Is there a place this can be tracked so we know when it's included in a release? There are lots of related issues in the docs repo about this problem that can be closed out with it |
Same issue here! Quite annoying, pls fix this asap! |
The LiveShare audio issue is tracked here: |
@jramsay I'm closing this issue ; can we get an update on one of the consolidated issues aruond MacOS and Audio and the newer MacOS permissions structure with recent OS updates? |
#95832 was previously rejected, however it seems required now that live-share audio requires these entitlements in order to allow live-share audio the ability to request these permissions. Not sure if there's a way that the extension can add these itself, but nonetheless vscode now needs the audio entitlement at minimum. There are several issues in the MicrosoftDocs/live-share issues related to Audio not working on newer MacOS versions (Mojave / Catalina); High Sierra seems to be OK. This issue shows the tccd log around audio events
If there are plans to include or add a feature to support video via live-share sessions... then the camera permission would be required - perhaps this can be a different feature request
Steps to Reproduce:
Does this issue occur when all extensions are disabled?: Yes
The text was updated successfully, but these errors were encountered: