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

Do not try to unlock DB if connection failed #677

Merged
merged 1 commit into from
Aug 18, 2020

Conversation

dolezel
Copy link
Contributor

@dolezel dolezel commented Aug 18, 2020

Closes #654

@dolezel dolezel merged commit 813c956 into master Aug 18, 2020
@dolezel dolezel deleted the fix_double_error_on_connection_fail branch August 18, 2020 07:25
ryands17 added a commit to ryands17/node-pg-migrate that referenced this pull request Sep 21, 2020
* Update all non-major dependencies

* Update typescript-eslint monorepo to v3 (major) (salsita#634)

* Update typescript-eslint monorepo to v3

* Fixing lint

Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: Jan Dolezel <dolezel.jan@gmail.com>

* Update all non-major dependencies

* Keep this bind in logger methods (salsita#638)

* Updated changelog

* 5.0.1

* Fix alter column collation (salsita#641)

* Updated changelog

* 5.0.2

* Ability to specify own template file (salsita#642)

* Updated changelog

* 5.1.0

* Update all non-major dependencies

* Fix creating migration (salsita#646)

* Fix creating migration

* Check migration creation

* Fix yaml

* another method of counting files

* Updated changelog

* 5.1.1

* Update all non-major dependencies

* Update dependency mocha to v8 (salsita#650)

Co-authored-by: Renovate Bot <bot@renovateapp.com>

* Update all non-major dependencies

* View options (salsita#656)

* View options

* Fix migration

* Fix creating SQL

* Updated changelog

* 5.2.0

* Update all non-major dependencies

* Update all non-major dependencies

* Update all non-major dependencies

* Update dependency @types/mocha to v8 (salsita#662)

Co-authored-by: Renovate Bot <bot@renovateapp.com>

* let console.error display all the properties on an err object, which can include useful debugging information. (salsita#660)

* Updated changelog

* 5.3.0

* Updated dependencies (salsita#663)

* Update all non-major dependencies

* Update all non-major dependencies

* Update all non-major dependencies

* docs: fixed small error (salsita#666)

* Update isExternalClient check (salsita#664)

* Update isExternalClient check

Problem using pg-promise - adopting the method used here: https://github.com/voxpelli/node-connect-pg-simple/blob/master/index.js#L95

* Update db.ts

* Do not use 'any'

Co-authored-by: Jan Dolezel <dolezel.jan@gmail.com>

* Updated changelog

* 5.4.0

* Update all non-major dependencies

* Regenerated package-lock.json (salsita#674)

* Allow opclass and sort per column in indexes (salsita#671)

* Allow opclass and sort per column in indexes

* Fixes for using schemalize

* Fix double escaping

* Using structured object instead of array

* Updated changelog

* 5.5.0

* Update all non-major dependencies

* Update all non-major dependencies Docker tags

* Do not try to unlock DB if connection failed (salsita#677)

* Updated changelog

* 5.5.1

* PgLiteral can be serialized (salsita#678)

* Updated changelog

* 5.6.0

* Using pg client in custom runner test (salsita#679)

* Update all non-major dependencies

* Update dependency ts-node to v9 (salsita#681)

Co-authored-by: Renovate Bot <bot@renovateapp.com>

* Update dependency typescript to v4 (salsita#682)

* Update dependency typescript to v4

* Refactored type computing

* Regenerated dependencies

Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: Jan Dolezel <dolezel.jan@gmail.com>

* Update all non-major dependencies

* Update all non-major dependencies

* Update all non-major dependencies

* Update dependency yargs to v16 (salsita#691)

Co-authored-by: Renovate Bot <bot@renovateapp.com>

* Eslint config (salsita#692)

* Simplified eslint config, eslint fixes

* Simplify prettier config

* Update all non-major dependencies

* Bump node-fetch from 2.6.0 to 2.6.1 (salsita#689)

Bumps [node-fetch](https://github.com/bitinn/node-fetch) from 2.6.0 to 2.6.1.
- [Release notes](https://github.com/bitinn/node-fetch/releases)
- [Changelog](https://github.com/node-fetch/node-fetch/blob/master/docs/CHANGELOG.md)
- [Commits](node-fetch/node-fetch@v2.6.0...v2.6.1)

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

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

Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Jan Dolezel <dolezel.jan@gmail.com>
Co-authored-by: Joel Hillacre <joel@403forbidden.ca>
Co-authored-by: 0xflotus <0xflotus@gmail.com>
Co-authored-by: Jon <jondb@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

Could not connect to postgres: Error: Client has already been connected. You cannot reuse a client.
1 participant