Skip to content
This repository has been archived by the owner on Nov 5, 2023. It is now read-only.

Add aggregator-proxy #200

Merged
merged 10 commits into from
May 5, 2022
Merged

Add aggregator-proxy #200

merged 10 commits into from
May 5, 2022

Conversation

voltrevo
Copy link
Collaborator

@voltrevo voltrevo commented May 5, 2022

What is this PR doing?

Adds bls-wallet-aggregator-proxy.

I've already published this so you can try it with yarn add bls-wallet-aggregator-proxy (in a separate test project).

This package makes it easy to provide an aggregator by proxying another. The primary use-case is to expose a free aggregator based on one that requires payment by augmenting the bundles with txs that pay tx.origin.

How can these changes be manually tested?

Use manualTests/echoServer on an existing aggregator and confirm that the extension/etc works correctly by pointing to it.

Does this PR resolve or contribute to any issues?

Contributes to #170.

Checklist

  • I have manually tested these changes
  • Post a link to the PR in the group chat

Guidelines

  • If your PR is not ready, mark it as a draft
  • The resolve conversation button is for reviewers, not authors
    • (But add a 'done' comment or similar)

@voltrevo voltrevo changed the title Aggregator proxy Add aggregator-proxy May 5, 2022
@jacque006 jacque006 merged commit 5f905c3 into main May 5, 2022
@jacque006 jacque006 deleted the aggregator-proxy branch May 5, 2022 21:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants