Skip to content

Implement allowlist admin api - #2609

Merged
Mirko-von-Leipzig merged 6 commits into
mirko/account-whitelistfrom
mirko/account-whitelist-admin
Sep 15, 2026
Merged

Mirko-von-Leipzig merged 6 commits into
mirko/account-whitelistfrom
mirko/account-whitelist-admin

Conversation

@Mirko-von-Leipzig

Copy link
Copy Markdown
Collaborator

Summary

Adds an optional administrator REST API to the sequencer which supports:

  1. inserting a new invitation code to the allowlist
  2. adding an account ID directly to the allowlist (no registration required)
  3. querying the state of an account or invitation code

Changelog

[[entry]]
scope       = "node"
impact      = "added"
description = "Added an admin REST API to the sequencer for managing the account allowlist"

Stack created with GitHub Stacks CLIGive Feedback 💬

@Mirko-von-Leipzig
Mirko-von-Leipzig added this pull request to stack #2611 September 11, 2026 08:23
@Mirko-von-Leipzig
Mirko-von-Leipzig force-pushed the mirko/account-whitelist-admin branch from 306c20c to bfb98c2 Compare September 11, 2026 08:33

@SantiagoPittella SantiagoPittella left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not a very deep review, but overall looked good. The endpoints are going to be limited at the proxy/LB level, right?

@igamigo
igamigo force-pushed the mirko/account-whitelist-admin branch from 1601b5b to 68c3006 Compare September 14, 2026 16:07

@kkovaacs kkovaacs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

Comment thread bin/node/src/admin/mod.rs Outdated
@Mirko-von-Leipzig
Mirko-von-Leipzig merged commit 09af566 into next Sep 15, 2026
35 of 57 checks passed
@Mirko-von-Leipzig
Mirko-von-Leipzig deleted the mirko/account-whitelist-admin branch September 15, 2026 12:01
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.

4 participants