xavier-hernandez / goaccess-for-nginxproxymanager

GoAccess Docker Image for Nginx Proxy Manager and more...
https://hub.docker.com/r/xavierh/goaccess-for-nginxproxymanager
MIT License
523 stars 39 forks source link

Stop working on parse errors #156

Closed webysther closed 1 year ago

webysther commented 1 year ago

Hi @xavier-hernandez, I have the same container (latest) a few while now and everything works just fine, but for some reason after redeploy last week to change the healthcheck in my container it stop working, for me is strange because get errors in same old log with same container:

==409== GoAccess - version 1.7.2 - Jul  3 2023 01:17:55
==409== Config file: /goaccess-config/goaccess.conf
==409== https://goaccess.io - <hello@goaccess.io>
==409== Released under the MIT License.
==409== FILE: /goaccess-logs/archives/proxy-host-24_access.log.4
==409== Parsed 1 lines producing the following errors:
==409==
==409== Token '-' doesn't match specifier '%m'

It still in the same error without stop and dont move to new file, this one for example have only 20KB.

version: '3.9'

services:
  goaccess-all: &goaccess
    image: xavierh/goaccess-for-nginxproxymanager:v1.1.18
    container_name: app-analytics-log-all
    env_file: ../goaccess/.env
    cpuset: '0'
    networks:
      goaccess:
    expose:
      - 7880
    volumes:
      - "$LOG/nginx-proxy-manager:/opt/log:ro"
    tmpfs:
      - /opt/custom
    healthcheck:
      test: [
        "CMD", "wget", "--spider", "-q", 
        "localhost:7880"
      ]
      interval: 300s
      retries: 5
      timeout: 10s
    restart: unless-stopped
PUID=1026
PGID=101
TZ=Europe/Rome
LOG=/volumeUSB2/usbshare/cache/log/docker
LOG_TYPE=NPM

As you can see, the cpu still high for while but stop, the loop still going...

image

log file

Do you have any clue?

webysther commented 1 year ago

Looks like in some cases the method don't appers in NPM:

[11/Jul/2023:20:10:05 +0200] - - 400 - - https auth.webysther.org "-" [Client 95.214.26.129] [Length 0] [Gzip -] [Sent-to ] "-" "-"

Maybe add --num-test=0 as a option as here: https://github.com/allinurl/goaccess/issues/891

xavier-hernandez commented 1 year ago

@webysther nice catch. I'll merge into develop then main.

xavier-hernandez commented 1 year ago

This has been deployed.