Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

refactor: dont load all tokens in verify token

Browse Source 
use sortedSetsRemove
This commit is contained in:
Barış Soner Uşaklı
2023-07-28 11:47:52 -04:00
parent 11bfeaf130
commit 2fe193d68f
2 changed files with 11 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/api/utils.js
View File

@@ -94,9 +94,11 @@ utils.tokens.roll = async (token) => {
const updates = [
db.rename(`token:${token}`, `token:${newToken}`),
db.sortedSetRemove(`tokens:createtime`, token),
db.sortedSetRemove(`tokens:uid`, token),
db.sortedSetRemove(`tokens:lastSeen`, token),
db.sortedSetsRemove([
`tokens:createtime`,
`tokens:uid`,
`tokens:lastSeen`,
], token),
db.sortedSetAdd(`tokens:createtime`, createTime, newToken),
db.sortedSetAdd(`tokens:uid`, uid, newToken),
];
@@ -113,9 +115,11 @@ utils.tokens.roll = async (token) => {
utils.tokens.delete = async (token) => {
await Promise.all([
db.delete(`token:${token}`),
db.sortedSetRemove(`tokens:createtime`, token),
db.sortedSetRemove(`tokens:uid`, token),
db.sortedSetRemove(`tokens:lastSeen`, token),
db.sortedSetsRemove([
`tokens:createtime`,
`tokens:uid`,
`tokens:lastSeen`,
], token),
]);
};

3
src/routes/authentication.js
View File

@@ -45,8 +45,7 @@ Auth.getLoginStrategies = function () {
};
Auth.verifyToken = async function (token, done) {
const tokens = await api.utils.tokens.list();
const tokenObj = tokens.filter((t => t.token === token)).pop();
const tokenObj = await api.utils.tokens.get(token);
const uid = tokenObj ? tokenObj.uid : undefined;
if (uid !== undefined) {