Skip to content

Commit ac99132

Browse files
committed
Enable WebRTC in the proxy for all users
1 parent ba5ad09 commit ac99132

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

src/model/proxy-store.ts

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,8 @@ function startServer(
5858
// This generally means that some of our config is bad.
5959

6060
if (e.message?.includes('unrecognized plugin: webrtc')) {
61-
// We have webrtc enabled, but the server is old and doesn't support it.
62-
// Skip that entirely then:
61+
// We have webrtc enabled, and the server is new enough to recognize plugins and try to
62+
// start them, but too old to actually support the WebRTC plugin. Skip that entirely then:
6363
config = {
6464
...config,
6565
webrtc: undefined
@@ -184,8 +184,6 @@ export class ProxyStore {
184184
}
185185

186186
private startIntercepting = flow(function* (this: ProxyStore) {
187-
const webRTCEnabled = this.accountStore.featureFlags.includes('webrtc');
188-
189187
this.adminClient = new PluggableAdmin.AdminClient<{
190188
http: any,
191189
webrtc: any
@@ -207,9 +205,7 @@ export class ProxyStore {
207205
},
208206
port: this.portConfig
209207
},
210-
...(webRTCEnabled ? {
211-
webrtc: {}
212-
} : {})
208+
webrtc: {}
213209
});
214210

215211
this.mockttpRequestBuilder = new MockttpPluggableAdmin.MockttpAdminRequestBuilder(

0 commit comments

Comments
 (0)