Skip to content

feat(grpc,core): Apply shared traffic controller to gRPC API and refactor#11689

Merged
jkrvivian merged 6 commits into
developfrom
node/traffic-controller-follow-up
Jun 9, 2026
Merged

feat(grpc,core): Apply shared traffic controller to gRPC API and refactor#11689
jkrvivian merged 6 commits into
developfrom
node/traffic-controller-follow-up

Conversation

@jkrvivian

@jkrvivian jkrvivian commented May 29, 2026

Copy link
Copy Markdown
Contributor

Description of change

  • Wires the shared TrafficController into the gRPC server as a tower
    Layer, so the same blocklist and tally apply to a given client IP
    regardless of whether it hits the JSON-RPC or gRPC surface.
  • Client-IP extraction is factored out of authority_server.rs and reused by the
    new layer.
  • Includes follow-up cleanup on the recent traffic-controller port: upstream(core): Port latest traffic controller #11598

Links to any relevant issues

Closes #11668

How the change has been tested

  • Basic tests (linting, compilation, formatting, unit/integration tests)
  • Patch-specific tests (correctness, functionality coverage)
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that new and existing unit tests pass locally with my changes

@jkrvivian jkrvivian requested review from a team as code owners May 29, 2026 09:51
@iota-ci iota-ci added core-protocol node Issues related to the Core Node team labels May 29, 2026
@jkrvivian jkrvivian self-assigned this May 29, 2026
@jkrvivian jkrvivian force-pushed the node/traffic-controller-follow-up branch from 3688452 to a40aaf9 Compare May 29, 2026 09:54
Comment thread crates/iota-grpc-server/src/traffic_control.rs Outdated
Comment thread crates/iota-grpc-server/src/traffic_control.rs Outdated
@jkrvivian jkrvivian force-pushed the node/traffic-controller-follow-up branch from a40aaf9 to 5b9189e Compare June 3, 2026 09:05
@jkrvivian jkrvivian requested a review from bingyanglin June 3, 2026 09:05
Comment thread crates/iota-grpc-server/src/traffic_control.rs Outdated
Comment thread crates/iota-grpc-server/src/traffic_control.rs Outdated
Comment thread crates/iota-core/src/authority_server.rs
@jkrvivian jkrvivian requested a review from muXxer June 4, 2026 14:12
@jkrvivian jkrvivian merged commit ec8465f into develop Jun 9, 2026
36 checks passed
@jkrvivian jkrvivian deleted the node/traffic-controller-follow-up branch June 9, 2026 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core-protocol node Issues related to the Core Node team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

task: add traffic controller to gRPC

5 participants