jquery / codeorigin.jquery.com

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

Switch Purge script to Highwinds API #44

Closed Krinkle closed 6 years ago

Krinkle commented 6 years ago

Ref jquery/infrastructure#438.

Krinkle commented 6 years ago

/cc @kborchers, @dmethvin

Krinkle commented 6 years ago

Reminder regarding the above. The jQuery Mobile release (https://github.com/jquery/codeorigin.jquery.com/commit/cfe94a7bde819b42f7dc3bf524f6e2cb686f51e9) is not showing up at https://codeorigin.jquery.com/mobile/ and https://codeorigin.jquery.com/ as result.

The pages did get updated automatically in WordPress, but the CDN purge is failing because it's connecting to the previous' CDN API still :)

I've already provisioned the new Nginx config and credentials file. To test it locally, create a local config.json file with the credentials from infrastructure.git (https://github.com/jquery/infrastructure/commit/dc203b3b3) and run one of the example commands, e.g. with REQUEST_URI ="/mobile/".

dmethvin commented 6 years ago

Pardon my short attention span ... exactly what needs to be done here @Krinkle and who can do it? I know I can't, I don't have any credentials for these.

Krinkle commented 6 years ago

@dmethvin The CDN for code.jquery.com changed sometime ago. Since then, changes to WordPress pages have stopped propagating to users due to the purge step not working anymore when jquery build server runs the script.

I got access to the Highwinds panel from @kborchers, where I generated an API token. I've committed those at https://github.com/jquery/infrastructure/commit/dc203b3b3.

What I need is for someone to review/merge the PHP code in this pull request, because I don't have write access here.

Krinkle commented 6 years ago

(Done)

dmethvin commented 6 years ago

@Krinkle LMK if this seems to be working. It seems to be working for me, I can tell by the Done! that it's the new code.

Krinkle commented 6 years ago

Yep, LGTM. Thanks!