Skip to content

fix: add per-token error isolation in bulk token registration#8788

Open
nvjsr wants to merge 1 commit into
masterfrom
CGD-722
Open

fix: add per-token error isolation in bulk token registration#8788
nvjsr wants to merge 1 commit into
masterfrom
CGD-722

Conversation

@nvjsr
Copy link
Copy Markdown
Contributor

@nvjsr nvjsr commented May 18, 2026

TICKET: CGD-722

Wrap createToken() in a try/catch in createTokenMapUsingConfigDetails() so a single malformed AMS entry is skipped with a structured warning instead of aborting the entire loop. Add empty-array guards before tokenConfigs[0] accesses in both createTokenMapUsingConfigDetails() and createTokenMapUsingTrimmedConfigDetails(). Wrap initCoinFactory() in registerAllTokens() for defence-in-depth error surfacing.

TICKET: CGD-722

Wrap createToken() in a try/catch in createTokenMapUsingConfigDetails()
so a single malformed AMS entry is skipped with a structured warning
instead of aborting the entire loop. Add empty-array guards before
tokenConfigs[0] accesses in both createTokenMapUsingConfigDetails() and
createTokenMapUsingTrimmedConfigDetails(). Wrap initCoinFactory() in
registerAllTokens() for defence-in-depth error surfacing.

TICKET: CGD-722
@nvjsr nvjsr requested review from a team as code owners May 18, 2026 07:57
@nvjsr nvjsr requested review from margueriteblair and ranga-r2 May 18, 2026 07:57
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 18, 2026

CGD-722

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