PhilippMundhenk / BrotherScannerDocker

Dockerized Brother Scanner driver
https://www.mundhenk.org
MIT License
54 stars 29 forks source link

Web server non-functional, no logs #55

Open pedropombeiro opened 2 hours ago

pedropombeiro commented 2 hours ago

I decided to give the webserver a try, but it is not triggering scans for me. I even tried running the curl command inside the container, but still no results, and nothing logged:

image

Any ideas what I could look for?

Here's my docker-compose config:

  brother-scanner:
    image: ghcr.io/philippmundhenk/brotherscannerdocker:pedropombeiro-convert-to-python3
    extends:
      file: ../common-services.yml
      service: baseapp
    container_name: docs_brother_scanner
    volumes:
      - ${paperlessConsumeDir}:/scans/
      - ../../log/brotherscanner/:/var/log/
    ports:
      - 33355:33355
      - 54925:54925/udp # Scan Key Tool UDP: mandatory, for scanner tools
      - 54921:54921 # Scan Key Tool TCP: mandatory, for scanner tools
      - 161:161/udp # SNMP: mandatory, for scanner tools
    environment:
      - NAME=Paperless
      - MODEL=MFC-J4340DW
      - IPADDRESS=192.168.46.12
      - UID=500 # note: network mount needs to have correct permissions!
      - GID=101 # note: network mount needs to have correct permissions!
      - HOST_IPADDRESS=192.168.96.10
      - RESOLUTION=300
      - REMOVE_BLANK_THRESHOLD=0.55

      - WEBSERVER=true # optional, activates GUI & API
      - PORT=33355 # optional, sets port for webserver (default: 80)
      - DISABLE_GUI_SCANTOIMAGE=true
      - RENAME_GUI_SCANTOFILE="Scan front pages"
      - RENAME_GUI_SCANTOOCR="Scan front pages (B&W)"
      - RENAME_GUI_SCANTOEMAIL="Scan rear pages"
    restart: unless-stopped
pedropombeiro commented 2 hours ago

Looks like the problem is in /etc/sudoers, it lists the NAS user, instead of the user in NAME. When I changed it with visudo to use the user listed in NAME, then web UI worked straight away.

PhilippMundhenk commented 2 hours ago

I think that may be fixed in #32. I missed that when fixing username back then.