Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add faucet for miden-node #224

Closed
wants to merge 22 commits into from
Closed

Add faucet for miden-node #224

wants to merge 22 commits into from

Conversation

phklive
Copy link
Contributor

@phklive phklive commented Feb 12, 2024

The Miden node generates 2 accounts by default at genesis:

  • a Faucet account
  • a regular account

We need to make the faucet available and usable on node spin-up, this would enable request of funds and testing of tx's from users of the network.

In this PR I propose to add a new crate called faucet to the node workspace and use actix-web to create a web-server that will serve html, css and js static files.

The faucet receives requests of funds through an endpoint where users on the frontend pass their AccountId. The server responds with the note generated by the tx. The end user is then able to import this newly received note into their client and consume it locally.

closes: #215

@bobbinth
Copy link
Contributor

Awesome! One question: would it make sense to add this feature into the miden-node crate so that the faucet could be started together with the node (maybe using a command line arg).

One benefit of this is fewer components for users to manage. The downside is potential extra complexity in the node. I'm not sure how these balance each other.

@phklive phklive closed this Feb 28, 2024
@bobbinth bobbinth deleted the phklive-faucet-website branch April 12, 2024 07:46
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.

Create a faucet website for node testing
2 participants