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

docs(readme): document 1account/proxy #353

Merged
merged 2 commits into from
Mar 28, 2025

Conversation

samlaf
Copy link
Collaborator

@samlaf samlaf commented Mar 24, 2025

Fixes Issue

Closes #250
Closes DAINT-246

Changes proposed

Screenshots (Optional)

Note to reviewers

@samlaf samlaf requested review from litt3 and ethenotethan March 24, 2025 21:36
README.md Outdated
@@ -202,6 +201,8 @@ This behavior is turned on by default, but configurable via the `--eigenda.confi

In order to disperse to the EigenDA V1 network in production, or at high throughput on testnet, please register your authentication ethereum address through [this form](https://forms.gle/3QRNTYhSMacVFNcU8). Your EigenDA authentication keypair address should not be associated with any funds anywhere. For EigenDA V2, please see our [payments](https://docs.eigenda.xyz/releases/payments) doc.

> Note: Proxy only supports using a single authn (v1) or payment (v2) key. We recommend running a single instance of the Proxy per Rollup.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
> Note: Proxy only supports using a single authn (v1) or payment (v2) key. We recommend running a single instance of the Proxy per Rollup.
> Note: Proxy only supports using a single authorization (v1) or payment (v2) key. We recommend running a single instance of the Proxy per Rollup (i.e, 1 proxy : 1 rollup) and avoid provisioning patterns like N rollups : 1 proxy.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Changed it to 505f8bd

ethenotethan
ethenotethan previously approved these changes Mar 24, 2025
Copy link
Collaborator

@ethenotethan ethenotethan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@ethenotethan ethenotethan left a comment

Choose a reason for hiding this comment

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

LGTM

@samlaf samlaf merged commit f1ad272 into main Mar 28, 2025
11 of 13 checks passed
@samlaf samlaf deleted the docs--document-single-account-per-proxy branch March 28, 2025 18:38
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.

Support multiple accountIds for RaaS support
2 participants