Destination file path already exists

[mhudasch]

Kind of adjacent to #194 .
When starting a tool download with the function downloadToolWithRetries and a subsequent Abort from the server causes the a retry which is ok. The problem is that a partially downloaded file is placed in the _temp folder of the agent and the first retry of download fails immediately because a destination file with the name already exists. In the catch block any destination file must be cleaned up to retry the download successfully.

see:

Agent.Version=3.218.0
Agent.TempDirectory=C:\az-fake\a1\_temp
testing directory 'C:\az-fake\a1\_temp'
Downloading: https://nodejs.org/dist/v19.9.0/node-v19.9.0-win-x64.7z
destination C:\az-fake\a1\_temp\node-v19.9.0-win-x64.7z
downloading
Content-Length of downloaded file: 18547594
creating stream
Attempt 1 failed. Retrying after 500 ms
download complete
Downloaded file size: 15129050 bytes
Content-Length (18547594 bytes) did not match downloaded file size (15129050 bytes).
node-v19.9.0-win-x64.7z
Agent.Version=3.218.0
Agent.TempDirectory=C:\az-fake\a1\_temp
testing directory 'C:\az-fake\a1\_temp'
Downloading: https://nodejs.org/dist/v19.9.0/node-v19.9.0-win-x64.7z
destination C:\az-fake\a1\_temp\node-v19.9.0-win-x64.7z
Attempt 2 failed. Retrying after 1000 ms
node-v19.9.0-win-x64.7z
Agent.Version=3.218.0
Agent.TempDirectory=C:\az-fake\a1\_temp
testing directory 'C:\az-fake\a1\_temp'
Downloading: https://nodejs.org/dist/v19.9.0/node-v19.9.0-win-x64.7z
destination C:\az-fake\a1\_temp\node-v19.9.0-win-x64.7z
task result: Failed
Waiting for the debugger to disconnect...

Destination file path already exists