Skip to content

Collect defi assets#871

Open
erokhinav wants to merge 3 commits into
masterfrom
collect_defi_assets
Open

Collect defi assets#871
erokhinav wants to merge 3 commits into
masterfrom
collect_defi_assets

Conversation

@erokhinav
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread pkg/references/defi.go
DefiProviderMeta: DefiProviderMeta{
Name: "Tonstakers",
URL: "https://app.tonstakers.com/",
Image: "https://cache.tonapi.io/imgproxy/qkMSC0tZBAgHZkpLfCvhQJMXnHVpFKjhGM4eMD7HWWQ/rs:fill:200:200:1/g:no/aHR0cHM6Ly90b25zdGFrZXJzLmNvbS9sb2dvLnBuZw.webp",
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.

этот урл сдохнет если сротируются ключи imgproxy
https://www.figma.com/design/RGMQMKmoahly785O8Z9Eqc/Tonstakers-public-graphics-assets?node-id=0-1&p=f&t=XlCYUN6wArg0ZChz-0 вот тут можно взять нормальное лого и закинуть куда-нибудь

Comment thread pkg/defi/utils.go Outdated
}
}

func fetchJSON(ctx context.Context, url string, dst any) error {
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.

так нельзя. ты спалишь все адреса бэкэндов тонапи всем сервисам которые опрашиваешь. 1) все запросы надо делать через список сокс проксей для исходящих вызов.
2) в идеале апишки вообще не должны в работе завязываться на вызовы сторонних сервисов. обычно в таком случае мы делали кэширование в миске и оттуда уже запрашивали

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