Skip to content
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

File hash check for local fs and https #76

Open
madiganz opened this issue Jun 8, 2022 · 3 comments
Open

File hash check for local fs and https #76

madiganz opened this issue Jun 8, 2022 · 3 comments
Labels
question What is going on??? :thinking emoji:

Comments

@madiganz
Copy link
Contributor

madiganz commented Jun 8, 2022

Not sure if this is the place for asking questions, but I see that for network robustness, when transferring a file, the hash is checked to make sure the destination matches the source for GCS and S3. Is there a reason why this check isn't also done for local file system and https transfers?

@william-silversmith
Copy link
Contributor

This is certainly a place for asking questions! The only reason these checks aren't done is:

a) For local files, there's no hash metadata to compare to. We could add a metadata file, but it would be an extra cost. This library was designed in a context where hundreds of millions of files could be generated and that was already wrecking filesystems, so adding another file would have doubled the load. We could add information somehow as an option. Did I understand your question correctly?

b) For HTTPS transfers, the main reason is that I haven't really looked into it. Is there a standard most web servers follow?

@william-silversmith william-silversmith added the question What is going on??? :thinking emoji: label Jun 9, 2022
@madiganz
Copy link
Contributor Author

madiganz commented Jun 9, 2022

Yes that answers my question and makes sense for local file systems. I am not sure if there is a standard that most web servers follow, so I think sounds like not checking for now is the way to go

@william-silversmith
Copy link
Contributor

Yea, I'm also happy to build in some special handling for important systems, but I think they need to be identified to me first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question What is going on??? :thinking emoji:
Projects
None yet
Development

No branches or pull requests

2 participants