Skip to content

Fix Docker Traefik .yml and SERVER_DNS#7858

Merged
Alkarex merged 4 commits intoFreshRSS:edgefrom
ejohnmarlo:edge
Aug 27, 2025
Merged

Fix Docker Traefik .yml and SERVER_DNS#7858
Alkarex merged 4 commits intoFreshRSS:edgefrom
ejohnmarlo:edge

Conversation

@ejohnmarlo
Copy link
Contributor

Closes #

Changes proposed in this pull request:

  • Rename tls.yml to tls.yaml so it will be align to the entry in docker-compose-proxy.yml:

    volumes:

    • ./tls.yaml:/etc/traefik/tls.yaml

    To prevent error:
    error="error reading configuration file: /etc/traefik/tls.yaml - read /etc/traefik/tls.yaml: is a directory"

  • Added parameter SERVER_DNS in the example.env

    To prevent warning:
    WARN[0000] The "SERVER_DNS" variable is not set. Defaulting to a blank string.

How to test the feature manually:

  1. Copy example.env to .env
  2. Run: docker compose -f docker-compose.yml -f docker-compose-proxy.yml -f docker-compose-db.yml up
  3. There should be no warning and error for the tls.yaml and SERVER_DNS

Pull request checklist:

  • clear commit messages
  • code manually tested
  • unit tests written (optional if too hard)
  • documentation updated

Additional information can be found in the documentation.

Rename tls.yaml so it will be align to the entry in docker-compose-proxy.yml:

volumes:
  - ./tls.yaml:/etc/traefik/tls.yaml

To prevent error:
error="error reading configuration file: /etc/traefik/tls.yaml - read /etc/traefik/tls.yaml: is a directory"
Added option for SERVER_URL

To prevent warning:
WARN[0000] The "SERVER_DNS" variable is not set. Defaulting to a blank string.
Change SERVER_URL to SERVER_DNS
@Alkarex Alkarex added this to the 1.28.0 milestone Aug 26, 2025
@Alkarex Alkarex added the Docker 🐋 Everything related to Docker label Aug 26, 2025
@Alkarex
Copy link
Member

Alkarex commented Aug 26, 2025

Thanks for the heads up 👍🏻
I have taken the other approach to consistently using .yml, as it looks like .yml is more common for Traefik
(see e.g. https://doc.traefik.io/traefik/getting-started/configuration-overview/ )
Could you please check whether that still works for you?

@Alkarex Alkarex changed the title Rename tls.yml to tls.yaml so it will be align to the entry in docker-compose-proxy.yml, added parameter SERVER_DNS in the example.env Fix Docker Traefik .yml and SERVER_DNS Aug 26, 2025
@Alkarex Alkarex merged commit 70f0d6d into FreshRSS:edge Aug 27, 2025
1 check passed
@ejohnmarlo
Copy link
Contributor Author

Thanks for the heads up 👍🏻 I have taken the other approach to consistently using .yml, as it looks like .yml is more common for Traefik (see e.g. https://doc.traefik.io/traefik/getting-started/configuration-overview/ ) Could you please check whether that still works for you?

I tested the approach of consistently using .yml, and it works without any issues on my end.

@Alkarex Alkarex modified the milestones: 1.28.0, 1.27.1 Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Docker 🐋 Everything related to Docker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants