vscode-azureappservice icon indicating copy to clipboard operation
vscode-azureappservice copied to clipboard

Display an empty "node_modules" directory after deploying successfully

Open v-dantch opened this issue 5 years ago • 7 comments

OS: Win10 Build Version: 20191121.1 Regression: Not a Regression

Repro Steps:

  1. Get a sample project https://github.com/Azure-Samples/nodejs-docs-hello-world/archive/master.zip
  2. Add request package by "npm install request --save" to this sample project --> Open this project in VS Code.
  3. Create a Linux web app with Node.js LTS runtime --> Set the deployment source to "Node".
  4. Deploy the sample project to the new created web app --> Pop up a prompting dialog --> Select "Yes". image
  5. Deploy. image
  6. Succeed to deploy. image
  7. Check the node_ folder.

Expect: All files display well under "node_modules".

Actual: The "node_modules" directory is empty. image

More Info:

  1. The website displays well. image
  2. This issue also reproduces for Linux web app with Node.js 10.16 and Node.js 12.9(We only verified this issue with these 3 runtimes).

v-dantch avatar Nov 22 '19 02:11 v-dantch

If you do a zip deploy without the .deployment file, it seems to work fine. Is Oryx no longer running npm install? @fiveisprime

nturinski avatar Nov 22 '19 17:11 nturinski

I'm worried that this is a timing issue. Notice that they have no .deployment in the file list? Seems like the deployment is happening before the user confirms to enable build.

I'm seeing inconsistent behavior on this right now. Sometimes I get the prompt and other times I don't.

fiveisprime avatar Dec 02 '19 17:12 fiveisprime

Based on the offline discussion, this seems to be caused because the wwwroot directory actually contains a symlink to the node_modules rather than having the node_modules being built in the wwwroot directory.

nturinski avatar Dec 02 '19 22:12 nturinski

Follow up with App Service team

nturinski avatar Dec 12 '19 23:12 nturinski

Verified on the latest build 20200105.1. this issue doesn't reproduce for now. So close it.

v-xinda avatar Jan 06 '20 08:01 v-xinda

This issue reproduces on the build 20200408.1, reopen it.

v-xinda avatar Apr 09 '20 10:04 v-xinda

Yeah this is what I see: Screen Shot 2020-04-13 at 3 27 28 PM

ejizba avatar Apr 13 '20 22:04 ejizba