Problem/Motivation

Per #3061248-12: Package quasi-patch for automatic updates, we need to handle lazy generation of the patches.

Requesting arbitrary update combinations like https://www.drupal.org/in-place-updates/drupal/drupal-8.7.4-to-8.7.7.zip will now be recorded, and respond with an HTTP 429 and a Retry-After: 10 header.

Proposed resolution

At least in D8, guzzle has a means of handling these and possibly retrying the connection. Investigate that.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3087196.patch2.17 KBheddn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heddn created an issue. See original summary.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC commented
Status: Active » Needs review
Issue tags: +Needs manual testing
FileSize
3087196.patch2.17 KB

This should handle this edge case.

  • heddn committed 7df0066 on 8.x-1.x 
    Issue #3087196 by heddn: Handle HTTP Status 429 from quasi patch files
heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented

Could still use more testing, but let's do that with it landed.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.