-
Notifications
You must be signed in to change notification settings - Fork 58
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
DC messages #870
DC messages #870
Conversation
// Waiting a second before transmitting to avoid | ||
// the known race between incoming and outgoing track. | ||
// This is needed as pion doesn't support perfect negotiation yet. | ||
time.Sleep(time.Second) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using DC further exacerbated the impact of signaling conflicts, so I had to add this to make it usable. Browsers are not affected as they support perfect negotiation (SDP rollback).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gotcha. 👍
// Waiting a second before transmitting to avoid | ||
// the known race between incoming and outgoing track. | ||
// This is needed as pion doesn't support perfect negotiation yet. | ||
time.Sleep(time.Second) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gotcha. 👍
@@ -5,8 +5,7 @@ import type {EmojiData, CallsClientJoinData} from '@mattermost/calls-common/lib/ | |||
|
|||
import {EventEmitter} from 'events'; | |||
|
|||
// @ts-ignore | |||
import {deflate} from 'pako/lib/deflate'; | |||
import {zlibSync, strToU8} from 'fflate'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My own interest, why this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cpoile It's ~40% faster, proper typescript and 5 times smaller bundle size. It got me convinced.
* lt: add incoming track handler * Enable RTC monitor in lt client * Update dependencies * Update server mocks * Bump dependencies
* Allow signaling through data channel * Update i18n * Update calls-common * DC messages (#870) * Replace pako with faster fflate * Mitigate signaling race * Update dependencies * Fix jest * [MM-60561] RTC client metrics (#874) * lt: add incoming track handler * Enable RTC monitor in lt client * Update dependencies * Update server mocks * Bump dependencies * Update snapshots
Summary
Not much of a change on this side. I removed the
pako
dependency since we now usefflate
. Also reworked logging and updated dependencies.Related PRs
mattermost/rtcd#158
mattermost/calls-common#40