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

Rds configurationoptions support #31

Merged
merged 2 commits into from
Mar 23, 2020
Merged

Rds configurationoptions support #31

merged 2 commits into from
Mar 23, 2020

Conversation

cklam2
Copy link
Contributor

@cklam2 cklam2 commented Mar 23, 2020

Pull request that solves #8:
Support for argument 'serviceConfigOptions' (AWS.ConfigurationOptions)

The change also requires an update in TypeORM. That part is already implemented but no pull request made yet. Needs to wait till this changed is merged into typeorm-aurora-data-api-driver.

Once merged I'll create a pull request in TypeORM for the changes in this branch.

@cklam2
Copy link
Contributor Author

cklam2 commented Mar 23, 2020

Example usage:

createConnection({
    name,
    type: 'aurora-data-api',
    ...
    serviceConfigOptions: {
        credentials,
        maxRetries: 3,
        retryDelayOptions: {
           base: 5000,
        }
    }
  });

@ArsenyYankovsky
Copy link
Owner

Looks good to me. I will however merge into a branch first to let it run the functional tests.

@ArsenyYankovsky ArsenyYankovsky changed the base branch from master to rds-configurationoptions-support March 23, 2020 16:49
@ArsenyYankovsky ArsenyYankovsky merged commit 56da11e into ArsenyYankovsky:rds-configurationoptions-support Mar 23, 2020
ArsenyYankovsky added a commit that referenced this pull request Mar 23, 2020
* Rds configurationoptions support (#31)

* build(deps): bump acorn from 5.7.3 to 5.7.4

Bumps [acorn](https://github.com/acornjs/acorn) from 5.7.3 to 5.7.4.
- [Release notes](https://github.com/acornjs/acorn/releases)
- [Commits](acornjs/acorn@5.7.3...5.7.4)

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

* Support for passing AWS.ConfigurationOptions towards data-api-client

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Service Config options fix

* Fix passing options

Co-authored-by: cklam2 <cklam2@outlook.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@ArsenyYankovsky
Copy link
Owner

@cklam2 The changes are released with version 1.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants