zervo/seb-win-refactoring
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

SEBWIN-449: Ensured server URL is also sanitized when not using SEB server and removed check for authentication token for Jitsi Meet.

Browse source
This commit is contained in:
Damian Büchel 2021-05-19 02:43:07 +02:00
parent fd2eab589c
commit b456f0821f
1 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
SafeExamBrowser.Proctoring/ProctoringController.cs
View file

@ -89,9 +89,10 @@ namespace SafeExamBrowser.Proctoring
if (settings.JitsiMeet.Enabled)
{
this.settings.JitsiMeet.ServerUrl = Sanitize(settings.JitsiMeet.ServerUrl);
start = !string.IsNullOrWhiteSpace(settings.JitsiMeet.RoomName);
start &= !string.IsNullOrWhiteSpace(settings.JitsiMeet.ServerUrl);
start &= !string.IsNullOrWhiteSpace(settings.JitsiMeet.Token);
}
else if (settings.Zoom.Enabled)
{
@ -117,7 +118,7 @@ namespace SafeExamBrowser.Proctoring
logger.Info("Proctoring instruction received.");
settings.JitsiMeet.RoomName = roomName;
settings.JitsiMeet.ServerUrl = serverUrl.Replace($"{Uri.UriSchemeHttp}{Uri.SchemeDelimiter}", "").Replace($"{Uri.UriSchemeHttps}{Uri.SchemeDelimiter}", "");
settings.JitsiMeet.ServerUrl = Sanitize(serverUrl);
settings.JitsiMeet.Token = token;
StopProctoring();
@ -242,5 +243,10 @@ namespace SafeExamBrowser.Proctoring
return html;
}
}
private string Sanitize(string serverUrl)
{
return serverUrl?.Replace($"{Uri.UriSchemeHttp}{Uri.SchemeDelimiter}", "").Replace($"{Uri.UriSchemeHttps}{Uri.SchemeDelimiter}", "");
}
}
}