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

SEBSP-107: Removed logging of browser URLs in metadata aggregator.

Browse source
This commit is contained in:
Damian Büchel 2024-02-22 17:29:17 +01:00
parent 4aacf85e9a
commit 91f2c14a77
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
SafeExamBrowser.Proctoring/ScreenProctoring/Data/MetaDataAggregator.cs
View file

@ -24,9 +24,11 @@ namespace SafeExamBrowser.Proctoring.ScreenProctoring.Data
private string applicationInfo;
private string browserInfo;
private string browserInfoWithoutUrls;
private TimeSpan elapsed;
private string triggerInfo;
private string urls;
private int urlCount;
private string windowTitle;
internal MetaData Data => new MetaData
@ -65,8 +67,7 @@ namespace SafeExamBrowser.Proctoring.ScreenProctoring.Data
CaptureMouseTrigger(mouse);
}
// TODO: Can only log URLs when allowed by policy in browser configuration!
logger.Debug($"Captured metadata: {applicationInfo} / {browserInfo} / {triggerInfo} / {urls} / {windowTitle}.");
logger.Debug($"Captured metadata: {applicationInfo} / {browserInfoWithoutUrls} / {triggerInfo} / {urlCount} URL(s) / {windowTitle}.");
}
private void CaptureApplicationData()
@ -88,7 +89,9 @@ namespace SafeExamBrowser.Proctoring.ScreenProctoring.Data
var windows = browser.GetWindows();
browserInfo = string.Join(", ", windows.Select(w => $"{(w.IsMainWindow ? "Main" : "Additional")} Window: {w.Title} ({w.Url})"));
browserInfoWithoutUrls = string.Join(", ", windows.Select(w => $"{(w.IsMainWindow ? "Main" : "Additional")} Window: {w.Title}"));
urls = string.Join(", ", windows.Select(w => w.Url));
urlCount = windows.Count();
}
private void CaptureIntervalTrigger(IntervalTrigger interval)