-
Notifications
You must be signed in to change notification settings - Fork 477
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
How to enable checksum? #81
Comments
As you correctly noticed, tusd does not support this extension. The reason is that it requires the upload to be stored somewhere in between while you are calculating the checksum, resulting in a far worse efficiency, in particular for larger uploads. Furthermore, the use case for checksums are rather special and there we did not saw an urge for this. Would you mind talking about your reasons here? Anyways, implementing this is probably not the easiest but doable. You would have to store the uploading chunk in a temporary file while you calculate the checksum and only after verifying its integrity, you may pass the chunk to the actual data store. However, I am always open to provide help to you. |
I understand your point. The main reason is to validate if the uploaded file is the same that was sent. So if not we could re-upload the file/part. This is not a big deal, it's more a question. By the way, I'm implementing Tus client in Go. I couldn't find any. It's almost done. How could I add it to the docs? |
That's good to hear. But I will still sink some thoughts into how we could implement this.
That's awesome, thank you. Feel free to open an issue in the tus.io repository (https://github.com/tus/tus.io/issues) and we take care of adding it to the list. |
Thanks. |
Since the original question has been answered, I will close this issue, but feel free to continue commenting if you want to add something to the discussion. |
Hi,
Looking into the code and the go docs I couldn't figure out how to enable checksum extension.
The only place that mention this extension is the specification.
Is this supported by tusd? How can I enable ou start implementing this?
The text was updated successfully, but these errors were encountered: