Add clarifying comments and improve code readability

Co-authored-by: whyour <22700758+whyour@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot] 2025-12-22 14:55:32 +00:00
parent 8bff4202c7
commit 60aab8f95d
2 changed files with 6 additions and 1 deletions

View File

@ -29,6 +29,8 @@ export default ({ app }: { app: Application }) => {
}
// Create base-URL-aware whitelist for JWT
// When baseUrl is empty, paths remain as-is (e.g., '/api/user/login')
// When baseUrl is set, paths are prefixed (e.g., '/qinglong/api/user/login')
const jwtWhitelist = config.apiWhiteList.map(path => `${config.baseUrl}${path}`);
// Exclude non-API/non-open paths from JWT requirement
// When baseUrl is set: exclude paths that don't start with baseUrl/api/ or baseUrl/open/
@ -84,6 +86,8 @@ export default ({ app }: { app: Application }) => {
}
}
// req.path already includes the full path with baseUrl
// e.g., when baseUrl=/qinglong and request is /qinglong/api/user/login, req.path=/qinglong/api/user/login
const originPath = req.path;
if (
!headerToken &&

View File

@ -93,8 +93,9 @@ const apiWhiteListBase = [
];
// Only modify paths if baseUrl is set and not the default '/'
// Frontend baseUrl always ends with '/', so we remove leading '/' from paths
const apiWhiteList = config.baseUrl && config.baseUrl !== '/'
? apiWhiteListBase.map(path => `${config.baseUrl}${path.replace(/^\//, '')}`)
? apiWhiteListBase.map(path => `${config.baseUrl}${path.substring(1)}`)
: apiWhiteListBase;