Skip to content

fix(plugin): grant Linux camera and microphone only to the locked origin - #18

Open
zippy wants to merge 1 commit into
mainfrom
security/linux-media-origin
Open

zippy wants to merge 1 commit into
mainfrom
security/linux-media-origin

Conversation

@zippy

@zippy zippy commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

Summary

On Linux the plugin used to grant WebKitGTK camera and microphone requests on every webview. It now answers each request against the origin the webview first loaded, using the record from the navigation lock, and denies any page on another origin. Plugin-built windows cannot leave their origin anyway, so this mainly covers windows an app builds itself.

TODO:

  • CHANGELOG updated with appropriate info
  • npm run ci passes

@coderabbitai

coderabbitai Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 0e52dd98-ddb2-459e-a8ba-b7e4c55c584a

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@zippy
zippy force-pushed the security/navigation-lock branch from f492d78 to c66a1d2 Compare September 24, 2026 18:13
@zippy
zippy force-pushed the security/linux-media-origin branch from 19abe73 to 9984894 Compare September 24, 2026 18:13
@zippy
zippy marked this pull request as ready for review September 25, 2026 14:38
@zippy
zippy requested a review from mattyg September 25, 2026 14:39
mattyg
mattyg previously approved these changes Sep 28, 2026
@zippy
zippy force-pushed the security/navigation-lock branch from fd6d87d to c4ca0a1 Compare September 30, 2026 15:22
@zippy
zippy requested a review from a team as a code owner September 30, 2026 15:22
@zippy
zippy force-pushed the security/navigation-lock branch from c4ca0a1 to 162aea0 Compare September 30, 2026 15:35
Base automatically changed from security/navigation-lock to main September 30, 2026 16:57
@zippy
zippy dismissed mattyg’s stale review September 30, 2026 16:57

The base branch was changed.

Answer each WebKitGTK user-media permission request against the origin the
webview first loaded, and deny it for any other page, instead of granting media
capture to every webview.
@zippy
zippy force-pushed the security/linux-media-origin branch from 9984894 to bc005f8 Compare September 30, 2026 18:28
@cocogitto-bot

cocogitto-bot Bot commented Sep 30, 2026

Copy link
Copy Markdown

✔️ bc005f8 - Conventional commits check succeeded.

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 this pull request may close these issues.

2 participants