{"push_id":949295196,"size":7,"distinct_size":7,"ref":"refs/heads/33","head":"6bdb9af633c9848d8d49114811bd20fefe98d10f","before":"d303a941b0c20f39f0e9d9647e005c1fded35d97","commits":[{"sha":"0cc9ff37a9dec6fd3a7a67d5978ad6b1c80a16ec","author":{"email":"c3f6b7d9de5243f9f0884ee043fd42ff9df5f177@gmail.com","name":"Joshua Marantz"},"message":"Fix flaw in HTTPCache::Find where the fallback value does not get\ndecompressed even when the cached response is compressed and the request\ndoes not have accept-encoding.  This was found by observing flakiness in\nvalgrind-system tests.\n\nFix flaw in the version of HTTPCache::Put that takes ResponseHeaders*\nand mutates it unexpectedly, by adding compression headers.\n\nFix a flaw in InflatingFetch that Reset didn't reset all the cached\nboolean bits.  I don't think this was the cause of anything in\nproduction becasue we only use Reset in tests (where we re-use a Fetch\nobject).","distinct":true,"url":"https://api.github.com/repos/pagespeed/mod_pagespeed/commits/0cc9ff37a9dec6fd3a7a67d5978ad6b1c80a16ec"},{"sha":"658ff6970c5e5489aa12caec5775b718a69927c5","author":{"email":"c3f6b7d9de5243f9f0884ee043fd42ff9df5f177@gmail.com","name":"Joshua Marantz"},"message":"Clear the destination HTTPValue before trying to write it.","distinct":true,"url":"https://api.github.com/repos/pagespeed/mod_pagespeed/commits/658ff6970c5e5489aa12caec5775b718a69927c5"},{"sha":"941ee0c8794a7d67c25271707844cafb63673278","author":{"email":"c3f6b7d9de5243f9f0884ee043fd42ff9df5f177@gmail.com","name":"Joshua Marantz"},"message":"Only rewrite an image once even if it's requested via ipro many times\nconcurrently.\n\nPrint elapsed time properly for subshell tests. Fix typo in SVG\nreference which was causing spurious timeout warnings in nginx tests,\nand then flagged by nginx system tests.\n\nRemove nginx test flakiness by using fetch_until an image gets small\nto indicate an ipro-rewrite is done, rather than testing for\nimage_ongoing_rewrites to be zero. The old technique might fail because\nwe might check that stat before the image starts being rewritten.\n\nNote: with this change, nginx system tests no longer flake for me,\nwhereas previously about 10% of the time it would flake on \"IPRO flow\nuses cache as expected\". That was also suppressed for valgrind runs,\nwhich is no longer needed.\n\nEven after this change, nginx system tests with valgrind tests still flake\nwith \"Fetch timed out\" log messages, which I am adding to the\nsuppressions, and with \"Embed image configuration in rewritten image\nURL.\", where the recursive-wget result is not optimized. I think that\nmight be a real user-facing bug, and I will report it to the nginx list.","distinct":true,"url":"https://api.github.com/repos/pagespeed/mod_pagespeed/commits/941ee0c8794a7d67c25271707844cafb63673278"},{"sha":"6a3a3879ee3e7bc7b5e1aef8ade91b240035ab05","author":{"email":"c3f6b7d9de5243f9f0884ee043fd42ff9df5f177@gmail.com","name":"Joshua Marantz"},"message":"Resolve flaky system test for counting image-rewrites on bursts of\nrequests by allowing 1 or 2 rewrites, instead of requiring 1.  Put in\ncomment explaining that this is due to 'best effort' locks.","distinct":true,"url":"https://api.github.com/repos/pagespeed/mod_pagespeed/commits/6a3a3879ee3e7bc7b5e1aef8ade91b240035ab05"},{"sha":"123e39ccf8e7080f35d50478c92d204900e55fdd","author":{"email":"65a2f0125c2b970856e8414d74fc56ab0a17c59c@google.com","name":"Jeff Kaufman"},"message":"Move InstawebHandler initialization after validity checks.\n\nCreating an instaweb handler will run MakeRequestUrl, which assumes that\nrequest->unparsed_uri is non-null.  So move the creation to after where\nwe check that it's non-null.  To be safe, move it all the way down to\nwhere it's first needed, in case some other validity checks end up being\nrelevant.\n\nFixes #1248","distinct":true,"url":"https://api.github.com/repos/pagespeed/mod_pagespeed/commits/123e39ccf8e7080f35d50478c92d204900e55fdd"},{"sha":"df749e835dd9ea3f1c5aad838aeef639635fade9","author":{"email":"e71773a9724be7c114f3c9592720fd960a75d365@google.com","name":"Maks Orlovich"},"message":"Make sure mod_access_compat is loaded for Apache 2.4\nso that \"allow from\" style of ACLs in our config works.\nRef: https://github.com/pagespeed/mod_pagespeed/issues/1229","distinct":true,"url":"https://api.github.com/repos/pagespeed/mod_pagespeed/commits/df749e835dd9ea3f1c5aad838aeef639635fade9"},{"sha":"6bdb9af633c9848d8d49114811bd20fefe98d10f","author":{"email":"65a2f0125c2b970856e8414d74fc56ab0a17c59c@google.com","name":"Jeff Kaufman"},"message":"1.10.33.2 -> 1.10.33.3","distinct":true,"url":"https://api.github.com/repos/pagespeed/mod_pagespeed/commits/6bdb9af633c9848d8d49114811bd20fefe98d10f"}]}