IceDBorn / pipewire-screenaudio

Extension to passthrough pipewire audio to WebRTC Screenshare
https://addons.mozilla.org/firefox/addon/pipewire-screenaudio/
GNU General Public License v3.0
167 stars 5 forks source link

Share All Desktop Audio crashes when starting it without nodes #66

Closed alansartorio closed 1 year ago

alansartorio commented 1 year ago

The problem comes from using contains($streamIds) because if there's no initial nodes, it ends up being contains(), and that causes an error. The line that causes it is: https://github.com/IceDBorn/pipewire-screenaudio/blob/8d84a6ec28cf0fbc4b58befa53453c0b6cee41a9/native/connector/connect-and-monitor.sh#L39

The error produced by jq is:

jq: error: syntax error, unexpected ')' (Unix shell quoting issues?) at <top-level>, line 1:
[ .[] | select(.type == "PipeWire:Interface:Port") | select(.info.direction == "output") | select(.info.props["node.id"] | contains()) ]