-
Notifications
You must be signed in to change notification settings - Fork 984
Open
Labels
FeatureNeeds DiscussionServeLabel to identify bottlenecks for serve staticLabel to identify bottlenecks for serve static
Description
- Used appropriate template for the issue type
- Searched both open and closed issues for duplicates of this issue
- Title adequately and concisely reflects the feature or the bug
Feature Request
Use Case
It would be super cool if we could just use the serve-static plugin from the express ecosystem instead of continuing to invest in maintaining a parallel (and broken) internal serveStatic plugin.
Why do you want this?
Example API
var restify = require('restify');
var serveStatic = require('serve-static');
var app = restify.createServer();
var serve = serveStatic('./static');
app.get('.*', function(req, res, next) {
res.addListener('finish', next);
serve(req, res, function (err) {
res.removeListener('finish', next);
next(err);
});
});
app.listen(8080);
It also looks like we need to support the req.originalUrl
pattern similar to express: https://github.com/expressjs/express/blob/c0136d8b48dd3526c58b2ad8666fb4b12b55116c/lib/router/index.js#L172
This should include code snippets and documentation for the proposed feature
Are you willing and able to implement this?
"Yes" or, if "no", what can current contributors do to help you create a PR?
Yup!
neoadventist, blakew, kokarn, kevinpeno and rodrigooler
Metadata
Metadata
Assignees
Labels
FeatureNeeds DiscussionServeLabel to identify bottlenecks for serve staticLabel to identify bottlenecks for serve static