Skip to content

fix: fast-track for RPC service#829

Merged
aloknerurkar merged 4 commits intomainfrom
rpc-fixes.11.2
Oct 15, 2025
Merged

fix: fast-track for RPC service#829
aloknerurkar merged 4 commits intomainfrom
rpc-fixes.11.2

Conversation

@aloknerurkar
Copy link
Copy Markdown
Collaborator

Describe your changes

Allow authenticated API to configure a provider set as a fast track group. If all the providers in the fast track group commit, we will preconfirm the transaction.

Issue ticket number and link

Fixes # (issue)

Checklist before requesting a review

  • I have added tests that prove my fix is effective or that my feature works
  • I have made corresponding changes to the documentation

Comment thread tools/preconf-rpc/main.go
EnvVars: []string{"PRECONF_RPC_WEBHOOK_URLS"},
}

optionAuthToken = &cli.StringFlag{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need a refresh interval of 1h for the auth token if it's only loaded on startup? Is the token intended to rotate every hour?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

No right now this is not being rotated. But we can rotate by updating AWS secrets which should trigger kubernetes to update the value. But I think the app will need to be restarted as it is configured on start.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Not yet sure if this is going to be a feature for the long run, so I prefer not having some elaborate JWT scheme for this right now.

@aloknerurkar aloknerurkar merged commit 9b61cb5 into main Oct 15, 2025
4 of 5 checks passed
@aloknerurkar aloknerurkar deleted the rpc-fixes.11.2 branch October 15, 2025 10:30
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.

2 participants