Skip to content

Conversation

ogtownsend
Copy link
Contributor

@ogtownsend ogtownsend commented Oct 8, 2025

  • Currently we try to fetch the native token prices for ALL supported chains even if these aren't enabled source chains in the OffRamp's source chain configs
  • This leads to spamming the Native token address is zero or empty warning log
  • This PR first checks whether or not the source chain is enabled before trying to get the native token address from the router config

@ogtownsend ogtownsend changed the title Only fetch wrapped native token prices for dest chain or enabled sour… Only fetch wrapped native token prices for dest chain or enabled source chains Oct 8, 2025
Copy link
Contributor

@archseer archseer left a comment

Choose a reason for hiding this comment

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

This was previously attempted but was abandoned and the log was made lower severity:

#862 (comment)

#863

@makramkd
Copy link
Collaborator

makramkd commented Oct 9, 2025

Yeah we can't use config poller directly but shouldn't we be able to use the information from the discovery processor, which is shared with all nodes?

Its worth noting that we're no longer pursuing a role DON setup in production - all nodes are currently connected to all chains.

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.

3 participants