Skip to content
This repository has been archived by the owner on Sep 3, 2019. It is now read-only.

Javascript not fully loading #7

Closed
eleshar opened this issue Feb 16, 2018 · 6 comments
Closed

Javascript not fully loading #7

eleshar opened this issue Feb 16, 2018 · 6 comments

Comments

@eleshar
Copy link

eleshar commented Feb 16, 2018

Hi!

After upgrading to latest build, we are seeing issues with Javascript files not fully loading in WordPress dashboard: Uncaught SyntaxError: Unexpected end of input

Testing from Chrome 64.0.3282.167 but users on Firefox and Safari also reported it.

Build was done on Nginx 1.13.8:

./configure --with-ld-opt='-ljemalloc -Wl,-z,relro -Wl,-rpath,/usr/local/lib' --with-cc-opt='-DTCP_FASTOPEN=23 -m64 -march=x86-64 -g -O3 -fstack-protector-strong -flto -fuse-ld=gold --param=ssp-buffer-size=4 -Wformat -Werror=format-security -Wimplicit-fallthrough=0 -fcode-hoisting -Wno-cast-function-type -Wp,-D_FORTIFY_SOURCE=2' --prefix=/usr/share --sbin-path=/usr/sbin/nginx --conf-path=/etc/nginx/nginx.conf --error-log-path=/var/log/nginx/error.log --http-log-path=/var/log/nginx/access.log --lock-path=/var/lock/nginx.lock --pid-path=/var/run/nginx.pid --http-client-body-temp-path=/var/lib/nginx/body --http-fastcgi-temp-path=/var/lib/nginx/fastcgi --http-proxy-temp-path=/var/lib/nginx/proxy --http-scgi-temp-path=/var/lib/nginx/scgi --http-uwsgi-temp-path=/var/lib/nginx/uwsgi --user=nginx --group=nginx --with-file-aio --with-http_ssl_module --with-http_realip_module --with-http_sub_module --with-http_dav_module --with-http_gzip_static_module --with-http_stub_status_module --with-http_v2_module --with-pcre=/root/sources/pcre-8.41 --with-pcre-jit --add-module=/root/sources/ngx_brotli

Please let me know should you require any additional information.

@eustas
Copy link
Owner

eustas commented Feb 17, 2018

Hello, Chris.

Does "latest build" means ngx_brotli @ 8cd9dd? Does it work normally with ngx_brotli @ 47550a?

@eleshar
Copy link
Author

eleshar commented Feb 17, 2018

Thank you for the reply. I seem to have missed a few commits.

I've gone through them all from latest backwards. I'm not seeing the issue on ngx_brotli @ bfd2885.

@eustas
Copy link
Owner

eustas commented Feb 17, 2018

Similar problem was fixed in <Fix "flush" buffer swapping> (47550a2), so I wonder if it is the same problem... If not, going to try do dig it out next week.

@eustas
Copy link
Owner

eustas commented Mar 7, 2018

One more problem (that, likely caused the issue you describe) is solved:
https://github.com/eustas/ngx_brotli/tree/rewrite

It would be nice, if you check if it fixes your problem (and does not cause new issues).

@eleshar
Copy link
Author

eleshar commented Mar 7, 2018

@eustas Excellent, thank you! Will test later tonight.

@eustas
Copy link
Owner

eustas commented Mar 21, 2018

Should be fixed after #10. If not, feel free to reopen the issue.

@eustas eustas closed this as completed Mar 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants