incubator-pagespeed-ngx.git
2 weeks agoremoving the word boundary \b (#1607) master
Dibyendu Das [Wed, 28 Nov 2018 08:59:25 +0000 (03:59 -0500)] 
removing the word boundary \b (#1607)

because of the word boundary \b sed fails to replace $BIT_SIZE_NAME from the binary url in macOS

4 weeks agoUpdate Nginx in Docker (#1605)
Nico Berlee [Thu, 15 Nov 2018 09:32:22 +0000 (10:32 +0100)] 
Update Nginx in Docker (#1605)

Stable from 1.14.0 to 1.14.1
Mainline from 1.15.5 to 1.15.6

6 weeks agoDependency: update mod_pagespeed to the latest version (#1602)
Otto van der Schaaf [Mon, 29 Oct 2018 07:46:08 +0000 (08:46 +0100)] 
Dependency: update mod_pagespeed to the latest version (#1602)

2 months agoNginx Mainline update to 1.15.5 (#1599)
Elena Tverdokhlib [Mon, 8 Oct 2018 16:13:55 +0000 (19:13 +0300)] 
Nginx Mainline update to 1.15.5 (#1599)

2 months agoDockerfiles for Alpine 3.8/edge build with Nginx 1.14/1.15 (stable/mainline), (direct...
Nico Berlee [Mon, 8 Oct 2018 07:28:19 +0000 (09:28 +0200)] 
Dockerfiles for Alpine 3.8/edge build with Nginx 1.14/1.15 (stable/mainline), (directory structure organised for DockerHub build compatibility). (#1598)

Alpine Linux is a security-oriented, lightweight Linux distribution based on musl libc and busy box. Alpine is therefor the most logical OS base for Docker Images.

Although it does not fix #1181 completely, it helps to overcome the problems of building the PSOL on Alpine,

3 months agoFixed Spelling. (#1582)
Jimmy Casey [Sun, 2 Sep 2018 22:13:56 +0000 (23:13 +0100)] 
Fixed Spelling. (#1582)

7 months agoProperly iterate modules (#1559)
David Sanders [Mon, 14 May 2018 21:49:53 +0000 (14:49 -0700)] 
Properly iterate modules (#1559)

7 months agoFix setting 'gzip_proxied' when applying gzip settings (#1558)
David Sanders [Sun, 13 May 2018 13:06:44 +0000 (06:06 -0700)] 
Fix setting 'gzip_proxied' when applying gzip settings (#1558)

7 months agoAdd wget as dependency for Debian-based distros in build script (#1553)
David Sanders [Sat, 28 Apr 2018 07:05:41 +0000 (01:05 -0600)] 
Add wget as dependency for Debian-based distros in build script (#1553)

7 months agoHonor assume yes when installing dependencies (#1552)
David Sanders [Fri, 27 Apr 2018 21:16:19 +0000 (15:16 -0600)] 
Honor assume yes when installing dependencies (#1552)

10 months agoLicensing: Changes for ASF license compliance (#1528)
Otto van der Schaaf [Fri, 19 Jan 2018 10:21:10 +0000 (11:21 +0100)] 
Licensing: Changes for ASF license compliance (#1528)

* Update/add license banners
* Strip "Author: " comments
* Initialize NOTICE

11 months agoUpdate mod_pagespeed dep to current head
Otto van der Schaaf [Fri, 12 Jan 2018 20:13:19 +0000 (20:13 +0000)] 
Update mod_pagespeed dep to current head

11 months agoUpdate repository links
Otto van der Schaaf [Fri, 12 Jan 2018 20:02:03 +0000 (20:02 +0000)] 
Update repository links

Update sources to reflect that the source of truth for most of the
repositories in pagespeed/ have moved to
github.com/apache/incubator-pagespeed-*

11 months agoUpdate README.md (#1525)
Otto van der Schaaf [Thu, 11 Jan 2018 20:44:27 +0000 (21:44 +0100)] 
Update README.md (#1525)

Fix travis links

11 months agoUnbreak install after repo transfers (#1523)
Otto van der Schaaf [Mon, 8 Jan 2018 16:57:39 +0000 (17:57 +0100)] 
Unbreak install after repo transfers (#1523)

The directory name in the zip we download from github's release
archive will be different now that the project was renamed.
It may be renamed again in the future, so try to obtain the directory
name from the zip.

Fixes https://github.com/apache/incubator-pagespeed-ngx/issues/1521

12 months agoF_SETPIPE_SZ: check for existence before using (#1505)
Otto van der Schaaf [Mon, 27 Nov 2017 22:21:03 +0000 (23:21 +0100)] 
F_SETPIPE_SZ: check for existence before using (#1505)

Kernels before 2.6.35 do not have F_SETPIPE_SZ defined.

Fixes https://github.com/pagespeed/ngx_pagespeed/issues/1503

13 months agoFix ngx_pagespeed installing dependency packages (#1498)
Otto van der Schaaf [Wed, 8 Nov 2017 23:58:49 +0000 (00:58 +0100)] 
Fix ngx_pagespeed installing dependency packages (#1498)

- Fix auto-install of required packages by fixing quoting (thanks @jeffkaufman!).
- Add libuuid-devel for redhat-based distro's too.

Fixes https://github.com/pagespeed/mod_pagespeed/issues/1680

13 months agobuild_ngx_pagespeed.sh: add uuid-dev (#1497)
Otto van der Schaaf [Wed, 8 Nov 2017 22:02:53 +0000 (23:02 +0100)] 
build_ngx_pagespeed.sh: add uuid-dev (#1497)

13 months agoUpdate mod_pagespeed to 569affb1a2baef30a34f9df6c2f152897c0fd896 (#1495)
Otto van der Schaaf [Wed, 1 Nov 2017 21:29:10 +0000 (22:29 +0100)] 
Update mod_pagespeed to 569affb1a2baef30a34f9df6c2f152897c0fd896 (#1495)

14 months agoSet buildtype from env PSOL_BUILDTYPE. (#1486) huibao-webp
Yukihiko SAWANOBORI [Fri, 6 Oct 2017 09:13:18 +0000 (18:13 +0900)] 
Set buildtype from env PSOL_BUILDTYPE. (#1486)

14 months agoFix nginx worker 100% cpu usage (spinning on write returning EAGAIN)
Otto van der Schaaf [Mon, 25 Sep 2017 08:19:29 +0000 (10:19 +0200)] 
Fix nginx worker 100% cpu usage (spinning on write returning EAGAIN)

Bump the pipe capacity, because running out of buffer space may cause
a write to spin indefinitely on EAGAIN.
Bumping the pipe capacity should eliminate the problem in practice,
though in theory the module could still be subject to it.
For now, leaving behind a todo with a suggested solution (should
the problem ever show up again).

Fixes https://github.com/pagespeed/ngx_pagespeed/issues/1380

14 months agonginx-1.13.4: use preprocessor define instead of const for comparison (#1483)
Otto van der Schaaf [Mon, 25 Sep 2017 20:56:58 +0000 (22:56 +0200)] 
nginx-1.13.4: use preprocessor define instead of const for comparison (#1483)

Address a mistake I made in https://github.com/pagespeed/ngx_pagespeed/pull/1458

While testing the draft for 1.12.34.3 I noticed that older nginx
versions would fail to compile. Switch kNginx13_1_4 to a #define so we
can use it to properly compare.

14 months agoFix compiling issue with older nginx (#1470)
PikachuEXE [Mon, 18 Sep 2017 05:27:59 +0000 (13:27 +0800)] 
Fix compiling issue with older nginx (#1470)

14 months ago ! Fix syntax of script (#1469)
PikachuEXE [Mon, 18 Sep 2017 05:27:03 +0000 (13:27 +0800)] 
 ! Fix syntax of script (#1469)

15 months agoFix ignored return code in ps_simple_handler() (#1468)
Otto van der Schaaf [Mon, 11 Sep 2017 19:11:50 +0000 (21:11 +0200)] 
Fix ignored return code in ps_simple_handler() (#1468)

Fix for https://github.com/pagespeed/ngx_pagespeed/issues/1465

15 months agoUpdate build script to pass --skip_deps when called with --no-deps-check (#1461)
PikachuEXE [Thu, 31 Aug 2017 15:53:41 +0000 (23:53 +0800)] 
Update build script to pass --skip_deps when called with --no-deps-check (#1461)

15 months agoUpdate mod_pagespeed testing dependency (#1460)
Otto van der Schaaf [Tue, 22 Aug 2017 14:27:41 +0000 (16:27 +0200)] 
Update mod_pagespeed testing dependency (#1460)

* Update mod_pagespeed testing dependency
(mod_pagespeed commit a7c39a1b5a14f1422fef19e8e9dcb11074ba1c65)
* Add explicit default server{} on secondary_port to so that the recent
tests for css parser improvements land on a vhost with the right filters
enabled.

15 months agoAdress @jmarantz his comment from #1453's review (#1458)
Otto van der Schaaf [Mon, 21 Aug 2017 19:00:49 +0000 (21:00 +0200)] 
Adress @jmarantz his comment from #1453's review (#1458)

PR #1453 was merged too soon, using this PR to address
https://github.com/pagespeed/ngx_pagespeed/pull/1453#discussion_r134222719

(Sorry Josh!)

15 months agonginx 1.13.4: Fix compilation error (#1453)
Otto van der Schaaf [Mon, 21 Aug 2017 10:25:50 +0000 (12:25 +0200)] 
nginx 1.13.4: Fix compilation error (#1453)

* Compilation of ngx_pagespeed broke with the following commit in
nginx: https://github.com/nginx/nginx/commit/129b06dc5dfab7b4513a4f274b3778cd9b8a6a22
This change intends to unbreak it while maintaining backwards
compatibility.

* updates the nginx testing-dependency to 1.13.4

Fixes https://github.com/pagespeed/ngx_pagespeed/issues/1451

17 months agopoint mod_pagespeed submodule to the latest rev (#1433)
Otto van der Schaaf [Tue, 27 Jun 2017 21:53:31 +0000 (23:53 +0200)] 
point mod_pagespeed submodule to the latest rev (#1433)

* point mod_pagespeed submodule to the latest rev
* Add missing vhost for test "Image rewrite with flush"

22 months agoDon't respond with an entitybody to HEAD requests for html (#1376)
Otto van der Schaaf [Tue, 7 Feb 2017 11:02:43 +0000 (12:02 +0100)] 
Don't respond with an entitybody to HEAD requests for html (#1376)

Mind r->header_only when processing the response body.

Fixes https://github.com/pagespeed/ngx_pagespeed/issues/1375

22 months agostandby: add standby mode (#1365)
Jeff Kaufman [Wed, 25 Jan 2017 18:24:15 +0000 (13:24 -0500)] 
standby: add standby mode (#1365)

* standby: add standby mode

Add standby mode for ngx_pagespeed, equivalent to "off" in mod_pagespeed.

With this change "off" is deprecated, and people should use "unplugged" instead.

* Update mps to include test file

* update mps

* update mps

* update mps

22 months agodevel: simplify ngx_pagespeed development flow (#1363)
Jeff Kaufman [Fri, 20 Jan 2017 14:29:34 +0000 (09:29 -0500)] 
devel: simplify ngx_pagespeed development flow (#1363)

* provide scripts/rebuild.sh so you don't have to run make commands in
  two directories
* make test/run_tests.sh able to run with no arguments by setting good defaults.

22 months agofix TEST_TO_RUN in ngx_pagespeed (#1364)
Jeff Kaufman [Fri, 20 Jan 2017 14:28:54 +0000 (09:28 -0500)] 
fix TEST_TO_RUN in ngx_pagespeed (#1364)

Eventually we'll convert remote_config_system_test.sh and nginx_system_test.sh
to use run_test, but until then when TEST_TO_RUN is specified we should skip
the tests in those files and just run system/system_test.sh

22 months agoAllow use of "unplugged" without crashing (#1362)
Jeff Kaufman [Thu, 19 Jan 2017 18:48:01 +0000 (13:48 -0500)] 
Allow use of "unplugged" without crashing (#1362)

* update mps

* stop crashing when unplugged

* formatting fixes

23 months agoinstall: add --additional-nginx-configure-arguments option (#1344)
Jeff Kaufman [Tue, 3 Jan 2017 12:07:50 +0000 (07:07 -0500)] 
install: add --additional-nginx-configure-arguments option (#1344)

Instead of requiring people to answer a prompt to specify which additional
arguments they would like nginx to be built with, allow people to specify
arguments like this on the command line.  The quoting is not ideal, since
it's important to pass spaces etc properly all the way through to the
underlying commands, but I give an example in the help text.

Fixes https://github.com/pagespeed/ngx_pagespeed/issues/1340

23 months agotravis: send emails to new pagespeed-ci group instead of individual devs (#1341)
Jeff Kaufman [Mon, 19 Dec 2016 18:45:56 +0000 (13:45 -0500)] 
travis: send emails to new pagespeed-ci group instead of individual devs (#1341)

23 months agotravis: get it building and passing tests (#1339)
Jeff Kaufman [Mon, 19 Dec 2016 18:21:33 +0000 (13:21 -0500)] 
travis: get it building and passing tests (#1339)

* travis: get it building and passing tests

I initially planned to get this working on Ubuntu 12, but decided it wasn't
worth it and switched to Ubuntu 14 (Trusty).  Now it passes tests on Travis, so
I've added other people's emails back to the set of people to send emails to.

Also update mps while we're at it.

* update mps

* update mps

* ngx_pagespeed depends on luuid

* update mps

23 months agoconfig: use Release binaries when building with --with-debug if that's all that's...
Jeff Kaufman [Wed, 14 Dec 2016 15:13:45 +0000 (10:13 -0500)] 
config: use Release binaries when building with --with-debug if that's all that's available (#1332)

* config: use Release binaries when building with --with-debug if that's all that's available

* prompt the user before going ahead and using release binaries

* exit 1, reword message

2 years agolicensing: correct bash license comments (#1336)
Jeff Kaufman [Mon, 12 Dec 2016 18:40:41 +0000 (13:40 -0500)] 
licensing: correct bash license comments (#1336)

A few bash files were missing license comments

2 years agoinstaller: allow --dynamic-module to be used with tags (#1334)
Jeff Kaufman [Thu, 8 Dec 2016 20:14:12 +0000 (15:14 -0500)] 
installer: allow --dynamic-module to be used with tags (#1334)

* installer: allow --dynamic-module to be used with tags

A user-friendly check with dynamic module building tries to warn people early if the version of ngx_pagespeed they're building is too old to be compiled as a dynamic module, but this can't work with tags.  If we're given something other than a numeric version number, don't try to perform this check.

Fixes https://github.com/pagespeed/mod_pagespeed/issues/1443

2 years agoAdd format_binary_url.sh (#1331)
hillsp [Tue, 6 Dec 2016 18:16:29 +0000 (13:16 -0500)] 
Add format_binary_url.sh (#1331)

2 years agobranch-names: now that trunk-tracking is master, update references (#1330)
Jeff Kaufman [Tue, 6 Dec 2016 17:01:50 +0000 (12:01 -0500)] 
branch-names: now that trunk-tracking is master, update references (#1330)

2 years agoDelete prepare_psol.sh (#1310) 1329/head
hillsp [Wed, 2 Nov 2016 17:06:29 +0000 (13:06 -0400)] 
Delete prepare_psol.sh (#1310)

Replaced with install/build_psol.sh  in mod_pagespeed.

2 years agoinstall: add support for building psol from source and setting up for development...
Jeff Kaufman [Tue, 6 Dec 2016 14:21:42 +0000 (09:21 -0500)] 
install: add support for building psol from source and setting up for development (#1313)

* adds support for `--psol-from-source` so you don't need binary modules, and `--devel` so you can run our tests without going and getting all our dependencies
* adds submodules for testing: mod_pagespeed, ngx_cache_purge etc
* adds support for running as:
```
git clone git@github.com:pagespeed/ngx_pagespeed.git
cd ngx_pagespeed/
git checkout <branch>
scripts/build_ngx_pagespeed.sh [options]
```
* depends on the scripts @hillsp is working on so that we can just check out mod_pagespeed and ask it to build and rebuild itself
* adds colors to output to make it easier to read

2 years agoMark CentralControllerPort as Experimental (#1326)
hillsp [Wed, 30 Nov 2016 19:43:55 +0000 (14:43 -0500)] 
Mark CentralControllerPort as Experimental (#1326)

2 years agoAdd a new server-name to help test a repro for broken pagespeed resources (#1324)
Joshua Marantz [Mon, 21 Nov 2016 19:11:56 +0000 (14:11 -0500)] 
Add a new server-name to help test a repro for broken pagespeed resources (#1324)

after ipro-recorded optimations.

for https://github.com/pagespeed/mod_pagespeed/issues/1145
and https://github.com/pagespeed/ngx_pagespeed/issues/1319 .

2 years agoNeed clone --recursive
hillsp [Mon, 21 Nov 2016 15:49:59 +0000 (10:49 -0500)] 
Need clone --recursive

2 years agoFirst pass at modernising travis.yml
hillsp [Mon, 21 Nov 2016 15:38:05 +0000 (10:38 -0500)] 
First pass at modernising travis.yml

2 years agotest config for disabling rewriters in ajax (#1320)
Joshua Marantz [Mon, 14 Nov 2016 13:09:44 +0000 (08:09 -0500)] 
test config for disabling rewriters in ajax (#1320)

2 years agotest: run_tests.sh should just assume 8050 and 8051 (#1315)
Jeff Kaufman [Thu, 3 Nov 2016 18:58:05 +0000 (14:58 -0400)] 
test: run_tests.sh should just assume 8050 and 8051 (#1315)

Right now run_tests.sh assumes it has 8053 and 9991 without causing any
problems.  Start assuming we have 8050 and 8051 as well instead of
requiring people to pass them in.

2 years agoremote-config: renaming paths to be more informative (#1312)
Jeff Kaufman [Thu, 3 Nov 2016 11:03:31 +0000 (07:03 -0400)] 
remote-config: renaming paths to be more informative (#1312)

2 years agoremote-config: use one port for testing, against a new python server (#1293)
Jeff Kaufman [Tue, 1 Nov 2016 17:29:25 +0000 (12:29 -0500)] 
remote-config: use one port for testing, against a new python server (#1293)

* remote-config: use one port for testing, against a new python-based server

* properly reference remote configu urls

2 years agoFix non-zero exit code even if the script succeed (#1302)
Sen Jiang [Tue, 25 Oct 2016 14:35:01 +0000 (07:35 -0700)] 
Fix non-zero exit code even if the script succeed (#1302)

The script succeeded but the exit code is 1 because it's not a dryrun so the last command failed.

2 years agoinstaller: reference the --no-deps-check option properly (#1305)
Jeff Kaufman [Mon, 24 Oct 2016 17:00:27 +0000 (12:00 -0500)] 
installer: reference the --no-deps-check option properly (#1305)

2 years agoFix sparsehash path (#1300)
hillsp [Thu, 20 Oct 2016 17:55:21 +0000 (13:55 -0400)] 
Fix sparsehash path (#1300)

2 years agoOur config tests need to be compiled with --std=c++11 now (#1295)
Maks Orlovich [Thu, 13 Oct 2016 18:50:41 +0000 (14:50 -0400)] 
Our config tests need to be compiled with --std=c++11 now (#1295)

2 years agoconfig: give full path to autoconf.err if 'checking for psol' fails. (#1294)
Jeff Kaufman [Thu, 13 Oct 2016 14:42:24 +0000 (10:42 -0400)] 
config: give full path to autoconf.err if 'checking for psol' fails. (#1294)

2 years agotesting: check_flushing needs to know the host name (#1292)
Jeff Kaufman [Wed, 12 Oct 2016 22:02:56 +0000 (18:02 -0400)] 
testing: check_flushing needs to know the host name (#1292)

2 years agoinstall: fix quoting (#1291)
Jeff Kaufman [Wed, 12 Oct 2016 16:22:37 +0000 (12:22 -0400)] 
install: fix quoting (#1291)

We want to parse $additional_configure_args into an array the way the
shell would, which we're using eval for.  But because we were missing
quotes whitespace was being stripped.  Basically:

    $ function lines() { for x in "$@"; do echo "$x"; done; }

    $ lines a "b   c" d
    a
    b   c
    d

    $ a='a "b   c" d'
    $ eval e=($a)
    $ eval f=("$a")

    $ lines "${e[@]}"
    a
    b c
    d

    $ lines "${f[@]}"
    a
    b   c
    d

We were doing this like e when we should have been doing it like f.

2 years agoFix travis.yaml for new DEPS file
hillsp [Sat, 8 Oct 2016 01:44:32 +0000 (21:44 -0400)] 
Fix travis.yaml for new DEPS file

2 years agotests: check flushing changed its arguments (#1288)
Jeff Kaufman [Fri, 7 Oct 2016 19:51:24 +0000 (15:51 -0400)] 
tests: check flushing changed its arguments (#1288)

2 years agobuild_ngx_pagespeed: handle quoted arguments (#1286)
Jeff Kaufman [Wed, 5 Oct 2016 12:26:04 +0000 (08:26 -0400)] 
build_ngx_pagespeed: handle quoted arguments (#1286)

* build_ngx_pagespeed: handle quoted arguments

If you enter --with-cc-opt='-arg1 -arg2' when the script asks for
additional arguments we want that to remain as one chunk all the way
through to being a single argument to ./configure

2 years agoinstall: default to latest-beta (#1283)
Jeff Kaufman [Mon, 3 Oct 2016 19:16:24 +0000 (15:16 -0400)] 
install: default to latest-beta (#1283)

* install: default to latest-beta

* latest stable instead

2 years agoAdd controller tests for nginx (#1280)
hillsp [Mon, 26 Sep 2016 17:55:22 +0000 (13:55 -0400)] 
Add controller tests for nginx (#1280)

2 years agoWe need -lstdc++ to come after psol.a (#1273) (#1274)
Jeff Kaufman [Sat, 24 Sep 2016 20:04:52 +0000 (16:04 -0400)] 
We need -lstdc++ to come after psol.a (#1273) (#1274)

2 years agoload-from-file: config for testing load from file with a low size limit (#1268)
Jeff Kaufman [Fri, 23 Sep 2016 22:20:26 +0000 (18:20 -0400)] 
load-from-file: config for testing load from file with a low size limit (#1268)

nginx side of the fix for https://github.com/pagespeed/mod_pagespeed/issues/1386

2 years agoinstall: add script to install nginx_pagespeed (#1263)
Jeff Kaufman [Wed, 21 Sep 2016 11:48:03 +0000 (07:48 -0400)] 
install: add script to install nginx_pagespeed (#1263)

install: add script to install nginx_pagespeed

Automated version of the instructions on https://developers.google.com/speed/pagespeed/module/build_ngx_pagespeed_from_source

Tested on fresh vms of debian8, centos 6 and 7, rhel 6 and 7, and ubuntu lts 12, 14, and 16

Fixes https://github.com/pagespeed/ngx_pagespeed/issues/1262

2 years agoAdd default setting for whether to test with a controller port. (#1278)
Joshua Marantz [Tue, 20 Sep 2016 19:42:47 +0000 (15:42 -0400)] 
Add default setting for whether to test with a controller port. (#1278)

* Add default setting for whether to test with a controller port.

* Use RUN_CONTROLLER_TEST, which will actually be defaulted in
system/system_test.sh.  Comment out the controller port configuration
in the default nginx template.

* Finish up the conditionalization of the babysitter tests.

* Run the reload tests even with the controller off.

* clean up the diffs a little.

2 years agothe remote config test is in a separate script now, so run it from the master test...
Joshua Marantz [Mon, 19 Sep 2016 23:25:24 +0000 (19:25 -0400)] 
the remote config test is in a separate script now, so run it from the master test script (#1260)

2 years agoWork-around broken GCE Ubuntu repo
hillsp [Thu, 15 Sep 2016 20:23:11 +0000 (16:23 -0400)] 
Work-around broken GCE Ubuntu repo

2 years agoAdd GLOBAL_STATISTICS_URL variable (#1270)
hillsp [Mon, 12 Sep 2016 17:52:06 +0000 (13:52 -0400)] 
Add GLOBAL_STATISTICS_URL variable (#1270)

2 years agospecial-responses: fix for hanging sometimes on special responses (#1238)
Otto van der Schaaf [Wed, 7 Sep 2016 08:49:53 +0000 (10:49 +0200)] 
special-responses: fix for hanging sometimes on special responses (#1238)

After calling ngx_http_filter_finalize_request return NGX_DONE
as we are still in pre-access phase. This bug shows up when the
"special" response is generated from, for example, php as opposed
to a static file.

Second part of the fix for
https://github.com/pagespeed/ngx_pagespeed/issues/1146

2 years agoAdditional suppressions for new RE2 (#1266)
hillsp [Tue, 6 Sep 2016 14:07:40 +0000 (10:07 -0400)] 
Additional suppressions for new RE2 (#1266)

2 years agoAlso look for headers in /url in mod_pagespeed tree (#1264)
Maks Orlovich [Fri, 2 Sep 2016 19:11:50 +0000 (15:11 -0400)] 
Also look for headers in /url in mod_pagespeed tree (#1264)

This is needed because the current GURL implementation there --- and hence
google_url.h needs an extra compat header that's there.

2 years agocombine-ids: add configuration for system test (#1259)
Jeff Kaufman [Thu, 1 Sep 2016 19:04:03 +0000 (15:04 -0400)] 
combine-ids: add configuration for system test (#1259)

2 years agoallow sloppy exit (#1258)
Joshua Marantz [Mon, 29 Aug 2016 21:06:26 +0000 (17:06 -0400)] 
allow sloppy exit (#1258)

* set up a test subdir with cc:public

* allow 'adding function to sequence' warnings on shutdown, without breaking the nginx debug tests

2 years agoMake NgxMessageHandler::FileMessage() add data into internal buffer as well (#1255)
Egor Suvorov [Thu, 25 Aug 2016 19:10:46 +0000 (15:10 -0400)] 
Make NgxMessageHandler::FileMessage() add data into internal buffer as well (#1255)

2 years agoTest shm checkpointing. Relies on shm checkpointing implementation in mod_pagespeed...
Jeff Kaufman [Tue, 23 Aug 2016 11:46:20 +0000 (07:46 -0400)] 
Test shm checkpointing.  Relies on shm checkpointing implementation in mod_pagespeed. (#930)

2 years agoAllow processing of script variablies in more options (#1239)
Otto van der Schaaf [Wed, 10 Aug 2016 19:04:47 +0000 (21:04 +0200)] 
Allow processing of script variablies in more options (#1239)

All directory- and process- scoped options are allowed with this
change, when "ProcessScriptVariables all" is specified.

Option list compiled from:
https://docs.google.com/spreadsheets/d/1qJSUZqd2Te-4SDtm1Av6u-AckNv_oHEp1qdaUjNqZoM/edit#gid=0

2 years agoUpdate .travis.yml
Jeffrey Crowell [Tue, 9 Aug 2016 21:45:19 +0000 (17:45 -0400)] 
Update .travis.yml

2 years agorewrite-options: don't turn on CoreFilters just because of query params (#1228)
Jeff Kaufman [Thu, 4 Aug 2016 17:44:46 +0000 (10:44 -0700)] 
rewrite-options: don't turn on CoreFilters just because of query params (#1228)

Fixes https://github.com/pagespeed/ngx_pagespeed/issues/1190

2 years agonginx side of https://github.com/pagespeed/mod_pagespeed/issues/1371 (#1235)
Jeffrey Crowell [Wed, 3 Aug 2016 18:09:22 +0000 (13:09 -0500)] 
nginx side of https://github.com/pagespeed/mod_pagespeed/issues/1371 (#1235)

nginx side of the change
https://github.com/pagespeed/mod_pagespeed/issues/1371

2 years agorewrite-domains: add test configuration for rewriting static assets (#1223)
Jeff Kaufman [Tue, 2 Aug 2016 18:46:20 +0000 (11:46 -0700)] 
rewrite-domains: add test configuration for rewriting static assets (#1223)

Needed for fixing https://github.com/pagespeed/mod_pagespeed/issues/1350

2 years agojson: add content-type and cache-control for example json file (#1224)
Jeff Kaufman [Mon, 18 Jul 2016 15:15:31 +0000 (11:15 -0400)] 
json: add content-type and cache-control for example json file (#1224)

2 years agoCheck options->IsAllowed() before rewriting html. (#1225)
Otto van der Schaaf [Fri, 15 Jul 2016 18:41:19 +0000 (20:41 +0200)] 
Check options->IsAllowed() before rewriting html. (#1225)

Fixes https://github.com/pagespeed/ngx_pagespeed/issues/1226

2 years agoMake ngx_pagespeed behave like mod_pagespeed with regard to Flushing (#1217)
Otto van der Schaaf [Tue, 12 Jul 2016 16:26:38 +0000 (18:26 +0200)] 
Make ngx_pagespeed behave like mod_pagespeed with regard to Flushing (#1217)

This change makes ngx_pagespeed listen to the FollowFlushes option.
When set to on (=default), ngx_pagespeed will forward incoming flushes
to ProxyFetch. When writing output, we'll now also set the flush flag on
the buffers we are about to send downstream.

Companion to mps commit: https://github.com/pagespeed/mod_pagespeed/commit/02de03e825bbd1f8d4ad4e1a1bef5263a16f3857

2 years agoFix for when there is only server-level configuration (#1218)
Otto van der Schaaf [Wed, 6 Jul 2016 14:13:05 +0000 (16:13 +0200)] 
Fix for when there is only server-level configuration (#1218)

When there is no top-level configuration but only server configuration,
we don't have main options to set and consider. This prevents
a NULL dereference in that case. (Later on we'll writes out a message
about why we cannot startup with this configuration).

2 years agoFix segfault when there's no http{} block in nginx.conf (#1221)
Otto van der Schaaf [Wed, 6 Jul 2016 14:12:04 +0000 (16:12 +0200)] 
Fix segfault when there's no http{} block in nginx.conf (#1221)

Should fix https://github.com/pagespeed/ngx_pagespeed/issues/1220

2 years agourl-valued-attributes: tests for mod_pagespeed/f2f0075
Jeff Kaufman [Fri, 24 Jun 2016 20:54:23 +0000 (16:54 -0400)] 
url-valued-attributes: tests for mod_pagespeed/f2f0075

2 years agoset up a test subdir with cc:public (#1215)
Joshua Marantz [Thu, 16 Jun 2016 13:54:06 +0000 (09:54 -0400)] 
set up a test subdir with cc:public (#1215)

2 years agoSuppress controller warnings in test, fix up kill signals (#1210)
hillsp [Fri, 10 Jun 2016 18:54:49 +0000 (14:54 -0400)] 
Suppress controller warnings in test, fix up kill signals (#1210)

* Suppress controller warnings in test, fix up kill signals

* Switch valgrind shutdown to QUIT

2 years agoREADME.md: fix link in travis badge
Jeffrey Crowell [Wed, 8 Jun 2016 17:06:25 +0000 (13:06 -0400)] 
README.md: fix link in travis badge

2 years agosvn of depot_tools is going away, migrate to git. (#1211)
Jeffrey Crowell [Wed, 8 Jun 2016 17:05:22 +0000 (13:05 -0400)] 
svn of depot_tools is going away, migrate to git. (#1211)

for some reason we were still using the svn for travis here, but git for everything else...

2 years agoUpdate for API change: kProcessScope is about to become kLegacyProcessScope (#1205)
Maks Orlovich [Mon, 30 May 2016 16:52:41 +0000 (12:52 -0400)] 
Update for API change: kProcessScope is about to become kLegacyProcessScope (#1205)

The renaming is to make it clear that kProcessScopeStrict (aka what
ngx_pagespeed does for everything already) is what new code should be
using, rather than current kProcessScope/upcoming kLegacyProcessScope
which does weird backwards compatible things in mod_pagespeed.

2 years agoMake sure RewriteDriverFactory::default_options gets the top-level proces-scope optio...
Maks Orlovich [Fri, 20 May 2016 14:07:24 +0000 (10:07 -0400)] 
Make sure RewriteDriverFactory::default_options gets the top-level proces-scope options. (#1203)

Needed for ImageMaxRewritesAtOnce to work with latest revisions.

2 years agoPass the install-path into Makefile (#1200)
Joshua Marantz [Mon, 16 May 2016 17:32:28 +0000 (13:32 -0400)] 
Pass the install-path into Makefile (#1200)

* Pass the install-path into Makefile, rather than having the Makefile
depend on PWD.

* Move the double-quotes be before the value, to be consistent with other
Makefiles.

2 years agoRemove stray -q on pgrep.
Steve Hill [Fri, 13 May 2016 21:47:09 +0000 (17:47 -0400)] 
Remove stray -q on pgrep.

2 years agoFix nginx_system_test so that it doesn't wait for verify_nginx_release.sh (#1199)
hillsp [Fri, 13 May 2016 20:06:38 +0000 (16:06 -0400)] 
Fix nginx_system_test so that it doesn't wait for verify_nginx_release.sh (#1199)

or wait indefinitely if it can't kill nginx.

2 years agoDuring system-tests, don't write into the source-controlled htdocs area. (#1197)
Joshua Marantz [Fri, 13 May 2016 12:44:14 +0000 (08:44 -0400)] 
During system-tests, don't write into the source-controlled htdocs area. (#1197)

Instead, use a new (not-yet-committed) target in Makefile.tests to build
a mirror htdocs in test/tmp/root that sym-links the large readonly
section and deep copies the test areas for purge and flush tests.