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

fix(remountable-routes): bug with user routes remounting to itself

Browse Source
This commit is contained in:
Julian Lam
2021-02-09 21:27:42 -05:00
parent 9021f071d4
commit bc68e990af
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/routes/index.js
View File

@@ -140,8 +140,9 @@ async function addCoreRoutes(app, router, middleware) {
account: 'user',
};
await Promise.all(remountable.map(async (mount) => {
const original = mount;
mount = mountHash[mount] || mount;
const method = mount;
const original = mountHash[mount] || mount;
mount = original;
({ mount } = await plugins.hooks.fire('filter:router.add', { mount }));
if (mount === null) { // do not mount at all
@@ -152,12 +153,12 @@ async function addCoreRoutes(app, router, middleware) {
if (mount !== original) {
// Set up redirect for fallback handling (some js/tpls may still refer to the traditional mount point)
winston.info(`[router] /${original} prefix re-mounted to /${mount}. Requests to /${original}/* will now redirect to /${mount}`);
router.use(new RegExp(`/(api/)?${mountHash[original] || original}`), (req, res) => {
router.use(new RegExp(`/(api/)?${original}`), (req, res) => {
controllerHelpers.redirect(res, `${nconf.get('relative_path')}/${mount}${req.path}`);
});
}
_mounts[original](router, mount, middleware, controllers);
_mounts[method](router, mount, middleware, controllers);
}));
const relativePath = nconf.get('relative_path');