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

Updating Docs - configuration table, shoutrr, Scrutiny, etc #1161

Merged
merged 5 commits into from
Sep 29, 2024

Conversation

OneWeekNotice
Copy link
Contributor

@OneWeekNotice OneWeekNotice commented Jul 27, 2024

Summary

  • removing WATCHTOWER_NOTIFICATIONS as this is legacy notifications. Reference watchtower legacy notifications documentation
  • adding Watchtower (shoutrrr) token authentication URL
  • adding logging configurations to to config option table
  • adding Scrutiny project to integration documentation. Scrutiny is a webUI for smartd S.M.A.R.T monitoring. This includes notifications through shoutrrr.
  • adding note about caddy. for beginners to caddy they should know they don't need additional configuration for WebSockets as caddy handles this (along with HTTP) with reverse_proxy

@@ -777,6 +777,7 @@ or the root domain:
```
# Note that this config is most certainly incomplete. Please help out and let me know what's missing
# via Discord/Matrix or in a GitHub issue.
# Note: Caddy automatically handles both HTTP and WebSockets with reverse_proxy
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Upon request I can supply a docker compose file with caddy and ntfy. But currently I am not sure if this adds any value (looking for confirmation)

  • adding a docker compose for caddy can get stale
  • caddy file will display caddy docker integration (which may not provide any value for the documentation of this project)
    • below is a sample where ntfy is the docker container name and 80 is the default ntfy docker port (not machine port but docker container port)
ntfy.sh, http://nfty.sh {
      reverse_proxy ntfy:80
}

@binwiederhier binwiederhier merged commit 81f05b3 into binwiederhier:main Sep 29, 2024
@binwiederhier
Copy link
Owner

Thank you

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.

2 participants