Problem/Motivation

Platform CDN purger can't invalidate URLs due to a typing error in the construction of the PURGE request options

Steps to reproduce

Run

drush9 pinv http://www.example.com/

Observe the purge fails and Platform CDN purger no longer works, complaining that credentials aren't set (but they are set).

Proposed resolution

Remove line 121, unset($request_opt['headers']['Fastly-Key']);, of src/AcquiaPlatformCdn/FastlyBackend.php because it is obviously a typing error.

Remaining tasks

None

CommentFileSizeAuthor
#3 fix_drush9_pinv_url.patch595 bytesmmcconnell

Comments

mmcconnell created an issue. See original summary.

mmcconnell’s picture

mmcconnell commented
Issue summary: View changes
mmcconnell’s picture

mmcconnell commented
StatusFileSize
new fix_drush9_pinv_url.patch595 bytes
japerry’s picture

japerry
Location KOMK
commented
Status: Active » Needs review

@nielsvm Is there a reason why this was added? Its not clear in the git log, but I'd think other customers would have this same issue and report it.

rubenvarela’s picture

rubenvarela commented

Got hit by this one. Patch works.

charginghawk’s picture

charginghawk commented
Priority: Normal » Critical

Changing this to critical since it completely breaks Fastly support and is such an easy fix.

  • japerry committed 264c338 on 8.x-1.x 
    Issue #3230755 by mmcconnell: Platform CDN purger can't invalidate URLs
japerry’s picture

japerry
Location KOMK
commented
Status: Needs review » Fixed

Fixed.

I don't see anything in the git log indicating -why- this was added, it was part of the original commit when Fastly support was added. If others report issues after this patch is added, we can add configuration to disable it on a case-by-case basis.

Status: Fixed » Closed (fixed)

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