godot-asset-library icon indicating copy to clipboard operation
godot-asset-library copied to clipboard

Automatic hash generation

Open bojidar-bg opened this issue 9 years ago • 8 comments

Currently, as done in 25809e0109222b821617451a66d008c2d5948e17, the asset's hash would be set manually by the moderator on asset accept, which isn't too good in the long run. We should either proxy asset zips and construct the hash while resending it, or downloading the zip and having mods ensure that they've downloaded the same zip.

bojidar-bg avatar Jun 21 '16 11:06 bojidar-bg

Why not use just simple hash (even md5 if you don't like long) for asset URL and internaly check everything by demanding developers to sign their packages with GPG?

Marqin avatar Jul 27 '16 09:07 Marqin

Most Godot users that would like to submit assets likely don't know how to use git, so when it comes to asking them to sign their packages with GPG...

Hashing the URL is not enough as the asset can change while the URL would stay the same (e.g. if users give the URL of their GitHub repo's master zip archive). So having the sha256sum of the archive itself is necessary not only for security, but also to keep track of changing versions (e.g. if the new state of the master branch of an approved asset breaks compatibility with Godot 2.x, we'd want to make sure this is reflected in the asset's properties).

akien-mga avatar Jul 27 '16 09:07 akien-mga

I never said to hash the URL.

Marqin avatar Jul 27 '16 09:07 Marqin

Why not use just simple hash (even md5 if you don't like long) for asset URL

Maybe I misunderstood.

akien-mga avatar Jul 27 '16 09:07 akien-mga

for

is not

of

...

for

Means that asset download URL is like http://example.com/assetstore/asset/d41d8cd98f00b204e9800998ecf8427e.zip

Marqin avatar Jul 27 '16 09:07 Marqin

Sorry, I'm obviously not a native English speaker.

akien-mga avatar Jul 27 '16 09:07 akien-mga

@Marqin You could have just said

simple hash (even md5 if you don't like long) [of the] asset

..And it would have been all good. But anyway, lets go on with the discusson.

bojidar-bg avatar Jul 27 '16 10:07 bojidar-bg

Not blocking for beta, though would be really nice to have ;)

akien-mga avatar Sep 14 '16 08:09 akien-mga

  • Superseded by https://github.com/godotengine/godot-asset-library/pull/209.

Calinou avatar Feb 21 '23 15:02 Calinou