FoundKeyGang/FoundKey
FoundKeyGang/FoundKey
7
19
Fork 22
Code Issues 37 Pull requests 4 Releases Activity

backend: refactor server/nodeinfo.ts #221

Merged
norm merged 5 commits from refactor-nodeinfo into main 2022-11-02 21:42:52 +00:00
Conversation 0 Commits 5 Files changed 1 +2 -1
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit f950b08dbf - Show all commits

3
packages/backend/src/server/nodeinfo.ts
View file

@ -4,6 +4,7 @@ import config from '@/config/index.js';
import { fetchMeta } from '@/misc/fetch-meta.js';
import { Users, Notes } from '@/models/index.js';
import { Cache } from '@/misc/cache.js';
import { MINUTE, SECOND } from '@/const.js';
const router = new Router();
@ -108,7 +109,7 @@ const nodeinfo2 = async (): Promise<NodeInfo2Base> => {
};
};
const cache = new Cache<NodeInfo2Base>(1000 * 60 * 10);
const cache = new Cache<NodeInfo2Base>(10 * MINUTE);
norm marked this conversation as resolved Outdated
Johann150 commented 2022-11-02 19:27:11 +00:00
Outdated
Review
Copy link

Is this cache even necessary? Maybe we should rely on the caching of the reverse proxy more.

Is this cache even necessary? Maybe we should rely on the caching of the reverse proxy more.
Michcio commented 2022-11-02 19:29:09 +00:00
Outdated
Review
Copy link

Does the reverse proxy cache it by default?

Does the reverse proxy cache it by default?
router.get(nodeinfo2_1path, async ctx => {
const base = await cache.fetch(null, () => nodeinfo2());