186526
/
unpkg
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Do not needlessly redirect

This commit is contained in:
MICHAEL JACKSON 2017-08-24 01:12:26 -06:00
parent 02b0dc91e2
commit e95dcf0028
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
server/middleware/fetchFile.js
View File

@ -5,6 +5,10 @@ const createPackageURL = require('../utils/createPackageURL')
const getPackage = require('./utils/getPackage') const getPackage = require('./utils/getPackage')
const getPackageInfo = require('./utils/getPackageInfo') const getPackageInfo = require('./utils/getPackageInfo')
function getBasename(file) {
return path.basename(file, path.extname(file))
}
/** /**
* File extensions to look for when automatically resolving. * File extensions to look for when automatically resolving.
*/ */
@ -111,10 +115,7 @@ function fetchFile(req, res, next) {
filename = file.replace(req.packageDir, '') filename = file.replace(req.packageDir, '')
// TODO: We are able to serve files w/out the ".js" extension so if (req.query.module != null && getBasename(req.filename) !== getBasename(filename)) {
// don't bother redirecting when req.filename is "/area" and filename
// is "/area.js"
if (req.query.module != null && req.filename !== filename) {
// Need to redirect to the module file so relative imports resolve // Need to redirect to the module file so relative imports resolve
// correctly. Cache module redirects for 1 minute. // correctly. Cache module redirects for 1 minute.
res.set({ res.set({