Pass hash and extension in the DistributorCopyFile payload instead of the URL

[daniil-berg]

It seems it would be more consistent with the rest of the Distributor API, if the hash and extension of a file to be copied were passed in the already existing DistributorCopyFile payload model:

videbo/src/videbo/distributor/api/models.py

Lines 35 to 37 in 2fc6190

	class DistributorCopyFile(BaseRequestModel):
	from_base_url: str
	file_size: int # in bytes

They are currently part of the URL path in the form /api/distributor/copy/<hash><ext>.

Aside from a more consistent interface, it would have the added benefit of making the payload model map nicely to the CopyingVideoFile constructor, which would further simplify both the logic and the call site of DistributorFileController.schedule_copying.

videbo/src/videbo/distributor/api/routes.py

Lines 109 to 117 in 2fc6190

	hash_, ext = request.match_info["hash"], request.match_info["file_ext"]
	file_controller = DistributorFileController()
	try:
	file_controller.schedule_copying(
	hash_,
	ext,
	data.from_base_url,
	data.file_size,
	)

Since this would be a purely internal API change, there would be no implications for external clients/LMS.

We would only have to adjust the implementation of the DistributorClient.copy method, which depends on the distributor API.