Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(release): 1.73.0 #11414

Merged
merged 51 commits into from
Nov 11, 2020
Merged

chore(release): 1.73.0 #11414

merged 51 commits into from
Nov 11, 2020

Conversation

aws-cdk-automation
Copy link
Collaborator

@aws-cdk-automation aws-cdk-automation commented Nov 11, 2020

See CHANGELOG

johanneswuerbach and others added 30 commits November 6, 2020 04:44
Add support for [aws-apigatewayv2](https://docs.aws.amazon.com/cdk/api/latest/docs/aws-apigatewayv2-readme.html#custom-domain) `DomainName` as target.

Closes #8941

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Bumps [@types/lodash](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/lodash) from 4.14.164 to 4.14.165.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/lodash)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
…#11323)

Including a template that was using an `Fn::FindInMap` expression for a boolean-typed property
would fail with an error similar to: `Expected 'true' or 'false' for boolean value, got: '${Token[TOKEN.151]}'`.
The reason is that our `Fn.findInMap()` function in `core` incorrectly assumes `Fn::FindInMap`
can only return string values.
Change `CfnParser` to use a new, module-private function `Fn._findInMap()` that returns an `IResolvable`.

Fixes #11300

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Make it clear we are interested in the **CDK** (and not `aws`) CLI version.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
In v2, the `@aws-cdk/core` and `@aws-cdk/cx-api` modules are
private.
Update the tests so that it will only operate on v1. In v2, we will need
to design a new set of tests. This will be included in the re-design of
metadata collection - aws/aws-cdk-rfcs#254


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…hat base class is not exported (#11056)

This reverts commit from #10925 which folded all implementations of `UserPoolIdentityProviderBase`
into a single file.

It was desirable to maintain our original code organization but we also did not want to export the base class
In light of guidance provided in aws/jsii#2159,  reverted back to original code organization and added
the `@internal` decorator on the base class in a private directory

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Fixes: #11065
Add default to JsonSchema


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
fix(apigateway): ApiKey not supported for SpecRestApi

closes #11079 

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…tachment (#11237)

When the `excludeCharacters` prop is updated the secret is correctly
regenerated but the database is not updated because the
`MasterUserPassword` prop does not change. Moreover the connection
fields created by the attachment are lost because the
`AWS::SecretsManager::SecretAttachment` doesn't "rerun" (it references
the same secret).

Introduce `fromGeneratedSecret()` and `fromPassword()` static methods
in `Credentials`. When used the `MasterUsername` prop of the database
references the username as a string instead of a dynamic reference to
the username field of the secret. Moreover the logical id of the secret
is a hash of its customization options. This way the secret gets replaced
when it's customized, the database gets updated and the attachement reruns.
This without updating the `MasterUsername` prop, avoiding a replacement
of the database.

For instances that were created from a snapshot the `MasterUsername` prop
is not specified so there's no replacement risk. Add a new static method
`fromGeneratedSecret()` in `SnapshotCredentials` to replace the secret
when its customization options change (also hash in logical id).

Deprecate `Credentials.fromUsername()` but keep it as default to avoid
a breaking change that would replace existing databases that were not
created from a snapshot.

Deprecate `SnapshotCredentials.fromGeneratedPassword()` which doesn't
replace the secret when customization options are changed.

Closes #11040

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…butes() (#11266)

Follow Up from #10879

BREAKING CHANGE: all `fromResourceName()` methods in the AppMesh module have been replaced with `fromResourceAttributes()`

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
I was a bit confused by the README at first. I believe this change has been overlooked when moving from .env to .venv 10 days ago in a4a41b5.

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…440 (#11349)

Bumps [parcel](https://github.com/parcel-bundler/parcel) from 2.0.0-nightly.438 to 2.0.0-nightly.440.
- [Release notes](https://github.com/parcel-bundler/parcel/releases)
- [Changelog](https://github.com/parcel-bundler/parcel/blob/v2/CHANGELOG.md)
- [Commits](https://github.com/parcel-bundler/parcel/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
…11277)

Follows the pattern set in #10879 and transitions the VirtualRouter listener to use protocol specific listeners.

BREAKING CHANGE: VirtualRouter's Listeners are no longer a struct; use the static factory methods of the `VirtualNodeListener` class to obtain instances of them
* **appmesh:** VirtualRouter accepts a list of listeners instead of a single listener

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…11207)

AppMesh is launched in `eu-south-1` region and has a separate account for storing Envoy images in ECR.
Updated the ECR account information.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Bumps [fast-check](https://github.com/dubzzz/fast-check) from 2.6.0 to 2.6.1.
- [Release notes](https://github.com/dubzzz/fast-check/releases)
- [Changelog](https://github.com/dubzzz/fast-check/blob/master/CHANGELOG.md)
- [Commits](dubzzz/fast-check@v2.6.0...v2.6.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.785.0 to 2.787.0.
- [Release notes](https://github.com/aws/aws-sdk-js/releases)
- [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md)
- [Commits](aws/aws-sdk-js@v2.785.0...v2.787.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Bumps [diff](https://github.com/kpdecker/jsdiff) from 4.0.2 to 5.0.0.
- [Release notes](https://github.com/kpdecker/jsdiff/releases)
- [Changelog](https://github.com/kpdecker/jsdiff/blob/master/release-notes.md)
- [Commits](kpdecker/jsdiff@v4.0.2...v5.0.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Bumps [ts-jest](https://github.com/kulshekhar/ts-jest) from 26.4.3 to 26.4.4.
- [Release notes](https://github.com/kulshekhar/ts-jest/releases)
- [Changelog](https://github.com/kulshekhar/ts-jest/blob/master/CHANGELOG.md)
- [Commits](kulshekhar/ts-jest@v26.4.3...v26.4.4)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Fixes #11174

As per the issue details, this uses the correct metric names as described in [the documentation](https://docs.aws.amazon.com/elasticsearch-service/latest/developerguide/es-managedomains-cloudwatchmetrics.html).

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…442 (#11361)

Bumps [parcel](https://github.com/parcel-bundler/parcel) from 2.0.0-nightly.440 to 2.0.0-nightly.442.
- [Release notes](https://github.com/parcel-bundler/parcel/releases)
- [Changelog](https://github.com/parcel-bundler/parcel/blob/v2/CHANGELOG.md)
- [Commits](https://github.com/parcel-bundler/parcel/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Adds `fromLookup()` methods to both Application and Network load balancers as well as their listeners.

Closes #11088 

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
The build environment for the ShellScriptAction is hardcoded and set to LINUX_4_0.

This fix allows the build environment to be specified in the ShellScriptAction
fixes #10919



----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
This adds support for the credentials_process feature.

Using the aws-sso-credential-process utility you can also use AWS SSO with this feature

This should fix #3008 
----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
The `lambda-nodejs` package sometimes exhibits a coverage drop
on the build server, failing the build with the following error:

```
@aws-cdk/aws-lambda-nodejs: Jest: "global" coverage threshold for branches (60%) not met: 53.75%
```

Repeating the build again typically fixes the error.

I could not reproduce the issue locally, "on my machine" the build
just works with sufficient coverage.

The current behavior is a drain on PR builds, so I'm lowering the
coverage requirement so builds don't fail on unrelated changes.
…-in (#10874)

In Python, `id()` is a builtin function that can cause linter warnings and some confusion while developing stacks in Python. This change simply renames `id` to `stack_id` in the init template.
)

fixes #10784


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
If an email address for cognito email settings contains a non-ascii character, the cognito cdk package applies punycode encoding, which cloudformation will pick up properly.

This pull request adds the [`punycode`](https://github.com/bestiejs/punycode.js) package to the dependencies, as previously discussed in #8473. The package occupies 42KB in `node_modules` after installation. I am not sure if I configured the package correctly to be included in the cdk build. The project itself added `punycode` to `aws-cdk-lib` and `monocdk`, I had to re-run `yarn install` in these packages.

Unit- and integration tests are added, but notice that I could not manage a manual e2e test, I think I need to own a domain with non-ascii letters to do so, which I currently do not.

A similar approach as in this PR will also be interesting for other cdk packages which deal with email addresses and domains, like hosted zones.

According to the [Cfn Domain Name Format Docs](https://docs.aws.amazon.com/Route53/latest/DeveloperGuide/DomainNameFormat.html), characters with code point below 33, or in `[127, 255]` should be escaped with an octal code, which is not applied in this pull request.

Fixes #8473

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Khufu-I and others added 21 commits November 9, 2020 16:18
CompositeAlarm ARN separator should be `:` instead of `/`

Fixes #11184

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Also allow adding resource server via `userPool.addResourceServer`


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…integrations' module (#11339)

We now have a `@aws-cdk/aws-apigatewayv2-integrations` module that is
intended to hold all integration of HTTP APIs with other services.

In line with this, move `LambdaProxyIntegration` and
`HttpProxyIntegration` classes to this module.

BREAKING CHANGE: `LambdaProxyIntegration` and `HttpProxyIntegration`
classes have moved to the `@aws-cdk/aws-apigatewayv2-integration` module.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.787.0 to 2.788.0.
- [Release notes](https://github.com/aws/aws-sdk-js/releases)
- [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md)
- [Commits](aws/aws-sdk-js@v2.787.0...v2.788.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
…443 (#11383)

Bumps [parcel](https://github.com/parcel-bundler/parcel) from 2.0.0-nightly.442 to 2.0.0-nightly.443.
- [Release notes](https://github.com/parcel-bundler/parcel/releases)
- [Changelog](https://github.com/parcel-bundler/parcel/blob/v2/CHANGELOG.md)
- [Commits](https://github.com/parcel-bundler/parcel/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Allow to customise different view types in GraphWidget. The default behavior is backward compatible.

Closes #11063

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…DataSource (#11185)

BackedDataSource make Grantable HttpDataSource.  I think this pr allows better use of the iam config: #10171

fixes #11183
)

Resolves #11305 

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…kStage addApplication() (#11376)

Currently there is no way of reordering stages/actions once they are created. Getting to the Prepare and Execute change set actions while adding an application stage, we identified a use case within our company that needs one or more intermediary sequential actions before actually executing the change set. Moreover, I can imagine there can be plenty of use cases with similar requirements of having certain actions to fulfil between the Prepare and actual Execution of the Change Sets.

To resolve the problem, I suggest extending the AddStageOptions interface with an optional property representing the number of intermediary actions you need room for.

Closes #11333


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Patterns in `.dockerignore` used to be interpreted as globs, which has different behavior from how `.dockerignore` is supposed to be interpreted. Specifically it was hard to properly ignore whole directories at once (such as a large `node_modules`).

This PR adds an `ignoreMode` flag which controls how the ignore patterns should be interpreted. The choices are:

* `IgnoreMode.GLOB` - interpret as before
* `IgnoreMode.GIT` - interpret as in `.gitignore`
* `IgnoreMode.DOCKER` - interpret as in `.dockerignore`

Users might have dependencies on the current behavior, which is why the default `ignoreMode` depends on a feature flag: `@aws-cdk/aws-ecr-assets:dockerIgnoreSupport` (which is set to `true` for new projects) enables the new, correct Docker-ignore behavior by default.

----

Closes #10921

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…1389)

Backported from 9299d4e

The pack script defensively checks if there are any artifacts versioned 0.0.0 under the `dist/` directory. Python artifacts are named like so: `aws_cdk.aws_iotanalytics-2.0.0a0-py3-none-any.whl` and since by default grep uses regular expressions, this matches `grep 0.0.0`.

The fix is to use `-F` so that the search string is not treated as regex.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Hi this is a try to fix #10601 
I didn't add proper documentation and tests now because discussions about how to implement that are stale so I've decided to push it forward by creating this PR

if you guys think this is a good approach I will proceed with the unit tests and proper documentation

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Bumps [promptly](https://github.com/moxystudio/node-promptly) from 3.1.0 to 3.2.0.
- [Release notes](https://github.com/moxystudio/node-promptly/releases)
- [Changelog](https://github.com/moxystudio/node-promptly/blob/master/CHANGELOG.md)
- [Commits](moxystudio/node-promptly@v3.1.0...v3.2.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
…ate (#11375)

Currently, ubergen excludes packages that are marked as 'private'.
In v2, all packages under `@aws-cdk/` will be marked as 'private'.
This will mean that 'monocdk' and 'aws-cdk-lib' modules will have no
packages included.

Introduce another field `ubergen.exclude` in `package.json` that will be
explicitly exclude packages from the monolithic modules.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…11227)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Bumps [nock](https://github.com/nock/nock) from 13.0.4 to 13.0.5.
- [Release notes](https://github.com/nock/nock/releases)
- [Changelog](https://github.com/nock/nock/blob/main/CHANGELOG.md)
- [Commits](nock/nock@v13.0.4...v13.0.5)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.788.0 to 2.789.0.
- [Release notes](https://github.com/aws/aws-sdk-js/releases)
- [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md)
- [Commits](aws/aws-sdk-js@v2.788.0...v2.789.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
In the v2 branch, `aws-cdk-lib` will be made public. Hence, include
attributions for these.

Update the pkglint rule to also check `aws-cdk-lib` in the `master`
branch.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
@aws-cdk-automation aws-cdk-automation added the pr/no-squash This PR should be merged instead of squash-merging it label Nov 11, 2020
@njlynch njlynch requested a review from a team November 11, 2020 11:52
@njlynch njlynch self-assigned this Nov 11, 2020
@aws-cdk-automation
Copy link
Collaborator Author

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject6AEA49D1-qxepHUsryhcu
  • Commit ID: a9e40fa
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@mergify
Copy link
Contributor

mergify bot commented Nov 11, 2020

Thank you for contributing! Your pull request will be updated from master and then merged automatically without squashing (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit eb6f3a9 into release Nov 11, 2020
@mergify mergify bot deleted the bump/1.73.0 branch November 11, 2020 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-squash This PR should be merged instead of squash-merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.