This repository was archived by the owner on Jul 12, 2026. It is now read-only.
Add healthcheck api - #1
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds a comprehensive health check API to the DDNS Hetzner service and enhances the CI/CD pipeline to support pull requests, providing better monitoring capabilities and development workflow.
- Introduces health check API with multiple endpoints for monitoring service status and health
- Enhances CI/CD workflows to build and test on pull requests without releasing
- Updates documentation with detailed health check usage examples and configuration
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Adds health check API documentation and fixes project name inconsistencies |
| HealthService.cs | New service class for tracking application health status and metrics |
| DdnsUpdate.cs | Integrates health service and implements HTTP health check API endpoints |
| .github/workflows/release.yml | Adds PR build support with artifact retention policies |
| .github/workflows/docker-build-publish.yml | Adds PR Docker build validation without publishing |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
visviva
force-pushed
the
add-healthcheck-api
branch
from
September 19, 2025 17:12
5a8accd to
55ad700
Compare
…d adjust push conditions
… releases for PRs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- Fix HttpListener to properly bind to all interfaces using '*' prefix when HEALTH_BIND_ADDRESS is 0.0.0.0 or * - Update README with correct Docker configuration examples - Add HEALTH_BIND_ADDRESS environment variable documentation
visviva
force-pushed
the
add-healthcheck-api
branch
from
September 19, 2025 18:44
55ad700 to
2dffa67
Compare
…nd ensure default bind address is set
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds a comprehensive health check API to the DDNS Hetzner service and enhances the CI/CD pipeline to support pull requests, providing better monitoring capabilities and development workflow.