Skip to content

Conversation

@Salamandar
Copy link

I thought a docker-compose file would be useful because daemon services usually are preferred for server installs :)

Copy link

@ScottBrenner ScottBrenner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -0,0 +1,15 @@
version: "3.6"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually that change doesn't work for me (Debian stable) :

ERROR: Version in "./docker-compose.yml" is unsupported. You might be seeing this error because you're using the wrong Compose file version. Either specify a supported version (e.g "2.2" or "3.3") and place your service definitions under the `services` key, or omit the `version` key and place your service definitions at the root of the file to use version 1.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverting should be fine to ensure compatibility. I don't think this compose file uses anything specific to 3.8

Salamandar and others added 2 commits December 14, 2020 12:47
@Salamandar
Copy link
Author

Readme was edited !

@jordy-u
Copy link

jordy-u commented Dec 20, 2020

Nice work. Can you add port 443 (as a comment) to the exposed ports? This makes it easier for the people who want to use HTTPS.

@Salamandar
Copy link
Author

Nice work. Can you add port 443 (as a comment) to the exposed ports? This makes it easier for the people who want to use HTTPS.

Oh, I thought HTTPS was listening on 9736 too !

@heyimblake
Copy link

Great work, used this as a guide for hosting my own instance :)

This pull request was closed.
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.

4 participants