Conversation
…dd-kindling-tester
…dd-kindling-tester
There was a problem hiding this comment.
Pull request overview
This PR adds a new kindling-tester tool for testing individual kindling transports (proxyless, fronted, amp, dnstt). The implementation refactors the kindling client initialization to support selectively enabling/disabling transports via a configurable map, and introduces a standalone CLI tool that can be packaged as Docker images.
Changes:
- Refactored kindling transport initialization to be configurable via an
EnabledTransportsmap - Added kindling-tester CLI tool for testing individual transports
- Added Dockerfile for building transport-specific test images
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
| kindling/client.go | Refactored NewKindling() to conditionally initialize transports based on EnabledTransports map; added dnstt transport support |
| cmd/kindling-tester/main.go | New CLI tool that tests a single kindling transport by setting environment variables and making HTTP requests |
| cmd/kindling-tester/README.md | Documentation for the kindling-tester tool with CLI and Docker usage examples |
| docker/Dockerfile.kindling-tester | Multi-stage Dockerfile for building kindling-tester binary |
| .gitignore | Added patterns to ignore local environment files for kindling-tester |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
One thing to keep in mind is that fronted isn't designed to work with any arbitrary site -- it only works with the ones we've configured it for, such as api.iantem.io. |
Yeah, in this case I'm considering pinging the |
related to getlantern/engineering#2976