search

jquery / codeorigin.jquery.com

jQuery CDN
https://releases.jquery.com
Other
57 stars 602 forks source link

purge: Fix CDN purge script (enable `purgeAllDynamic` option) #48

Closed Krinkle closed 5 years ago

Krinkle commented 5 years ago

While not yet documented or announced, it seems the Purge API from StackPath/Highwinds/StrikeTracker no longer purges variants by default as of October this year.

The old behaviour can be regained through the undocumented purgeAllDynamic boolean option which appears to be false by default.

I'm not sure what variants we have that cause this behaviour, but it might be due to something like Vary:Accepted-Encoding (gzip etc.).

Fixes #45.

Krinkle commented 5 years ago

I've confirmed locally that REQUEST_URI="/mobile/" php purge.php results in https://code.jquery.com/mobile/ being purged and now includes jQuery Mobile 1.5.0-rc1.

Previously, the CDN url still showing jQuery Mobile 1.5.0-alpha.1.

arschmitz commented 5 years ago

@Krinkle Thank you so much there have been tons of people complaining about it not existing

dmethvin commented 5 years ago

@Krinkle I added you to the maintainers team so you can land this whenever you feel good about it. Thanks!

Krinkle commented 5 years ago

Thanks @dmethvin. I'll try it out now and will revert if not working as expected.