Want more timely Centmin Mod News Updates?
Become a Member

Beta Branch update nginx.conf add additional log_format templates

Discussion in 'Centmin Mod Github Commits' started by eva2000, Oct 29, 2018.

  1. eva2000

    eva2000 Administrator Staff Member

    59,284
    12,509
    113
    May 24, 2014
    Brisbane, Australia
    Ratings:
    +19,141
    Local Time:
    1:46 PM
    Nginx 1.31.x
    MariaDB 10.x/11.4+/12.3+
  2. rdan

    rdan Well-Known Member

    5,452
    1,418
    113
    May 25, 2014
    Ratings:
    +2,212
    Local Time:
    11:46 AM
    Mainline
    10.2
    Is log_format ddos-proxy still needed?
    For what purpose?
    Sucuri blocking?
     
  3. eva2000

    eva2000 Administrator Staff Member

    59,284
    12,509
    113
    May 24, 2014
    Brisbane, Australia
    Ratings:
    +19,141
    Local Time:
    1:46 PM
    Nginx 1.31.x
    MariaDB 10.x/11.4+/12.3+
    yeah it was for when you have a reverse proxy in front of centmin mod nginx it isn't used unless access_log is defined to use log_format = ddos-proxy so generally no harm it having it in nginx.conf

    though did come across a bug in this commit, which is fixed here Beta Branch - fix recently updated log_format in 123.09beta01

    the new log_format main_extssl referenced $brotli_ratio variable which is unknown to nginx by default as ngx_brotli isn't enabled by default. This fix at Beta Branch - fix recently updated log_format in 123.09beta01 corrects this for new installs of 123.09beta01.

    Existing 123.09beta01 are not affected as this additional log_format named main_extssl isn't auto updated in existing server's /usr/local/nginx/conf/nginx.conf