api/core: merge isApiKey and isSession into authType

cuz they can't be true at the same time
This commit is contained in:
wukko 2025-06-28 17:05:18 +06:00
parent bc8c16f469
commit d70180b23c
No known key found for this signature in database
GPG Key ID: 3E30B3F26C7B4AA2
2 changed files with 6 additions and 7 deletions

View File

@ -156,7 +156,7 @@ export const runAPI = async (express, app, __dirname, isPrimary = true) => {
return fail(res, `error.api.auth.key.${error}`);
}
req.isApiKey = true;
req.authType = "key";
return next();
});
@ -185,7 +185,7 @@ export const runAPI = async (express, app, __dirname, isPrimary = true) => {
}
req.rateLimitKey = hashHmac(token, 'rate');
req.isSession = true;
req.authType = "session";
} catch {
return fail(res, "error.api.generic");
}
@ -267,8 +267,7 @@ export const runAPI = async (express, app, __dirname, isPrimary = true) => {
host: parsed.host,
patternMatch: parsed.patternMatch,
params: normalizedRequest,
isSession: req.isSession ?? false,
isApiKey: req.isApiKey ?? false,
authType: req.authType ?? "none",
});
res.status(result.status).json(result.body);

View File

@ -32,7 +32,7 @@ import xiaohongshu from "./services/xiaohongshu.js";
let freebind;
export default async function({ host, patternMatch, params, isSession, isApiKey }) {
export default async function({ host, patternMatch, params, authType }) {
const { url } = params;
assert(url instanceof URL);
let dispatcher, requestIP;
@ -69,7 +69,7 @@ export default async function({ host, patternMatch, params, isSession, isApiKey
let youtubeHLS = params.youtubeHLS;
const hlsEnv = env.enableDeprecatedYoutubeHls;
if (hlsEnv === "never" || (hlsEnv === "key" && !isApiKey)) {
if (hlsEnv === "never" || (hlsEnv === "key" && authType !== "key")) {
youtubeHLS = false;
}
@ -313,7 +313,7 @@ export default async function({ host, patternMatch, params, isSession, isApiKey
let localProcessing = params.localProcessing;
const lpEnv = env.forceLocalProcessing;
const shouldForceLocal = lpEnv === "always" || (lpEnv === "session" && isSession);
const shouldForceLocal = lpEnv === "always" || (lpEnv === "session" && authType === "session");
const localDisabled = (!localProcessing || localProcessing === "none");
if (shouldForceLocal && localDisabled) {