Skip to content

Comments

add kindling tester#331

Open
WendelHime wants to merge 7 commits intomainfrom
feat/add-kindling-tester
Open

add kindling tester#331
WendelHime wants to merge 7 commits intomainfrom
feat/add-kindling-tester

Conversation

@WendelHime
Copy link
Contributor

@WendelHime WendelHime commented Feb 19, 2026

related to getlantern/engineering#2976

@WendelHime WendelHime marked this pull request as ready for review February 20, 2026 13:42
Copilot AI review requested due to automatic review settings February 20, 2026 13:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 EnabledTransports map
  • 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.

@myleshorton
Copy link
Contributor

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.

@WendelHime
Copy link
Contributor Author

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 healthz endpoint

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.

2 participants