Description
express and aws-sdk are installed in the Dockerfile, but aren't listed in package.json or package-lock.json
Expected behavior
express and aws-sdk should be listed as dependencies in package.json and package-lock.json and the install commands for them removed from the Dockerfile so that everything is installed with just npm ci.
Why the expected behavior is an improvement
This allows the dependencies to managed in the normal Node way and ensures the dependency versions will be identical in all Docker images.
Additional details
aws-sdk probably isn't needed any more, but it can be removed later/separately along with anything else that's no longer needed.
Description
expressandaws-sdkare installed in the Dockerfile, but aren't listed in package.json or package-lock.jsonExpected behavior
expressandaws-sdkshould be listed as dependencies in package.json and package-lock.json and the install commands for them removed from the Dockerfile so that everything is installed with justnpm ci.Why the expected behavior is an improvement
This allows the dependencies to managed in the normal Node way and ensures the dependency versions will be identical in all Docker images.
Additional details
aws-sdkprobably isn't needed any more, but it can be removed later/separately along with anything else that's no longer needed.