search

tus / TUSKit

The tus client for iOS.
https://tus.io/
MIT License
214 stars 115 forks source link

Fixed updatedata cache cleanup #180

Closed MartinLau7 closed 11 months ago

MartinLau7 commented 11 months ago

@Acconut @donnywals On macOS, TUSKit did not properly clean up cache files when tasks ended and were removed. This PR fixes this issue.

I'm not sure if the same issue exists on iOS. Can someone help verify? thx

donnywals commented 11 months ago

Thanks for this @MartinLau7! Change looks good to me.

As far as I know this line should handle iOS

try FileManager.default.removeItem(at: filePath)

I will verify iOS works as intended before approving and merging this one.

donnywals commented 11 months ago

Turns out iOS had the same problem and this PR fixes it :) thanks!