Skip to content

Conversation

@chris-okuda
Copy link
Collaborator

@chris-okuda chris-okuda commented Oct 23, 2025

Scope of changes

Initial implementation which was copied and modified from Envoy.

  • Changed the Configure function to be Initialize and take the config AND templates
  • Added configs for the important backoff options so the user can control timing
  • using our x/backoff implementation instead of an external package that was used in Envoy
  • setup an example template for the live email test under testdata/

Type of change

  • bug fix
  • new feature
  • documentation
  • testing
  • other (describe)

Acceptance criteria

No special requirements.

Author checklist

  • I have manually tested the change and/or added automation in the form of unit tests or integration tests
  • I have updated the dependencies list
  • I have added new test fixtures as needed to support added tests
  • I have added or updated the documentation
  • Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)

Reviewer(s) checklist

  • Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.
  • To the best of my ability, I believe that this PR represents a good solution to the specified problem and that it should be merged into the main code base.

@chris-okuda chris-okuda self-assigned this Oct 23, 2025
@chris-okuda chris-okuda requested a review from bbengfort October 24, 2025 00:07
chris-okuda and others added 2 commits October 23, 2025 14:13
fix one more path

Signed-off-by: Chris Okuda <chris@rotational.io>
@bbengfort bbengfort merged commit a5ce44a into main Oct 24, 2025
2 checks passed
@bbengfort bbengfort deleted the chrisokuda/initial branch October 24, 2025 04:44
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