Skip to content

Conversation

@vm-001
Copy link
Collaborator

@vm-001 vm-001 commented Oct 17, 2025

Summary

This PR added proxy support for the deliverer client. HTTP, HTTPS, and mTLS proxies are supported,

@vm-001 vm-001 marked this pull request as draft October 17, 2025 16:07
@vm-001 vm-001 force-pushed the feat/deliverer-http-proxy branch from 9418f15 to 9677dab Compare October 22, 2025 08:51
@vm-001 vm-001 marked this pull request as ready for review October 22, 2025 08:51
@vm-001 vm-001 force-pushed the feat/deliverer-http-proxy branch 2 times, most recently from b43c305 to 1d9a34a Compare October 22, 2025 08:59
@codecov
Copy link

codecov bot commented Oct 22, 2025

Codecov Report

❌ Patch coverage is 87.67123% with 9 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
worker/deliverer/http.go 80.85% 5 Missing and 4 partials ⚠️
Flag Coverage Δ
integration 78.85% <79.45%> (+0.36%) ⬆️
unit 13.48% <19.17%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
app/app.go 84.37% <100.00%> (+1.94%) ⬆️
config/worker.go 100.00% <100.00%> (ø)
worker/deliverer/deliverer.go 100.00% <ø> (ø)
worker/deliverer/http.go 82.56% <80.85%> (-2.28%) ⬇️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vm-001 vm-001 force-pushed the feat/deliverer-http-proxy branch 2 times, most recently from 78040ee to 83ede48 Compare October 22, 2025 16:33
@vm-001 vm-001 force-pushed the feat/deliverer-http-proxy branch from 83ede48 to 59da689 Compare October 27, 2025 10:28
@vm-001 vm-001 changed the title feat(worker): forward proxy feat(worker): deliverer proxy Oct 27, 2025
@webhookx-x webhookx-x merged commit 1cf38c1 into main Oct 27, 2025
8 checks passed
@webhookx-x webhookx-x deleted the feat/deliverer-http-proxy branch October 27, 2025 15:48
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.

3 participants