Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
People have been reporting a lot of "wrong md5 checksum" errors, for example:
- #786454: wrong md5 checksum in 6.x-4.0-beta3.tar.gz
- #1118122: File logintoboggan-6.x-1.9.tar.gz is corrupt (wrong md5 checksum)
- #1469784: drush up drupal fails with md5 error
This issue comes from a very simple inconsistency:
- The XML release file (that contains the md5 checksums) is cached by default for 24 hours,
- The package file is cached by default only for 4 hours (for dev release)
That leaves a potential window of inconsistency of 20 hours during which the package file is tested against the wrong XML release file.
Solution: cache the XML release file and the package file the same amount of time.
Comments
Comment #1
moshe weitzman CreditAttribution: moshe weitzman commentedMoving to project queue, packages component.
Comment #2
moshe weitzman CreditAttribution: moshe weitzman commentedoh wait - you mean drush caching. ok, assigning back. thanks for the research.
Comment #3
moshe weitzman CreditAttribution: moshe weitzman commentedJust committed a fix and big improvement. We now include the date of the dev snapshot in the cache key so we now safely cache dev snapshots for a year
Thanks again for the research, Damien.
Comment #4
dwwThanks for the quick fix. For now, I've just hard-coded --no-cache when the d.o packaging system invokes drush, but we can experiment with ripping that back out once there's an official drush release that includes this fix (presumably 5.5).
Cheers,
-Derek
Comment #5
Damien Tournoud CreditAttribution: Damien Tournoud commentedThat would work, thanks!
I think you should also consider reducing the general cache duration. Caching anything 24h or more seems highly excessive to me.
Comment #7
moonray CreditAttribution: moonray commentedIt's back.
Comment #8
moshe weitzman CreditAttribution: moshe weitzman commentedJust worked for me.
Comment #9
greg.1.anderson CreditAttribution: greg.1.anderson commentedThis issue was marked
closed (won't fix)
because Drush has moved to Github.If desired, you may copy this bug to our Github project and then post a link here to the new issue. Please also change the status of this issue to
closed (duplicate)
.Please ask support questions on Drupal Answers.