-
Notifications
You must be signed in to change notification settings - Fork 81
feat: added IS_MODULARIZED axios header for MirrorNodeClient #3626
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
Conversation
Signed-off-by: Logan Nguyen <[email protected]>
Test Results 22 files - 1 298 suites +20 34m 40s ⏱️ - 3m 1s For more details on these failures, see this check. Results for commit fa4a3a5. ± Comparison against base commit 7077237. This pull request removes 5 and adds 4 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Signed-off-by: Logan Nguyen <[email protected]>
Description
This PR adds support for the
Is-Modularized
header in Mirror Node requests through a new configuration flag,USE_MIRROR_NODE_MODULARIZED_SERVICES
, allowing controlled routing of traffic to modularized services within the Mirror Node.Context
Currently, only a small percentage of Mirror Node traffic on Previewnet is routed to modularized services, with a gradual increase planned until full adoption across all networks.
For now, the Relay explicitly sets this to
false
to ensure traffic follows the non-modularized flow. Once the Mirror Node fully transitions, this will be updated totrue
as a default value.Changes
USE_MIRROR_NODE_MODULARIZED_SERVICES
configuration flag (default: false)Is-Modularized
with appropriate string conversionRelated issue(s):
Fixes #3623
Checklist