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
Fix: log error when getting services from Docker server fails #3147
Conversation
Not raising an error when a server fails to get services from Docker could lead to confusion when the server does not appear to be functioning correctly. This commit adds an error log when a server fails to get services from Docker.
See the Discord discussion for details. TLDR: My homepage failed to resolve any Docker services. After some debugging I found out that a silent error was thrown, because homepage couldn't access the secrets. |
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.
Thats fine. I cant remember if there isnt logging there for a reason but seems reasonable thanks
…0.8.10@fc0d6e8 by renovate (#19733) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ghcr.io/gethomepage/homepage](https://togithub.com/gethomepage/homepage) | patch | `v0.8.9` -> `v0.8.10` | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>gethomepage/homepage (ghcr.io/gethomepage/homepage)</summary> ### [`v0.8.10`](https://togithub.com/gethomepage/homepage/releases/tag/v0.8.10) [Compare Source](https://togithub.com/gethomepage/homepage/compare/v0.8.9...v0.8.10) #### What's Changed - Fix: field parsing can fail with docker labels by [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3101 - Fix: Add alternative 'offline' status to EspHome widget by [@​RoboMagus](https://togithub.com/RoboMagus) in [gethomepage/homepage#3107 - Fix: log error when getting services from Docker server fails by [@​she11sh0cked](https://togithub.com/she11sh0cked) in [gethomepage/homepage#3147 - Fix: Glances cpu & info widgets information by [@​sunnycloudy1337](https://togithub.com/sunnycloudy1337) and [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3164 - Enhancement: support Jackett widget with admin password by [@​technowhizz](https://togithub.com/technowhizz) and [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3165 - Fix: correctly handle direct tab navigation with encoded chars by [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3172 #### New Contributors - [@​she11sh0cked](https://togithub.com/she11sh0cked) made their first contribution in [gethomepage/homepage#3147 - [@​sunnycloudy1337](https://togithub.com/sunnycloudy1337) made their first contribution in [gethomepage/homepage#3164 - [@​technowhizz](https://togithub.com/technowhizz) made their first contribution in [gethomepage/homepage#3165 **Full Changelog**: gethomepage/homepage@v0.8.9...v0.8.10 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://togithub.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yNjkuMiIsInVwZGF0ZWRJblZlciI6IjM3LjI2OS4yIiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIn0=-->
…v0.8.10 ) (#3396) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ghcr.io/gethomepage/homepage](https://togithub.com/gethomepage/homepage) | patch | `v0.8.9` -> `v0.8.10` | --- ### Release Notes <details> <summary>gethomepage/homepage (ghcr.io/gethomepage/homepage)</summary> ### [`v0.8.10`](https://togithub.com/gethomepage/homepage/releases/tag/v0.8.10) [Compare Source](https://togithub.com/gethomepage/homepage/compare/v0.8.9...v0.8.10) #### What's Changed - Fix: field parsing can fail with docker labels by [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3101 - Fix: Add alternative 'offline' status to EspHome widget by [@​RoboMagus](https://togithub.com/RoboMagus) in [gethomepage/homepage#3107 - Fix: log error when getting services from Docker server fails by [@​she11sh0cked](https://togithub.com/she11sh0cked) in [gethomepage/homepage#3147 - Fix: Glances cpu & info widgets information by [@​sunnycloudy1337](https://togithub.com/sunnycloudy1337) and [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3164 - Enhancement: support Jackett widget with admin password by [@​technowhizz](https://togithub.com/technowhizz) and [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3165 - Fix: correctly handle direct tab navigation with encoded chars by [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3172 #### New Contributors - [@​she11sh0cked](https://togithub.com/she11sh0cked) made their first contribution in [gethomepage/homepage#3147 - [@​sunnycloudy1337](https://togithub.com/sunnycloudy1337) made their first contribution in [gethomepage/homepage#3164 - [@​technowhizz](https://togithub.com/technowhizz) made their first contribution in [gethomepage/homepage#3165 **Full Changelog**: gethomepage/homepage@v0.8.9...v0.8.10 </details> <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yODAuMCIsInVwZGF0ZWRJblZlciI6IjM3LjI4MC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9jb250YWluZXIiLCJ0eXBlL3BhdGNoIl19--> Co-authored-by: repo-jeeves[bot] <106431701+repo-jeeves[bot]@users.noreply.github.com>
…v0.8.10 ) (#3396) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ghcr.io/gethomepage/homepage](https://togithub.com/gethomepage/homepage) | patch | `v0.8.9` -> `v0.8.10` | --- ### Release Notes <details> <summary>gethomepage/homepage (ghcr.io/gethomepage/homepage)</summary> ### [`v0.8.10`](https://togithub.com/gethomepage/homepage/releases/tag/v0.8.10) [Compare Source](https://togithub.com/gethomepage/homepage/compare/v0.8.9...v0.8.10) #### What's Changed - Fix: field parsing can fail with docker labels by [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3101 - Fix: Add alternative 'offline' status to EspHome widget by [@​RoboMagus](https://togithub.com/RoboMagus) in [gethomepage/homepage#3107 - Fix: log error when getting services from Docker server fails by [@​she11sh0cked](https://togithub.com/she11sh0cked) in [gethomepage/homepage#3147 - Fix: Glances cpu & info widgets information by [@​sunnycloudy1337](https://togithub.com/sunnycloudy1337) and [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3164 - Enhancement: support Jackett widget with admin password by [@​technowhizz](https://togithub.com/technowhizz) and [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3165 - Fix: correctly handle direct tab navigation with encoded chars by [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3172 #### New Contributors - [@​she11sh0cked](https://togithub.com/she11sh0cked) made their first contribution in [gethomepage/homepage#3147 - [@​sunnycloudy1337](https://togithub.com/sunnycloudy1337) made their first contribution in [gethomepage/homepage#3164 - [@​technowhizz](https://togithub.com/technowhizz) made their first contribution in [gethomepage/homepage#3165 **Full Changelog**: gethomepage/homepage@v0.8.9...v0.8.10 </details> <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yODAuMCIsInVwZGF0ZWRJblZlciI6IjM3LjI4MC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9jb250YWluZXIiLCJ0eXBlL3BhdGNoIl19--> Co-authored-by: repo-jeeves[bot] <106431701+repo-jeeves[bot]@users.noreply.github.com>
…v0.8.10 ) (#3396) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ghcr.io/gethomepage/homepage](https://togithub.com/gethomepage/homepage) | patch | `v0.8.9` -> `v0.8.10` | --- ### Release Notes <details> <summary>gethomepage/homepage (ghcr.io/gethomepage/homepage)</summary> ### [`v0.8.10`](https://togithub.com/gethomepage/homepage/releases/tag/v0.8.10) [Compare Source](https://togithub.com/gethomepage/homepage/compare/v0.8.9...v0.8.10) #### What's Changed - Fix: field parsing can fail with docker labels by [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3101 - Fix: Add alternative 'offline' status to EspHome widget by [@​RoboMagus](https://togithub.com/RoboMagus) in [gethomepage/homepage#3107 - Fix: log error when getting services from Docker server fails by [@​she11sh0cked](https://togithub.com/she11sh0cked) in [gethomepage/homepage#3147 - Fix: Glances cpu & info widgets information by [@​sunnycloudy1337](https://togithub.com/sunnycloudy1337) and [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3164 - Enhancement: support Jackett widget with admin password by [@​technowhizz](https://togithub.com/technowhizz) and [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3165 - Fix: correctly handle direct tab navigation with encoded chars by [@​shamoon](https://togithub.com/shamoon) in [gethomepage/homepage#3172 #### New Contributors - [@​she11sh0cked](https://togithub.com/she11sh0cked) made their first contribution in [gethomepage/homepage#3147 - [@​sunnycloudy1337](https://togithub.com/sunnycloudy1337) made their first contribution in [gethomepage/homepage#3164 - [@​technowhizz](https://togithub.com/technowhizz) made their first contribution in [gethomepage/homepage#3165 **Full Changelog**: gethomepage/homepage@v0.8.9...v0.8.10 </details> <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yODAuMCIsInVwZGF0ZWRJblZlciI6IjM3LjI4MC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9jb250YWluZXIiLCJ0eXBlL3BhdGNoIl19--> Co-authored-by: repo-jeeves[bot] <106431701+repo-jeeves[bot]@users.noreply.github.com>
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion for related concerns. See our contributing guidelines for more details. |
Proposed change
This PR addresses an issue where a server fails to retrieve services from Docker without raising an error, potentially causing confusion regarding server functionality. The fix adds an error log to provide clarity in such scenarios.
Type of change
Checklist: