Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Issue #200 #201

Closed
wants to merge 1 commit into from
Closed

Conversation

WilliamRagstad
Copy link

Error: mime.lookup is not a function
Solution: Function is checked before use. No difference in functionality but fixes critical error for some dependent modules.

I do not know why mime.lookup is undefined, but this has stopped me from using other packages that are dependent on send and has required me to continuously change this piece of code each time to be able to build my project. I would be grateful if this was merged and included in the next release.

TypeError: mime.lookup is not a function is checked before use.
No difference in functionality but fixes important error for dependent modules.
@dougwilson
Copy link
Contributor

This just gets around one error you will see.this module and modules that use it expect the mime function to work and the content-type header needs to be in your responses for your assets to load properly in web browsers.

If you can include a change to the test suite that validates this change we can reconsider.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants