spring / upq

extracts metadata of game and map files of the spring engine
https://springfiles.springrts.com/
GNU General Public License v3.0
2 stars 5 forks source link

sdz link not updated when game is available as rapid #61

Closed abma closed 3 years ago

abma commented 3 years ago

https://springfiles.com/spring/games/metal-factions-39

->

https://api.springfiles.com/?springname=Metal+Factions+v1.44

silentwings commented 3 years ago

Allowing users to upload their own sdz with the same title as a rapid archive has the potential to break the sync/hash checks, in real cases and when the archives were zipped differently.

Imo better that rapid autouploads its own sdz to SF for major game releases (only) and users are disallowed to upload sdz for stuff on rapid.

abma commented 3 years ago

the .sdz should be only mapped when the spring hash is the same...

updating a file by filename or "spring name" (=name + version) is not possible and shouldn't be possible, if fully agree.

currently only missing is, to update the filename / download mirror here:

https://github.com/spring/upq/blob/master/jobs/extract_metadata.py#L143

abma commented 3 years ago

auto uploading .sdz requires to many disk space, thats why it isn't made.

springraaar commented 3 years ago

symlinking /tmp/springfiles-upload/metalfactions-v1.50.sdz /home/upq/upq/tmp/tmp739ab8fw/games/metalfactions-v1.50.sdz using unitsync version 100 SDP 9055e2dd3084b874e9ea6add297a8c19 Extracting data from metalfactions-v1.50.sdz Reading bitmaps/loadpictures/loadscreen1.jpg Wrote /home/upq/upq/metadata/d8e521b2a24ef384255734e73d0dfe14.jpg {'Type': 'game', 'Description': 'Metal Factions', 'Units': [{'UnitName': 'aven_ace', 'FullUnitName': 'Ace'}, {'UnitName': (...)['d8e521b2a24ef384255734e73d0dfe14.jpg']} Updated game 'Metal Factions' version 'v1.50' sdp '9055e2dd3084b874e9ea6add297a8c19' in the mirror-system File doesn't exist: /tmp/springfiles-upload/metalfactions-v1.50.sdz b'GetPrimaryModDescription: deprecated unitsync function called: GetPrimaryModDescription'

even if you don't want people to upload every sdz directly, the metadata (namely the date) should be fixed and the link to the available sdzs on the repos.springrts.com should be presented

abma commented 3 years ago

should be fixed by https://github.com/spring/upq/commit/7289c245ec78e5e736dbb27eb22cf7e8aec06ed9

abma commented 3 years ago

(the files in "builds" are still ignored, but thats part of maybe #72)