Fix SIP connection test and improve logging for SIP¨Application class.

showhand · Jul 2, 2018 · 1c6611a · 1c6611a
1 parent 854816b
commit 1c6611a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/bbb-client-check/src/org/bigbluebutton/clientcheck/service/RTMPTunnelingService.as b/bbb-client-check/src/org/bigbluebutton/clientcheck/service/RTMPTunnelingService.as
@@ -43,6 +43,7 @@ package org.bigbluebutton.clientcheck.service
 		private static var RECORD_MOCK:Boolean=false;
 		private static var EXTERNAL_USER_ID_MOCK:String="123456";
 		private static var INTERNAL_USER_ID_MOCK:String="654321";
+		private static var CLIENT_CONN_ID:String="client-conn-id";
 		private static var LOCK_ON_MOCK:Boolean=true;
 
 		public function init():void
@@ -60,7 +61,7 @@ package org.bigbluebutton.clientcheck.service
 						// sip has a different way of connecting to the red5 server, need to fake connection data.
 						if (systemConfiguration.rtmpApps[i].applicationUri.indexOf("sip") > 0)
 						{
-							_netConnection.connect(systemConfiguration.rtmpApps[i].applicationUri, ROOM_MOCK, EXTERNAL_USER_ID_MOCK, USER_NAME_MOCK, INTERNAL_USER_ID_MOCK);
+							_netConnection.connect(systemConfiguration.rtmpApps[i].applicationUri, ROOM_MOCK, EXTERNAL_USER_ID_MOCK, USER_NAME_MOCK, INTERNAL_USER_ID_MOCK, CLIENT_CONN_ID);
 							continue;
 						}
 						else

diff --git a/bbb-voice/src/main/java/org/bigbluebutton/voiceconf/red5/Application.java b/bbb-voice/src/main/java/org/bigbluebutton/voiceconf/red5/Application.java
@@ -86,7 +86,7 @@ public void appStop(IScope scope) {
 	public boolean appConnect(IConnection conn, Object[] params) {
 
 		if(params.length != 5) {
-			log.error("Invalid number of parameters. param length=" + params.length);
+			log.error("Invalid number of parameters. Provided parameters={} .Required parameters=5", params.length);
 			return false;
 		}