Skip to content

docs: warn about priceTTLInMS vs refreshDelayInMS interaction #95

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.

Sign up for GitHub

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

Open
munikes wants to merge 1 commit into
base: main
Choose a base branch
from
Open

docs: warn about priceTTLInMS vs refreshDelayInMS interaction #95

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions src/server/config/config-networks.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,15 @@ const MINIMUM_BALANCE_FOR_AVAILABILITY_L2 = 0.01;

// 10 minute timeout on ticket prices retrieved from API.
// Shorter is safer, but Coingecko free tier can lag by 15-30 minutes.
//
// IMPORTANT: this MUST stay >= the price refresher's refreshDelayInMS
// (configTokenPriceRefresher.tokenPriceRefreshers[...].refreshDelayInMS,
// overridable in MY-CONFIG.ts). If you raise the refresh interval (e.g. to
// save CoinGecko free-tier quota) without raising this TTL by at least as
// much, every cycle gets a guaranteed window - equal to the gap between the
// two values - where cached prices are considered "expired" and the
// broadcaster silently stops publishing fees for every token (NO TOKEN
// PRICE warnings), with no underlying error: the API call itself succeeds.
const defaultTokenPriceTTL = 10 * 60 * 1000;

const networksConfig: NetworksConfig = {
Expand Down