Skip to content

Comments

fix sendFile deprecated message.#7

Open
ttousai wants to merge 3 commits intorauchg:masterfrom
ttousai:js
Open

fix sendFile deprecated message.#7
ttousai wants to merge 3 commits intorauchg:masterfrom
ttousai:js

Conversation

@ttousai
Copy link

@ttousai ttousai commented Oct 2, 2014

No description provided.

ttousai and others added 3 commits October 2, 2014 19:15
Just replacing "sendfile" with "sendFile" results in "TypeError: path must be absolute or specify root to res.sendFile". I think specifying an absolute path utilizing the __dirname Node.js global is best, rather than introducing the options object for sendFile.
http://nodejs.org/docs/latest/api/globals.html#globals_dirname
http://expressjs.com/api.html#res.sendFile
sendFile requires absolute path or specified root
TheSachinSBhat added a commit to TheSachinSBhat/chat-example-1 that referenced this pull request Feb 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants