fix: refetch metadata token #93

vladkolotvin · 2022-10-27T17:31:33Z

closes #89

DavyJohnes · 2022-10-31T07:14:21Z

src/token-service/metadata-token-service.ts

@@ -67,7 +67,7 @@ export class MetadataTokenService implements TokenService {
                `failed to fetch token from metadata service: ${lastError}`,
            );
        }
-        setTimeout(async () => {
+        setInterval(async () => {
            try {
                this.token = await this.fetchToken();
            } catch {


А давай, пожалуйста, ошибку все же обработаетм? Хотя бы ворнинг в консоль таки выбросим.

nikolaymatrosov · 2022-10-31T07:49:29Z

У сервиса метаданных есть rate limiter (60rps). Если пересоздавать Session и не чистить за собой, может сложиться так, что накопится много интервальных колбеков, которые упрутся в лимит.

DavyJohnes · 2022-10-31T08:04:46Z

У сервиса метаданных есть rate limiter (60rps). Если пересоздавать Session и не чистить за собой, может сложиться так, что накопится много интервальных колбеков, которые упрутся в лимит.

Справедливо!
Кажется с учетом этого правильным будет такой подход:

при каждом обращении к сервису метадаты сохранять в поле класса tokenTimestamp.
при кажом вызове getToken проверять что currentTimestamp - tokenTimestamp <= 30, в противном случае перезапрашивать токен.

DavyJohnes · 2023-08-07T13:42:44Z

Will be fixed in https://github.com/yandex-cloud/nodejs-sdk/pull/135/files

fix: refetch metadata token

4002149

DavyJohnes reviewed Oct 31, 2022

View reviewed changes

DavyJohnes closed this Aug 7, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: refetch metadata token #93

fix: refetch metadata token #93

vladkolotvin commented Oct 27, 2022

DavyJohnes Oct 31, 2022

nikolaymatrosov commented Oct 31, 2022

DavyJohnes commented Oct 31, 2022 •

edited

Loading

DavyJohnes commented Aug 7, 2023

fix: refetch metadata token #93

fix: refetch metadata token #93

Conversation

vladkolotvin commented Oct 27, 2022

DavyJohnes Oct 31, 2022

Choose a reason for hiding this comment

nikolaymatrosov commented Oct 31, 2022

DavyJohnes commented Oct 31, 2022 • edited Loading

DavyJohnes commented Aug 7, 2023

DavyJohnes commented Oct 31, 2022 •

edited

Loading