Recursive copy in dockerfile should be avoided.

[rohit-joy]

This line in the dockerfile seems to be copying everything in the repo, but everything is not needed to build and run the app.

ADD . $APP_PATH

Can you modify this to just COPY over what is required?

[arielr-lt]

yes! definitely

[arielr-lt]

there is one detail, not sure if you noticed that there is a .dockerignore file, so it is not strictly a recursive copy, what I'm going to add to the file is the Dockerfile as you suggested, among other stuff

[rohit-joy]

That can work, but .dockerignore can quickly go out of date. I would prefer that we copy just the folders or files that we need.

[arielr-lt]

sounds good @rohit-joy I will go down this road!