• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

umputun / reproxy / 26734019528
79%

Build:
DEFAULT BRANCH: master
Ran 01 Jun 2026 03:48AM UTC
Jobs 1
Files 19
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

01 Jun 2026 03:46AM UTC coverage: 79.494% (+0.3%) from 79.214%
26734019528

push

github

web-flow
fix: critical and major issues from code audit (#254)

* docs: add plan for critical and major audit findings

* fix: dedup consul services with multiple reproxy tags

* fix: release plugin lock before blocking RPC call

* fix: guard mappersCache against concurrent map write

* fix: close health-ping response body

* docs: document only-from header trust assumption

* fix: bound discovery shutdown and stop signal goroutine leak on run() return

cancel the discovery context on every run() return path and await a bounded
discoveryDone before exit so provider watcher goroutines don't outlive run.
register signal.Stop and a catch-all defer cancel so the signal goroutine
can't leak across run() calls.

* fix: make docker and consul event sends cancellation-aware

the provider watcher goroutines did a bare send on the events channel, so
on shutdown a producer blocked mid-send (once the consumer stopped on ctx
cancel) could leak. guard both sends with a select on ctx.Done so the
discovery shutdown wait can't be left hanging on a stuck producer.

* test: cover plugin lock-release and discovery shutdown; fix Middleware godoc

- add TestConductor_MiddlewareReleasesLockDuringCall: a blocking plugin RPC,
  asserting a write-lock op succeeds while the call is in flight (proves the
  read lock is released before the RPC, not held across it)
- assert run() returns well under the 5s shutdown bound on the error path,
  proving discovery's ctx is cancelled before the wait
- correct Conductor.Middleware godoc: a call error returns HTTP 500 and stops
  the chain, a reply status >= 400 stops with that status

* docs: move audit-findings plan to completed

41 of 47 new or added lines in 6 files covered. (87.23%)

2609 of 3282 relevant lines covered (79.49%)

38.41 hits per line

Uncovered Changes

Lines Coverage ∆ File
6
85.78
0.23% app/main.go
Jobs
ID Job ID Ran Files Coverage
1 26734019528.1 01 Jun 2026 03:48AM UTC 19
79.49
GitHub Action Run
Source Files on build 26734019528
  • Tree
  • List 19
  • Changed 7
  • Source Changed 0
  • Coverage Changed 7
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • d311e7ae on github
  • Prev Build on master (#25770167686)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc