Add duration to metadata

[narickmann]

This should fix #738

[LukasKalbertodt]

One code style and one semantic thingy.

[LukasKalbertodt]

This would set the start point to the upload time and the end point to that plus duration, which is not accurate. It would already be better if we would use now as end point and now - duration as start point. But there can still be an arbitrary amount of time between stopping the recording and actually uploading.

It would be better to store the time of when the recording was stopped. And then subtract the duration from that to get the start point.

Interesting bit: due to the fact that the recording can be paused, end - recording_duration is not necessarily the recording start point. So we have the choice between three options:

• Use the correct start/end points. Pausing means that end - start does NOT equal the video duration.
• Use correct end point and duration, with start point being wrong if the recording was paused.
• Use correct start point and duration, with end point being wrong if the recording was paused.

I personally don't care which of these options we choose. But maybe Opencast will break if we take the first option?

(Oh, I just saw Lars mentioned this solution in the original issue. I would probably still say we should use the correct recording stop time.)

[LukasKalbertodt]

Suggested change

	dispatch({ type: 'SET_DURATION', value: calcDuration(fullDuration, start, end) });
	dispatch({
	type: 'SET_DURATION',
	value: (end ?? fullDuration) - (start ?? 0),
	});

This should do the same, and this way we can avoid the extra function.

[github-actions]

This pull request has conflicts ☹
Please resolve those so we can review the pull request.
Thanks.

[LukasKalbertodt]

Replaced by the rebased version: #1077