[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r224992690 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -95,6 +95,12 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r224992690 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -95,6 +95,12 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r224992740 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -95,6 +95,12 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r224992613 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -95,6 +95,12 @@

[GitHub] injust commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
injust commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r225034179 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -71,11 +71,11 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r225014034 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -71,11 +71,11 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r225014000 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -71,11 +71,11 @@

[GitHub] oschaaf commented on issue #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-16 Thread GitBox
oschaaf commented on issue #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#issuecomment-430149547 This looks good to me; did you get a chance to test this? Is this ready for merging?

[GitHub] oschaaf commented on issue #1803: defer_javascript always fails on last script of page

2018-10-19 Thread GitBox
oschaaf commented on issue #1803: defer_javascript always fails on last script of page URL: https://github.com/apache/incubator-pagespeed-mod/issues/1803#issuecomment-431285562 Looking at the callstack when catching an exception, I observed stuff like: ``` new

[GitHub] alexgran commented on issue #1803: defer_javascript always fails on last script of page

2018-10-19 Thread GitBox
alexgran commented on issue #1803: defer_javascript always fails on last script of page URL: https://github.com/apache/incubator-pagespeed-mod/issues/1803#issuecomment-431295751 Hi Otto, thanks for the feedback. Yes, this was my initial assumption as well. ShipperHQ is doing some

[GitHub] alexgran opened a new issue #1803: defer_javascript always fails on last script of page

2018-10-17 Thread GitBox
alexgran opened a new issue #1803: defer_javascript always fails on last script of page URL: https://github.com/apache/incubator-pagespeed-mod/issues/1803 Hi, it looks like defer_javascript has an issue (off-by-one?) that makes it stumble on the last script of the page. I spend few

[GitHub] vitaliytv commented on issue #1599: Nginx Mainline update to 1.15.5

2018-10-18 Thread GitBox
vitaliytv commented on issue #1599: Nginx Mainline update to 1.15.5 URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1599#issuecomment-431003253 @nberlee any news from support? thanks This is an automated message

[GitHub] oschaaf closed pull request #69: rename .src.rpm file to current file name.

2018-10-16 Thread GitBox
oschaaf closed pull request #69: rename .src.rpm file to current file name. URL: https://github.com/apache/incubator-pagespeed-cpanel/pull/69 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] oschaaf commented on issue #69: rename .src.rpm file to current file name.

2018-10-16 Thread GitBox
oschaaf commented on issue #69: rename .src.rpm file to current file name. URL: https://github.com/apache/incubator-pagespeed-cpanel/pull/69#issuecomment-430150254 @mattcheah thanks. I merged https://github.com/apache/incubator-pagespeed-cpanel/pull/48, let's close this one then.

[GitHub] oschaaf closed pull request #48: incompatible with mod_ruid2

2018-10-16 Thread GitBox
oschaaf closed pull request #48: incompatible with mod_ruid2 URL: https://github.com/apache/incubator-pagespeed-cpanel/pull/48 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-16 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r225511479 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -88,21 +88,30 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-16 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r225510986 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -88,21 +88,30 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-16 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r225510970 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -88,21 +88,30 @@

[GitHub] redbox-cdn opened a new issue #1600: centos7 gcc4.8.5 compile error

2018-10-23 Thread GitBox
redbox-cdn opened a new issue #1600: centos7 gcc4.8.5 compile error URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1600 Hi does anyone know install devtoolset-2-gcc-c++ on centos7 gcc 4.8.5? I tried manual install incubator-pagespeed-ngx-1.12.34.3-stable but error

[GitHub] redbox-cdn commented on issue #1600: centos7 gcc4.8.5 compile error

2018-10-24 Thread GitBox
redbox-cdn commented on issue #1600: centos7 gcc4.8.5 compile error URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1600#issuecomment-432600308 > Why do you try to install the Scientific Linux 6 devtoolset-2 on EL7? > The dependencies simply can't be resolved for

[GitHub] redbox-cdn closed issue #1600: centos7 gcc4.8.5 compile error

2018-10-24 Thread GitBox
redbox-cdn closed issue #1600: centos7 gcc4.8.5 compile error URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1600 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] danielbeardsley commented on issue #1076: Apache pagespeed segmentation fault

2018-10-24 Thread GitBox
danielbeardsley commented on issue #1076: Apache pagespeed segmentation fault URL: https://github.com/apache/incubator-pagespeed-mod/issues/1076#issuecomment-432777246 We have had nearly the exact same problem (with apache 2.4 though) with mod_pagespeed. I've noticed that it works fine

[GitHub] danielbeardsley commented on issue #1076: Apache pagespeed segmentation fault

2018-10-24 Thread GitBox
danielbeardsley commented on issue #1076: Apache pagespeed segmentation fault URL: https://github.com/apache/incubator-pagespeed-mod/issues/1076#issuecomment-432777432 Looks like there was a similar issue on reload for the ngx module:

[GitHub] oschaaf commented on issue #1076: Apache pagespeed segmentation fault

2018-10-24 Thread GitBox
oschaaf commented on issue #1076: Apache pagespeed segmentation fault URL: https://github.com/apache/incubator-pagespeed-mod/issues/1076#issuecomment-432812088 I had a look, and I think that while https://github.com/apache/incubator-pagespeed-ngx/issues/1120 may be similar in symptoms at

[GitHub] choungchamnab opened a new issue #1804: Completely remove mod_pagespeed for HTTPD (Centos 7.0)

2018-10-24 Thread GitBox
choungchamnab opened a new issue #1804: Completely remove mod_pagespeed for HTTPD (Centos 7.0) URL: https://github.com/apache/incubator-pagespeed-mod/issues/1804 After install mod_pagespeed on my server, The web pages is displayed incorrectly. Is there a way to complete remove it from

[GitHub] danielbeardsley commented on issue #1076: Apache pagespeed segmentation fault

2018-10-24 Thread GitBox
danielbeardsley commented on issue #1076: Apache pagespeed segmentation fault URL: https://github.com/apache/incubator-pagespeed-mod/issues/1076#issuecomment-432845442 > Are you able to get a reliable reproduction of the problem based on that? Yes, every time. As simple as removing

[GitHub] oschaaf commented on issue #1076: Apache pagespeed segmentation fault

2018-10-24 Thread GitBox
oschaaf commented on issue #1076: Apache pagespeed segmentation fault URL: https://github.com/apache/incubator-pagespeed-mod/issues/1076#issuecomment-432846809 awesome -- let me see if I can get a VM set up with a reproduction based on that information

[GitHub] oschaaf opened a new pull request #1602: Dependency: update mod_pagespeed to the latest version

2018-10-28 Thread GitBox
oschaaf opened a new pull request #1602: Dependency: update mod_pagespeed to the latest version URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1602 This is an automated message from the Apache Git Service. To

[GitHub] gradient22 opened a new issue #1603: Fetch and render - CSS, images, javascript temporarily unreachable

2018-10-29 Thread GitBox
gradient22 opened a new issue #1603: Fetch and render - CSS, images, javascript temporarily unreachable URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1603 Hello, I fetch and render google, but the fetching is partial and it is showing messages for images, css and

[GitHub] oschaaf closed pull request #1602: Dependency: update mod_pagespeed to the latest version

2018-10-29 Thread GitBox
oschaaf closed pull request #1602: Dependency: update mod_pagespeed to the latest version URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1602 This is an automated message from the Apache Git Service. To respond

[GitHub] oschaaf commented on issue #1806: mod_pagespeed behind https terminating proxy ignores X-Forwarded-Proto header

2018-11-01 Thread GitBox
oschaaf commented on issue #1806: mod_pagespeed behind https terminating proxy ignores X-Forwarded-Proto header URL: https://github.com/apache/incubator-pagespeed-mod/issues/1806#issuecomment-435057680 Yes, see https://www.modpagespeed.com/doc/https_support#RespectXForwardedProto

[GitHub] minamijoyo commented on issue #1805: Fix typo in doc/https_support.html

2018-10-31 Thread GitBox
minamijoyo commented on issue #1805: Fix typo in doc/https_support.html URL: https://github.com/apache/incubator-pagespeed-mod/pull/1805#issuecomment-434911870 I signed it! This is an automated message from the Apache Git

[GitHub] minamijoyo opened a new pull request #1805: Fix typo in doc/https_support.html

2018-10-31 Thread GitBox
minamijoyo opened a new pull request #1805: Fix typo in doc/https_support.html URL: https://github.com/apache/incubator-pagespeed-mod/pull/1805 This is an automated message from the Apache Git Service. To respond to the

[GitHub] googlebot commented on issue #1805: Fix typo in doc/https_support.html

2018-10-31 Thread GitBox
googlebot commented on issue #1805: Fix typo in doc/https_support.html URL: https://github.com/apache/incubator-pagespeed-mod/pull/1805#issuecomment-434907065 Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look

[GitHub] jmarantz closed pull request #1805: Fix typo in doc/https_support.html

2018-10-31 Thread GitBox
jmarantz closed pull request #1805: Fix typo in doc/https_support.html URL: https://github.com/apache/incubator-pagespeed-mod/pull/1805 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] AlexeyKosov opened a new issue #1604: Disable pagespeed optimization for specific user agent

2018-11-01 Thread GitBox
AlexeyKosov opened a new issue #1604: Disable pagespeed optimization for specific user agent URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1604 Is it possible to disable some or all optimizations for a certain user agent (crawler)? When it crawls the site, CPU usage for

[GitHub] craigmj opened a new issue #1806: mod_pagespeed behind https terminating proxy ignores X-Forwarded-Proto header

2018-11-01 Thread GitBox
craigmj opened a new issue #1806: mod_pagespeed behind https terminating proxy ignores X-Forwarded-Proto header URL: https://github.com/apache/incubator-pagespeed-mod/issues/1806 I'm running HAProxy, forwarding to Apache running mod_pagespeed. HAProxy terminates the https connection, so I

[GitHub] justinmayer opened a new issue #1601: Brotli image compression

2018-10-26 Thread GitBox
justinmayer opened a new issue #1601: Brotli image compression URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1601 Now that the [previous ngx-pagespeed-alpine repo](https://github.com/We-Amp/ngx-pagespeed-alpine) has been [moved

[GitHub] jmarantz commented on issue #1804: Completely remove mod_pagespeed for HTTPD (Centos 7.0)

2018-10-25 Thread GitBox
jmarantz commented on issue #1804: Completely remove mod_pagespeed for HTTPD (Centos 7.0) URL: https://github.com/apache/incubator-pagespeed-mod/issues/1804#issuecomment-433054283 How did you install it? E.g. if by "yum" can't you uninstall by "yum"? In any case, try ```

[GitHub] jmarantz closed issue #1804: Completely remove mod_pagespeed for HTTPD (Centos 7.0)

2018-10-25 Thread GitBox
jmarantz closed issue #1804: Completely remove mod_pagespeed for HTTPD (Centos 7.0) URL: https://github.com/apache/incubator-pagespeed-mod/issues/1804 This is an automated message from the Apache Git Service. To respond to

[GitHub] jmarantz commented on issue #1601: Brotli image compression

2018-10-26 Thread GitBox
jmarantz commented on issue #1601: Brotli image compression URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1601#issuecomment-433534686 I just want to cross-reference this with https://github.com/apache/incubator-pagespeed-mod/issues/1148 which has a lot more history and

[GitHub] iishtiaque commented on issue #1213: Supporting pagespeed on ARM devices?

2018-10-25 Thread GitBox
iishtiaque commented on issue #1213: Supporting pagespeed on ARM devices? URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1213#issuecomment-433157640 +1 It would be a really nice thing to have This is an

[GitHub] oschaaf commented on issue #1601: Brotli image compression

2018-10-26 Thread GitBox
oschaaf commented on issue #1601: Brotli image compression URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1601#issuecomment-433536832 I, like Josh, would also like deep integration when thinking about Brotli. But in the context of the ngx-pagespeed-alpine image I would love

[GitHub] wolfgangpue commented on issue #1572: Fetch as Google & images Temporarily unreachable

2018-10-27 Thread GitBox
wolfgangpue commented on issue #1572: Fetch as Google & images Temporarily unreachable URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1572#issuecomment-433635601 I also have temporarily unreachable errors in "fetch as google" and I have disallow: in my robots.txt. The

[GitHub] Lofesa commented on issue #1604: Disable pagespeed optimization for specific user agent

2018-11-05 Thread GitBox
Lofesa commented on issue #1604: Disable pagespeed optimization for specific user agent URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1604#issuecomment-435823446 Hi @AlexeyKosov Have you tried the DisableBackgroundFetchesForBots filter? Or you can use some like

[GitHub] craigmj commented on issue #1806: mod_pagespeed behind https terminating proxy ignores X-Forwarded-Proto header

2018-11-01 Thread GitBox
craigmj commented on issue #1806: mod_pagespeed behind https terminating proxy ignores X-Forwarded-Proto header URL: https://github.com/apache/incubator-pagespeed-mod/issues/1806#issuecomment-435265394 Many thanks - I even read that page (I think) and somehow missed that!

[GitHub] ms217 commented on issue #1600: centos7 gcc4.8.5 compile error

2018-10-24 Thread GitBox
ms217 commented on issue #1600: centos7 gcc4.8.5 compile error URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1600#issuecomment-432560844 Why do you try to install the Scientific Linux 6 devtoolset-2 on EL7? The dependencies simply can't be resolved for

[GitHub] AngelDeaD commented on issue #876: Content Security Policy support.

2018-11-05 Thread GitBox
AngelDeaD commented on issue #876: Content Security Policy support. URL: https://github.com/apache/incubator-pagespeed-mod/issues/876#issuecomment-435999044 Any update on this? I have problem with lazy load option in pagespeed. It adds static js code with "onload=" which CSP does not

[GitHub] oschaaf commented on issue #876: Content Security Policy support.

2018-11-06 Thread GitBox
oschaaf commented on issue #876: Content Security Policy support. URL: https://github.com/apache/incubator-pagespeed-mod/issues/876#issuecomment-436456872 @AngelDeaD could you add a concrete example of how the module would rewrite its input to illustrate your goal?

[GitHub] AngelDeaD commented on issue #876: Content Security Policy support.

2018-11-06 Thread GitBox
AngelDeaD commented on issue #876: Content Security Policy support. URL: https://github.com/apache/incubator-pagespeed-mod/issues/876#issuecomment-436462444 This is line that I have problem with: `https://www..XXX/XXX.jpg; srcset="https://www..XXX/XXX.jpg 828w,

[GitHub] jmarantz commented on issue #1605: Update Nginx in Docker

2018-11-08 Thread GitBox
jmarantz commented on issue #1605: Update Nginx in Docker URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1605#issuecomment-436971578 anyone know what's up with the CI failure? This is an automated message from

[GitHub] oschaaf commented on issue #1605: Update Nginx in Docker

2018-11-08 Thread GitBox
oschaaf commented on issue #1605: Update Nginx in Docker URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1605#issuecomment-436993823 CI passed This is an automated message from the Apache Git Service. To respond

[GitHub] nberlee opened a new pull request #1605: Update Nginx in Docker

2018-11-08 Thread GitBox
nberlee opened a new pull request #1605: Update Nginx in Docker URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1605 Stable from 1.14.0 to 1.14.1 Mainline from 1.15.5 to 1.15.6 http://nginx.org/en/security_advisories.html

[GitHub] oschaaf commented on issue #1598: Dockerfiles for Alpine 3.8/edge build with Nginx 1.14/1.15 (stable/ma…

2018-10-03 Thread GitBox
oschaaf commented on issue #1598: Dockerfiles for Alpine 3.8/edge build with Nginx 1.14/1.15 (stable/ma… URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1598#issuecomment-426615327 (actually, mps already has libpng 1.6 on the bleeding edge version:

[GitHub] oschaaf commented on issue #1598: Dockerfiles for Alpine 3.8/edge build with Nginx 1.14/1.15 (stable/ma…

2018-10-03 Thread GitBox
oschaaf commented on issue #1598: Dockerfiles for Alpine 3.8/edge build with Nginx 1.14/1.15 (stable/ma… URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1598#issuecomment-426614356 Re: "Should we also go ahead & upgrade the trunk to png 1.6 to avoid having to do this patch?"

[GitHub] oschaaf commented on issue #69: rename .src.rpm file to current file name.

2018-10-10 Thread GitBox
oschaaf commented on issue #69: rename .src.rpm file to current file name. URL: https://github.com/apache/incubator-pagespeed-cpanel/pull/69#issuecomment-428473680 That makes sense. Did things work out for you after making this change?

[GitHub] Lofesa edited a comment on issue #1801: image nor resized with page-speed

2018-10-02 Thread GitBox
Lofesa edited a comment on issue #1801: image nor resized with page-speed URL: https://github.com/apache/incubator-pagespeed-mod/issues/1801#issuecomment-426335311 These directives near the ModPagespeedSslCertDirectory "/opt/bitnami/common/openssl/certs" line.

[GitHub] milux closed issue #1799: Empty CSS files not removed by combine_css

2018-09-30 Thread GitBox
milux closed issue #1799: Empty CSS files not removed by combine_css URL: https://github.com/apache/incubator-pagespeed-mod/issues/1799 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] milux edited a comment on issue #1799: Empty CSS files not removed by combine_css

2018-09-30 Thread GitBox
milux edited a comment on issue #1799: Empty CSS files not removed by combine_css URL: https://github.com/apache/incubator-pagespeed-mod/issues/1799#issuecomment-425721034 Did so, but for whatever weird reason the issue has disappeared.  First found out that I can fix this

[GitHub] mattcheah commented on issue #69: rename .src.rpm file to current file name.

2018-10-10 Thread GitBox
mattcheah commented on issue #69: rename .src.rpm file to current file name. URL: https://github.com/apache/incubator-pagespeed-cpanel/pull/69#issuecomment-428592484 I actually had an issue with the `rpmbuild --rebuild ea-apache24-mod_pagespeed-[file].src.rpm` line, and while I tried to

[GitHub] oschaaf commented on issue #69: rename .src.rpm file to current file name.

2018-10-10 Thread GitBox
oschaaf commented on issue #69: rename .src.rpm file to current file name. URL: https://github.com/apache/incubator-pagespeed-cpanel/pull/69#issuecomment-428644027 Ok, in that case it probably doesn't make sense to merge this, right? Should we close this?

[GitHub] mattcheah commented on issue #69: rename .src.rpm file to current file name.

2018-10-10 Thread GitBox
mattcheah commented on issue #69: rename .src.rpm file to current file name. URL: https://github.com/apache/incubator-pagespeed-cpanel/pull/69#issuecomment-428644985 Not sure if other people will be able to get the instructions working, but they definitely won't if the URL is pointing

[GitHub] vitaliytv commented on issue #1599: Nginx Mainline update to 1.15.5

2018-10-11 Thread GitBox
vitaliytv commented on issue #1599: Nginx Mainline update to 1.15.5 URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1599#issuecomment-428858524 @nberlee when you plan update dockerhub, thank you? This is an

[GitHub] nberlee edited a comment on issue #1599: Nginx Mainline update to 1.15.5

2018-10-11 Thread GitBox
nberlee edited a comment on issue #1599: Nginx Mainline update to 1.15.5 URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1599#issuecomment-428863893 @vitaliytv It isn't possible in the dockerhub webgui to change the github source repository. I asked docker support to change

[GitHub] nberlee commented on issue #1599: Nginx Mainline update to 1.15.5

2018-10-11 Thread GitBox
nberlee commented on issue #1599: Nginx Mainline update to 1.15.5 URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1599#issuecomment-428863893 @etverdokhlib It isn't possible in the dockerhub webgui to change the github source repository. I asked docker support to change the

[GitHub] timeassistant commented on issue #1148: Support Brotli as an encoding scheme when client has Accept-Encoding

2018-10-11 Thread GitBox
timeassistant commented on issue #1148: Support Brotli as an encoding scheme when client has Accept-Encoding URL: https://github.com/apache/incubator-pagespeed-mod/issues/1148#issuecomment-428895199 @GuillaumeRossolini thanks, I already have the brotli module enabled. I was just curious

[GitHub] petecooper commented on issue #1587: Installation

2018-10-11 Thread GitBox
petecooper commented on issue #1587: Installation URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1587#issuecomment-428972945 What version of `gcc` are you using? I had the same error as you -- `checking for psol ... not found ./configure: error: module ngx_pagespeed

[GitHub] Lofesa commented on issue #1801: image nor resized with page-speed

2018-10-01 Thread GitBox
Lofesa commented on issue #1801: image nor resized with page-speed URL: https://github.com/apache/incubator-pagespeed-mod/issues/1801#issuecomment-425900453 Hi @HASSN237 Well, the problem can be a missconfiguration or some other causes. Can be a NON authorized domain, can be a non

[GitHub] biomedicus commented on issue #1013: PageSpeed 404s origin files with .pagespeed. in them.

2018-09-30 Thread GitBox
biomedicus commented on issue #1013: PageSpeed 404s origin files with .pagespeed. in them. URL: https://github.com/apache/incubator-pagespeed-mod/issues/1013#issuecomment-425728941 hi everyone, i have encountered the same error. every time i am new to this but every time i

[GitHub] jmarantz commented on issue #1013: PageSpeed 404s origin files with .pagespeed. in them.

2018-09-30 Thread GitBox
jmarantz commented on issue #1013: PageSpeed 404s origin files with .pagespeed. in them. URL: https://github.com/apache/incubator-pagespeed-mod/issues/1013#issuecomment-425729357 Awesome job finding the right issue to append this question. I think the answer is in the comment above with

[GitHub] Lofesa edited a comment on issue #1801: image nor resized with page-speed

2018-10-01 Thread GitBox
Lofesa edited a comment on issue #1801: image nor resized with page-speed URL: https://github.com/apache/incubator-pagespeed-mod/issues/1801#issuecomment-425900453 Hi @HASSN237 Well, the problem can be a missconfiguration or some other causes. Can be a NON authorized domain, can be a

[GitHub] oschaaf commented on issue #1598: Dockerfiles for Alpine 3.8/edge build with Nginx 1.14/1.15 (stable/ma…

2018-10-04 Thread GitBox
oschaaf commented on issue #1598: Dockerfiles for Alpine 3.8/edge build with Nginx 1.14/1.15 (stable/ma… URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1598#issuecomment-426954368 Maybe we should also link to the readme in

[GitHub] mattcheah opened a new pull request #69: rename .src.rpm file to current file name.

2018-10-09 Thread GitBox
mattcheah opened a new pull request #69: rename .src.rpm file to current file name. URL: https://github.com/apache/incubator-pagespeed-cpanel/pull/69 File in this repo: https://github.com/Prajithp/cpanel/raw/master/EA4/ is now named ea-apache24-mod_pagespeed-latest-stable.src.rpm

[GitHub] oschaaf closed pull request #1582: Fixed Spelling.

2018-09-02 Thread GitBox
oschaaf closed pull request #1582: Fixed Spelling. URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1582 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] Lofesa commented on issue #1792: CSS optimization breaks IE11, how to disable?

2018-09-03 Thread GitBox
Lofesa commented on issue #1792: CSS optimization breaks IE11, how to disable? URL: https://github.com/apache/incubator-pagespeed-mod/issues/1792#issuecomment-418081312 if ($http_user_agent ~* "^xxx$") { pagespeed DisableFilter combine_css; } where xxx is the user-agent

[GitHub] dvershinin opened a new issue #1593: missing-call-to-setgroups

2018-09-04 Thread GitBox
dvershinin opened a new issue #1593: missing-call-to-setgroups URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1593 Once an RPM package is built for the dynamic ngx_pagespeed module, checking it with`rpmlint` tool produce these points: nginx-module-nps.x86_64: W:

[GitHub] phillipseamore commented on issue #1592: Pagespeed conflicts with nchan module

2018-09-03 Thread GitBox
phillipseamore commented on issue #1592: Pagespeed conflicts with nchan module URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1592#issuecomment-418194898 The messages get sent to the server via Curl without a problem (that curl command outputs the server status back and I

[GitHub] Lofesa commented on issue #1795: Image not loading

2018-09-04 Thread GitBox
Lofesa commented on issue #1795: Image not loading URL: https://github.com/apache/incubator-pagespeed-mod/issues/1795#issuecomment-418287060 As far as I can see, some images are not from your domain, because that are not rewrited by pagespeed. Other imagein the ul.s are loaded from

[GitHub] bbhagan commented on issue #1221: extend_cache_images should add caching headers to SVG images

2018-09-20 Thread GitBox
bbhagan commented on issue #1221: extend_cache_images should add caching headers to SVG images URL: https://github.com/apache/incubator-pagespeed-mod/issues/1221#issuecomment-423312878 My client uses mps on all of their properties and relies on it heavily. SVG cache extension would

[GitHub] NisariAIT edited a comment on issue #1596: Pagespeed conflicts with Nginx Headers module

2018-09-19 Thread GitBox
NisariAIT edited a comment on issue #1596: Pagespeed conflicts with Nginx Headers module URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1596#issuecomment-422780488 @oschaaf , Compiling nginx with pagespeed as a dynamic module and using the Position_aux flag did not remove

[GitHub] NisariAIT edited a comment on issue #1596: Pagespeed conflicts with Nginx Headers module

2018-09-19 Thread GitBox
NisariAIT edited a comment on issue #1596: Pagespeed conflicts with Nginx Headers module URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1596#issuecomment-422780488 @oschaaf , Compiling nginx with pagespeed as a dynamic module and using the Position_aux flag did not remove

[GitHub] SilverFoxA opened a new issue #1597: NGINX pagespeed CloudFlare ssl gives “NET::ERR_CERT_AUTHORITY_INVALID”

2018-09-22 Thread GitBox
SilverFoxA opened a new issue #1597: NGINX pagespeed CloudFlare ssl gives “NET::ERR_CERT_AUTHORITY_INVALID” URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1597 I have rebuilt nginx with pagespeed running on ubuntu. Here's my config file. When accessing the website via port

[GitHub] SilverFoxA opened a new issue #1597: NGINX pagespeed CloudFlare ssl gives “NET::ERR_CERT_AUTHORITY_INVALID”

2018-09-22 Thread GitBox
SilverFoxA opened a new issue #1597: NGINX pagespeed CloudFlare ssl gives “NET::ERR_CERT_AUTHORITY_INVALID” URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1597 I have rebuilt nginx with pagespeed running on ubuntu. Here's my config file. When accessing the website via port

[GitHub] SilverFoxA closed issue #1597: NGINX pagespeed CloudFlare ssl gives “NET::ERR_CERT_AUTHORITY_INVALID”

2018-09-22 Thread GitBox
SilverFoxA closed issue #1597: NGINX pagespeed CloudFlare ssl gives “NET::ERR_CERT_AUTHORITY_INVALID” URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1597 This is an automated message from the Apache Git

[GitHub] znerd opened a new issue #1797: Document scope of TestingCoreFilters

2018-09-20 Thread GitBox
znerd opened a new issue #1797: Document scope of TestingCoreFilters URL: https://github.com/apache/incubator-pagespeed-mod/issues/1797 Please document what is in scope for the `testing` filter / `TestingCoreFilters`. Currently the documentation does describe what's in the `core`

[GitHub] znerd opened a new issue #1798: Filter not documented: defer_iframe

2018-09-20 Thread GitBox
znerd opened a new issue #1798: Filter not documented: defer_iframe URL: https://github.com/apache/incubator-pagespeed-mod/issues/1798 Please document the `defer_iframe` filter. The documentation page “Configuring PageSpeed Filters” at: -

[GitHub] oschaaf commented on issue #1800: Copyright headers are being wrongfully removed by rewrite_javascript

2018-09-27 Thread GitBox
oschaaf commented on issue #1800: Copyright headers are being wrongfully removed by rewrite_javascript URL: https://github.com/apache/incubator-pagespeed-mod/issues/1800#issuecomment-425035872 See `ModPagespeedRetainComment` over at https://www.modpagespeed.com/doc/filter-comment-remove

[GitHub] N0ps32 commented on issue #1800: Copyright headers are being wrongfully removed by rewrite_javascript

2018-09-27 Thread GitBox
N0ps32 commented on issue #1800: Copyright headers are being wrongfully removed by rewrite_javascript URL: https://github.com/apache/incubator-pagespeed-mod/issues/1800#issuecomment-425037326 I just found this which I think is related to this issue:

[GitHub] jmarantz closed issue #964: insert_dns_preftech not working

2018-09-28 Thread GitBox
jmarantz closed issue #964: insert_dns_preftech not working URL: https://github.com/apache/incubator-pagespeed-mod/issues/964 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] jmarantz edited a comment on issue #964: insert_dns_preftech not working

2018-09-28 Thread GitBox
jmarantz edited a comment on issue #964: insert_dns_preftech not working URL: https://github.com/apache/incubator-pagespeed-mod/issues/964#issuecomment-425461122 I just tried refreshing https://www.modpagespeed.com/examples/insert_dns_prefetch.html?PageSpeed=on=insert_dns_prefetch a

[GitHub] jmarantz commented on issue #1799: Empty CSS files not removed by combine_css

2018-09-28 Thread GitBox
jmarantz commented on issue #1799: Empty CSS files not removed by combine_css URL: https://github.com/apache/incubator-pagespeed-mod/issues/1799#issuecomment-425411043 Can you try enabling the 'debug' filter and looking at the HTML again? That filter annotates some of the decisions

[GitHub] lamariabdo opened a new issue #1612: install ngx_pagespeed dynamic with centos-webpanel

2019-01-06 Thread GitBox
lamariabdo opened a new issue #1612: install ngx_pagespeed dynamic with centos-webpanel URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1612 Hello Please how to install ngx_pagespeed dynamic module with centos-webpanel nginx already installed in **usr/local/nginx/**

[GitHub] nberlee opened a new pull request #1852: update libwebp to 1.0.1

2019-01-06 Thread GitBox
nberlee opened a new pull request #1852: update libwebp to 1.0.1 URL: https://github.com/apache/incubator-pagespeed-mod/pull/1852 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] nberlee opened a new pull request #1853: WIP: Update grpc to v1.17.2 to make grpc OpenSSL 1.1 compatible

2019-01-06 Thread GitBox
nberlee opened a new pull request #1853: WIP: Update grpc to v1.17.2 to make grpc OpenSSL 1.1 compatible URL: https://github.com/apache/incubator-pagespeed-mod/pull/1853 DO NOT MERGE YET In order to have OpenSSL compatibility we need to update grpc to a > 1.16.0 version. (also need to

[GitHub] ddrager commented on issue #438: ngx_pagespeed causes nginx to stop responding to requests

2019-01-16 Thread GitBox
ddrager commented on issue #438: ngx_pagespeed causes nginx to stop responding to requests URL: https://github.com/apache/incubator-pagespeed-ngx/issues/438#issuecomment-454790975 Closing out old issue. This is an

[GitHub] ZTzha closed issue #1621: rewrite css image but get 404

2019-01-16 Thread GitBox
ZTzha closed issue #1621: rewrite css image but get 404 URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1621 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] Lofesa commented on issue #1621: rewrite css image but get 404

2019-01-17 Thread GitBox
Lofesa commented on issue #1621: rewrite css image but get 404 URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1621#issuecomment-455138100 Is css.j.cc.cn your domain?, most relevant, is pagespeed module runing in that domain?. If you have pagespeed Domain

[GitHub] oschaaf commented on issue #1620: can i add some infomation to pagespeed outputs name?

2019-01-14 Thread GitBox
oschaaf commented on issue #1620: can i add some infomation to pagespeed outputs name? URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1620#issuecomment-454049927 resource_namer.cc lives here:

[GitHub] oschaaf commented on issue #1617: Update Nginx in Docker

2019-01-14 Thread GitBox
oschaaf commented on issue #1617: Update Nginx in Docker URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1617#issuecomment-454045485 lgtm, but let's see if @nberlee has any remarks This is an automated message

[GitHub] oschaaf commented on issue #1853: WIP: Update grpc to v1.17.2 to make grpc OpenSSL 1.1 compatible

2019-01-14 Thread GitBox
oschaaf commented on issue #1853: WIP: Update grpc to v1.17.2 to make grpc OpenSSL 1.1 compatible URL: https://github.com/apache/incubator-pagespeed-mod/pull/1853#issuecomment-454050536 @nberlee let me know if you need any help with this

<    4   5   6   7   8   9   10   11   12   13   >