catchpoint / WebPageTest.bulk-tester

Google App Script for spreadsheet that uses the WPT API to bulk test URLs
MIT License
259 stars 22 forks source link

Script fails when rewrite isn't enabled on server #12

Open andydavies opened 8 years ago

andydavies commented 8 years ago

webpagetest.org rewrites http://www.webpagetest.org/results.php?test=160321_KK_4eb6b407b53c44fa142d3188ca433841 to http://www.webpagetest.org/result/160321_KK_4eb6b407b53c44fa142d3188ca433841

If the rewrites aren't enabled on a private instance the script attempts to add a second query string.

https://github.com/andydavies/WPT-Bulk-Tester/blob/master/webpagetest.gs#L174 needs to check from presence of query string and f=json as "&f=json" instead