[IMPROVE] VoIP admin page cleanup: remove unused settings

apps/meteor/app/lib/server/startup/settings.ts

@@ -3210,7 +3210,6 @@ settingsRegistry.addGroup('Call_Center', function () {
 		this.add('VoIP_Enabled', false, {
 			type: 'boolean',
 			public: true,
-			alert: 'Experimental_Feature_Alert',
 			enableQuery: {
 				_id: 'Livechat_enabled',
 				value: true,
@@ -3225,22 +3224,6 @@ settingsRegistry.addGroup('Call_Center', function () {
 			},
 		});
 		this.section('Server_Configuration', function () {
-			this.add('VoIP_Server_Host', '', {
-				type: 'string',
-				public: true,
-				enableQuery: {
-					_id: 'VoIP_Enabled',
-					value: true,
-				},
-			});
-			this.add('VoIP_Server_Websocket_Port', 0, {
-				type: 'int',
-				public: true,
-				enableQuery: {
-					_id: 'VoIP_Enabled',
-					value: true,
-				},
-			});
 			this.add('VoIP_Server_Name', '', {
 				type: 'string',
 				public: true,

apps/meteor/client/providers/CallProvider/hooks/useVoipClient.ts

@@ -19,6 +19,8 @@ const empty = {};
 
 const isSignedResponse = (data: any): data is { result: string } => typeof data?.result === 'string';
 
+// Currently we only support the websocket connection and the SIP proxy connection being from the same host,
+// we need to add a new setting for SIP proxy if we want to support different hosts for them.
 export const useVoipClient = (): UseVoipClientResult => {
 	const [voipEnabled, setVoipEnabled] = useSafely(useState(useSetting('VoIP_Enabled')));
 	const voipRetryCount = useSetting('VoIP_Retry_Count');

apps/meteor/packages/rocketchat-i18n/i18n/en.i18n.json

@@ -4861,7 +4861,7 @@
   "VoIP_Server_Host": "Server Host",
   "VoIP_Server_Websocket_Port": "Websocket Port",
   "VoIP_Server_Name": "Server Name",
-  "VoIP_Server_Websocket_Path": "Websocket Path",
+  "VoIP_Server_Websocket_Path": "Websocket URL",
   "VoIP_Retry_Count": "Retry Count",
   "VoIP_Retry_Count_Description": "Defines the number of times the client will try to reconnect to the VoIP server if the connection is lost.",
   "VoIP_Management_Server": "VoIP Management Server",

apps/meteor/packages/rocketchat-i18n/i18n/pt-BR.i18n.json

@@ -4748,7 +4748,7 @@
   "VoIP_Server_Host": "Hoste do servidor",
   "VoIP_Server_Websocket_Port": "Porta do webSocket",
   "VoIP_Server_Name": "Nome do servidor",
-  "VoIP_Server_Websocket_Path": "Caminho do webSocket",
+  "VoIP_Server_Websocket_Path": "URL do webSocket",
   "VoIP_Management_Server": "Servidor de gerenciamento de VoIP",
   "VoIP_Management_Server_Host": "Host de servidor",
   "VoIP_Management_Server_Port": "Porta do servidor",

apps/meteor/server/services/voip/lib/Helper.ts

@@ -10,7 +10,6 @@ export function getServerConfigDataFromSettings(type: ServerType): IVoipCallServ
 				host: settings.get<string>('VoIP_Server_Host'),
 				name: settings.get<string>('VoIP_Server_Name'),
 				configData: {
-					websocketPort: Number(settings.get<number>('VoIP_Server_Websocket_Port')),
 					websocketPath: settings.get<string>('VoIP_Server_Websocket_Path'),
 				},
 			};

apps/meteor/server/startup/migrations/index.ts

@@ -95,4 +95,5 @@ import './v268';
 import './v269';
 import './v270';
 import './v271';
+import './v272';
 import './xrun';

apps/meteor/server/startup/migrations/v272.ts

@@ -0,0 +1,14 @@
+import { Settings } from '@rocket.chat/models';
+
+import { addMigration } from '../../lib/migrations';
+
+addMigration({
+	version: 272,
+	async up() {
+		await Settings.deleteMany({
+			_id: {
+				$in: ['VoIP_Server_Host', 'VoIP_Server_Websocket_Port'],
+			},
+		});
+	},
+});

packages/core-typings/src/IVoipExtension.ts

@@ -49,5 +49,4 @@ export const isIQueueMembershipDetails = (obj: any): obj is IQueueMembershipDeta
 export const isIExtensionDetails = (prop: any): prop is IExtensionDetails =>
 	prop.extension !== undefined && prop.password !== undefined && prop.authtype !== undefined && prop.state !== undefined;
 
-export const isIRegistrationInfo = (prop: any): prop is IRegistrationInfo =>
-	prop.hasOwnProperty('host') && prop.hasOwnProperty('callServerConfig') && prop.hasOwnProperty('extensionDetails');
+export const isIRegistrationInfo = (prop: any): prop is IRegistrationInfo => 'callServerConfig' in prop && 'extensionDetails' in prop;

packages/core-typings/src/IVoipServerConfig.ts

@@ -7,17 +7,18 @@ export enum ServerType {
 
 export interface IVoipServerConfigBase {
 	type: ServerType;
-	host: string;
 	name: string;
 }
 
 export interface IVoipCallServerConfig extends IVoipServerConfigBase {
+	host: string;
 	type: ServerType.CALL_SERVER;
 	configData: ICallServerConfigData;
 }
 
 export interface IVoipManagementServerConfig extends IVoipServerConfigBase {
 	type: ServerType.MANAGEMENT;
+	host: string;
 	configData: IManagementConfigData;
 }
 
@@ -27,5 +28,4 @@ export interface IManagementConfigData {
 	password: string;
 }
 
-export const isICallServerConfigData = (obj: any): obj is ICallServerConfigData =>
-	Number.isInteger(obj.websocketPort) && String(obj.websocketPath) === obj.websocketPath;
+export const isICallServerConfigData = (obj: any): obj is ICallServerConfigData => String(obj.websocketPath) === obj.websocketPath;

packages/core-typings/src/voip/IRegistrationInfo.ts

@@ -6,7 +6,6 @@
  */
 
 export interface ICallServerConfigData {
-	websocketPort: number;
 	websocketPath: string;
 }
 export interface IExtensionDetails {