Stop FreshRSS traffic from health probes #35
Loading…
Reference in a new issue
No description provided.
Delete branch "fix/rss-health-cache"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
The RSS module health endpoint performed three authenticated FreshRSS requests on every health probe. With frequent Body and container probes, this created unnecessary DNS traffic, especially AAAA lookups against the IPv4-only AdGuard rewrite.\n\nThis change makes /health return the last backend status recorded by the normal poll loop, keeps /healthz as the cheap liveness endpoint, and adds a small stdlib HTTP/1.1 connection pool so actual upstream calls can reuse connections. Poll failures still update health to down.\n\nTests: focused RSS, real module contract, architecture, module contract, and discovery tests pass (76 passed). The full suite was also attempted but currently hits the unrelated env-key inventory mismatch on main.
View command line instructions
Manual merge helper
Use this merge commit message when completing the merge manually.
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.