backend: refactor server/nodeinfo.ts #221

Merged
norm merged 5 commits from refactor-nodeinfo into main 2022-11-02 21:42:52 +00:00
Owner

This fixes a few type errors like removing software.respository in
NodeInfo 2.0 and updating metadata.repositoryUrl to not use the
now removed meta repositoryUrl field.

This fixes a few type errors like removing `software.respository` in NodeInfo 2.0 and updating `metadata.repositoryUrl` to not use the now removed meta `repositoryUrl` field.
norm added 1 commit 2022-10-31 22:54:21 +00:00
This fixes a few type errors like removing `software.respository` in
NodeInfo 2.0 and updating `metadata.repositoryUrl` to not use the
now removed meta `repositoryUrl` field.
norm added 1 commit 2022-10-31 23:00:13 +00:00
norm added 1 commit 2022-10-31 23:00:50 +00:00
Johann150 approved these changes 2022-11-02 19:28:04 +00:00
@ -83,3 +110,3 @@
};
const cache = new Cache<Awaited<ReturnType<typeof nodeinfo2>>>(1000 * 60 * 10);
const cache = new Cache<NodeInfo2Base>(10 * MINUTE);
Owner

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.
Contributor

Does the reverse proxy cache it by default?

Does the reverse proxy cache it by default?
norm marked this conversation as resolved
norm added 1 commit 2022-11-02 21:40:13 +00:00
norm added 1 commit 2022-11-02 21:41:20 +00:00
norm merged commit e8ecd71f8a into main 2022-11-02 21:42:52 +00:00
norm deleted branch refactor-nodeinfo 2022-11-02 21:42:53 +00:00
Sign in to join this conversation.
No reviewers
No labels
feature
fix
upkeep
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: FoundKeyGang/FoundKey#221
No description provided.