-
Notifications
You must be signed in to change notification settings - Fork 104
Fixes default authentication in docker-compose.yml #319
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR attempts to fix authentication issues in the docker-compose.yml configuration by modifying the volume mount path for the authentication database file.
- Changes the commented volume mount from
/cwa/config/path/app.db:/auth/app.db:roto/cwa/config/path/app.db:/auth:ro
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # This is the location of CWA's app.db, which contains authentication | ||
| # details. Comment out to disable authentication | ||
| #- /cwa/config/path/app.db:/auth/app.db:ro | ||
| #- /cwa/config/path/app.db:/auth:ro |
Copilot
AI
Nov 19, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The volume mount path change is inconsistent with the CWA_DB_PATH environment variable. Line 17 sets CWA_DB_PATH: /auth/app.db, but this change mounts the database file to /auth instead of /auth/app.db. This will cause the error "CWA_DB_PATH is set to /auth/app.db but this is not a valid path" because the file won't be found at the expected location.
Either:
- Keep the volume mount as
/cwa/config/path/app.db:/auth/app.db:ro(revert this change), OR - Update line 17 to
CWA_DB_PATH: /authto match this volume mount change
| #- /cwa/config/path/app.db:/auth:ro | |
| #- /cwa/config/path.app.db:/auth.app.db:ro |
|
Could you share what is your current docker compose looks like ? |
|
@calibrain Here you go services:
calibre-web-automated-book-downloader:
image: ghcr.io/calibrain/calibre-web-automated-book-downloader:latest
# Uncomment to build the image from the Dockerfile for local testing changes.
# Remember to comment out the image line above.
#build: .
container_name: calibre-web-automated-book-downloader
environment:
FLASK_PORT: 8084
LOG_LEVEL: info
BOOK_LANGUAGE: en
USE_BOOK_TITLE: true
TZ: America/New_York
APP_ENV: prod
UID: 1000
GID: 100
CWA_DB_PATH: /auth/app.db # Comment out to disable authentication
# Queue management settings
MAX_CONCURRENT_DOWNLOADS: 3
DOWNLOAD_PROGRESS_UPDATE_INTERVAL: 5
ports:
- 8084:8084
restart: unless-stopped
volumes:
# This is where the books will be downloaded to, usually it would be
# the same as whatever you gave in "calibre-web-automated"
- /tmp/data/calibre-web/ingest:/cwa-book-ingest
# This is the location of CWA's app.db, which contains authentication
# details. Comment out to disable authentication
- /cwa/config/path/app.db:/auth:roIt's the same as the default, but with the authentication enabled & with the one line change. |
The current docker compose does not allow for authentication & provides a less than clear error (
CWA_DB_PATH is set to /auth/app.db but this is not a valid path).