Giter Site home page Giter Site logo

Comments (5)

hieuhtr avatar hieuhtr commented on August 12, 2024 2

Hi guys, sorry for late update. I tried several ways but there is no hope on latest codebase.

I reverted build step by this commit hash 6e975bcb015f62e1f303054897783355e2a877dc thanks @NicolasCARPi for recommendation.

Here is my simple solution.

FROM debian:bullseye-slim AS builder

ARG NGINX_VERSION=1.24.0
ARG MOD_SECURITY_VERSION=3.0.8

# https://github.com/google/ngx_brotli/issues/166
ENV NGX_BROTLI_COMMIT_HASH=6e975bcb015f62e1f303054897783355e2a877dc

WORKDIR /root/

# Download nginx source
RUN wget http://nginx.org/download/nginx-${NGINX_VERSION}.tar.gz \
    && tar zxf nginx-${NGINX_VERSION}.tar.gz

# Install & Compile dynamic module Brotli
RUN git clone https://github.com/google/ngx_brotli \
    && cd ngx_brotli && git reset --hard $NGX_BROTLI_COMMIT_HASH \
    && git submodule update --init \
    && cd ../nginx-${NGINX_VERSION} \
    && ./configure --with-compat --add-dynamic-module=../ngx_brotli \
    && make modules

...

from ngx_brotli.

simneu avatar simneu commented on August 12, 2024 1

hi @hieuhtr,

I'm facing similar issues with a recent (but new, we didn't use brotli before) release. Did you find a resolution yet?

On my side:

  • exit on signal 4 is happening every time we have brotli on in our configuration and nginx tries to serve a br compressed page. Removing brotli on from config fixes the issue.
  • setup is using nginx:1.18.0 and ngx_brotli module was compiled and then packed into an AWS AMI, than we then are trying to use on our final instances. The instance building the AWS AMI and the actual end instance using the AMI are both amd64 (if that can hint to a particular direction, cause I thought this could be a module compilation issue.
  • I tried debugging (with limited knowledge) what causes the signal 4, but couldn't find any hints yet.

Thanks.

from ngx_brotli.

NicolasCARPi avatar NicolasCARPi commented on August 12, 2024 1

Had the same issue (elabftw/elabftw#4600). Had to revert to using commit 6e975bc because newer version cause this crash. Hope it can be adressed, or we'll all be stuck to this working commit ;)

It's with nginx 1.24. Full Dockerfile is available here: https://github.com/elabftw/elabimg/blob/master/Dockerfile

from ngx_brotli.

simneu avatar simneu commented on August 12, 2024

Awesome @NicolasCARPi, thanks for taking the time to help. 👍

I'll try commit you suggested when I get few minutes, and will update here with news.

from ngx_brotli.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.