mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 2 Wiki Activity Actions

calling: update getIceServers to use new turn routing endpoint

Browse source 
Co-authored-by: Adel Lahlou <adel@signal.com>
This commit is contained in:
adel-signal 2024-03-29 10:17:56 -07:00 committed by GitHub
parent 32223acf5b
commit 94c7de6c65
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
ts/services/calling.ts
View file

@ -2613,12 +2613,20 @@ export class CallingClass {
const isContactUntrusted = !isInSystemContacts(conversation.attributes);
const callSettings = {
// only include hostname with urlsWithIps
// proritize ice servers with IPs to avoid DNS
iceServers: [
{
hostname: iceServer.hostname,
username: iceServer.username,
password: iceServer.password,
urls: (iceServer.urlsWithIps ?? []).slice(),
},
{
hostname: '',
username: iceServer.username,
password: iceServer.password,
urls: iceServer.urls.slice(),
urls: (iceServer.urls ?? []).slice(),
},
],
hideIp: shouldRelayCalls || isContactUntrusted,

4
ts/textsecure/WebAPI.ts
View file

@ -503,7 +503,7 @@ const URL_CALLS = {
discovery: 'v1/discovery',
getGroupAvatarUpload: 'v1/groups/avatar/form',
getGroupCredentials: 'v1/certificate/auth/group',
getIceServers: 'v1/accounts/turn',
getIceServers: 'v1/calling/relays',
getOnboardingStoryManifest:
'dynamic/desktop/stories/onboarding/manifest.json',
getStickerPackUpload: 'v1/sticker/pack/form',
@ -735,6 +735,8 @@ export type GetIceServersResultType = Readonly<{
username: string;
password: string;
urls: ReadonlyArray<string>;
urlsWithIps: ReadonlyArray<string>;
hostname: string;
}>;
export type GetDevicesResultType = ReadonlyArray<