Skip to content
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

Dockerize project #2

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Dockerize project #2

wants to merge 8 commits into from

Conversation

twilsonco
Copy link

@twilsonco twilsonco commented Dec 3, 2024

Hi, I love your project here! I wanted to run it more easily, so I dockerized it.

  • Easy setup as a docker container with built-in apache web server
    • Using multistage Docker build so that the resulting image is as slim as can be
    • There are two Dockerfiles, the default one for modern Linux, and one for old kernels (I'm using it on my Synology NAS which runs Linux kernel v3.10)
  • Can still run in non-Docker mode (not tested)
  • All configuration is now done via the config.json file. No more edits to source code necessary
  • Real-time checking of changes to config.json file so you can make adjustments without needing to restart manually
  • Updated readme and setup files
  • Resends status messages to Discord channel if they're deleted
  • Some additional error handling added in the process that I decided to leave in there

(js and php)
relocate config to common location
and last cleanup of debug statements in source
and last cleanup of debug statements in source
mount index.js for rapid development
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant