Merge branch '4295-windows-ecs-support' of github.com:arhea/aws-cdk i…

…nto 4295-windows-ecs-support

* '4295-windows-ecs-support' of github.com:arhea/aws-cdk:
  chore(deps-dev): bump @types/lodash from 4.14.146 to 4.14.147 (aws#5021)
  Revert "fix(assets): support exceptions to exclude patterns (aws#4473)" (aws#5022)
  chore(deps): bump jsii-pacmak from 0.20.3 to 0.20.5 (aws#5003)
  chore(deps): bump codemaker from 0.20.3 to 0.20.5 (aws#5007)
  chore(deps-dev): bump @types/jest from 24.0.22 to 24.0.23 (aws#4993)
  chore(deps): bump jsii from 0.20.3 to 0.20.5 (aws#5006)
  chore(deps-dev): bump jsii-diff from 0.20.3 to 0.20.5 (aws#5005)
  chore(deps): bump jsii-spec from 0.20.3 to 0.20.5 (aws#5008)
  chore(core): resolve tokens before publishing tree.json (aws#4984)
  feat(cli): adding new option to `cdk deploy` to indicate whether ChangeSet should be executed (aws#4852)
  chore: move semantic.yaml to .github/
  fix(core): unable to find stack by name using the cli in legacy mode (aws#4998)
  fix(ecs-patterns): Fix issue related to protocol being passed to target group (aws#4988)

.github/semantic.yaml

@@ -0,0 +1,17 @@
+# Configuration for Semantic Pull Requests
+titleOnly: true
+
+types:
+  - feat
+  - fix
+  - docs
+  - style
+  - refactor
+  - perf
+  - test
+  - build
+  - ci
+  - chore
+  - revert
+  - release
+

package.json

@@ -13,14 +13,14 @@
     "build-all": "tsc -b"
   },
   "devDependencies": {
-    "@types/jest": "^24.0.22",
+    "@types/jest": "^24.0.23",
     "@types/node": "^10.17.5",
     "@types/nodeunit": "^0.0.30",
     "conventional-changelog-cli": "^2.0.27",
     "fs-extra": "^8.1.0",
     "jest": "^24.9.0",
-    "jsii-diff": "^0.20.3",
-    "jsii-pacmak": "^0.20.3",
+    "jsii-diff": "^0.20.5",
+    "jsii-pacmak": "^0.20.5",
     "lerna": "^3.18.4",
     "nodeunit": "^0.11.3",
     "nyc": "^14.1.1",

packages/@aws-cdk/assert/package.json

@@ -26,7 +26,7 @@
   },
   "license": "Apache-2.0",
   "devDependencies": {
-    "@types/jest": "^24.0.22",
+    "@types/jest": "^24.0.23",
     "cdk-build-tools": "1.16.1",
     "pkglint": "1.16.1"
   },

packages/@aws-cdk/assets/lib/fs/copy.ts

@@ -2,38 +2,56 @@ import fs = require('fs');
 import path = require('path');
 import { CopyOptions } from './copy-options';
 import { FollowMode } from './follow-mode';
-import { mkdirpSync } from './mkdirpSync';
-import { listFilesRecursively, shouldFollow } from './utils';
+import { shouldExclude, shouldFollow } from './utils';
 
 export function copyDirectory(srcDir: string, destDir: string, options: CopyOptions = { }, rootDir?: string) {
   const follow = options.follow !== undefined ? options.follow : FollowMode.EXTERNAL;
+  const exclude = options.exclude || [];
 
   rootDir = rootDir || srcDir;
 
   if (!fs.statSync(srcDir).isDirectory()) {
     throw new Error(`${srcDir} is not a directory`);
   }
 
-  for (const assetFile of listFilesRecursively(srcDir, {...options, follow}, rootDir)) {
-    const filePath = assetFile.relativePath;
-    const destFilePath = path.join(destDir, filePath);
+  const files = fs.readdirSync(srcDir);
+  for (const file of files) {
+    const sourceFilePath = path.join(srcDir, file);
 
-    if (follow !== FollowMode.ALWAYS) {
-      if (assetFile.isSymbolicLink) {
-        const targetPath = path.normalize(path.resolve(srcDir, assetFile.symlinkTarget));
-        if (!shouldFollow(follow, rootDir, targetPath)) {
-          fs.symlinkSync(assetFile.symlinkTarget, destFilePath);
+    if (shouldExclude(exclude, path.relative(rootDir, sourceFilePath))) {
+      continue;
+    }
+
+    const destFilePath = path.join(destDir, file);
+
+    let stat: fs.Stats | undefined = follow === FollowMode.ALWAYS
+      ? fs.statSync(sourceFilePath)
+      : fs.lstatSync(sourceFilePath);
+
+    if (stat && stat.isSymbolicLink()) {
+      const target = fs.readlinkSync(sourceFilePath);
 
-          continue;
-        }
+      // determine if this is an external link (i.e. the target's absolute path
+      // is outside of the root directory).
+      const targetPath = path.normalize(path.resolve(srcDir, target));
+
+      if (shouldFollow(follow, rootDir, targetPath)) {
+        stat = fs.statSync(sourceFilePath);
+      } else {
+        fs.symlinkSync(target, destFilePath);
+        stat = undefined;
       }
     }
 
-    if (!assetFile.isDirectory) {
-      mkdirpSync(path.dirname(destFilePath));
-      fs.copyFileSync(assetFile.absolutePath, destFilePath);
-    } else {
-      mkdirpSync(destFilePath);
+    if (stat && stat.isDirectory()) {
+      fs.mkdirSync(destFilePath);
+      copyDirectory(sourceFilePath, destFilePath, options, rootDir);
+      stat = undefined;
+    }
+
+    if (stat && stat.isFile()) {
+      fs.copyFileSync(sourceFilePath, destFilePath);
+      stat = undefined;
     }
   }
 }

packages/@aws-cdk/assets/lib/fs/fingerprint.ts

@@ -3,7 +3,7 @@ import fs = require('fs');
 import path = require('path');
 import { CopyOptions } from './copy-options';
 import { FollowMode } from './follow-mode';
-import { listFilesRecursively, shouldFollow } from './utils';
+import { shouldExclude, shouldFollow } from './utils';
 
 const BUFFER_SIZE = 8 * 1024;
 const CTRL_SOH = '\x01';
@@ -38,28 +38,40 @@ export function fingerprint(fileOrDirectory: string, options: FingerprintOptions
   const rootDirectory = fs.statSync(fileOrDirectory).isDirectory()
     ? fileOrDirectory
     : path.dirname(fileOrDirectory);
+  const exclude = options.exclude || [];
+  _processFileOrDirectory(fileOrDirectory);
 
-  for (const assetFile of listFilesRecursively(fileOrDirectory, {...options, follow}, rootDirectory)) {
-    const relativePath = path.relative(fileOrDirectory, assetFile.absolutePath);
+  return hash.digest('hex');
+
+  function _processFileOrDirectory(symbolicPath: string, realPath = symbolicPath) {
+    if (shouldExclude(exclude, symbolicPath)) {
+      return;
+    }
+
+    const stat = fs.lstatSync(realPath);
+    const relativePath = path.relative(fileOrDirectory, symbolicPath);
 
-    if (assetFile.isSymbolicLink) {
-      const resolvedLinkTarget = path.resolve(path.dirname(assetFile.absolutePath), assetFile.symlinkTarget);
-      if (!shouldFollow(follow, rootDirectory, resolvedLinkTarget)) {
-        _hashField(hash, `link:${relativePath}`, assetFile.symlinkTarget);
+    if (stat.isSymbolicLink()) {
+      const linkTarget = fs.readlinkSync(realPath);
+      const resolvedLinkTarget = path.resolve(path.dirname(realPath), linkTarget);
+      if (shouldFollow(follow, rootDirectory, resolvedLinkTarget)) {
+        _processFileOrDirectory(symbolicPath, resolvedLinkTarget);
       } else {
-        _hashField(hash, `file:${relativePath}`, _contentFingerprint(assetFile.absolutePath, assetFile.size));
+        _hashField(hash, `link:${relativePath}`, linkTarget);
       }
-    } else if (assetFile.isFile) {
-      _hashField(hash, `file:${relativePath}`, _contentFingerprint(assetFile.absolutePath, assetFile.size));
-    } else if (!assetFile.isDirectory) {
-      throw new Error(`Unable to hash ${assetFile.absolutePath}: it is neither a file nor a directory`);
+    } else if (stat.isFile()) {
+      _hashField(hash, `file:${relativePath}`, _contentFingerprint(realPath, stat));
+    } else if (stat.isDirectory()) {
+      for (const item of fs.readdirSync(realPath).sort()) {
+        _processFileOrDirectory(path.join(symbolicPath, item), path.join(realPath, item));
+      }
+    } else {
+      throw new Error(`Unable to hash ${symbolicPath}: it is neither a file nor a directory`);
     }
   }
-
-  return hash.digest('hex');
 }
 
-function _contentFingerprint(file: string, size: number): string {
+function _contentFingerprint(file: string, stat: fs.Stats): string {
   const hash = crypto.createHash('sha256');
   const buffer = Buffer.alloc(BUFFER_SIZE);
   // tslint:disable-next-line: no-bitwise
@@ -73,7 +85,7 @@ function _contentFingerprint(file: string, size: number): string {
   } finally {
     fs.closeSync(fd);
   }
-  return `${size}:${hash.digest('hex')}`;
+  return `${stat.size}:${hash.digest('hex')}`;
 }
 
 function _hashField(hash: crypto.Hash, header: string, value: string | Buffer | DataView) {