The Wayback Machine - https://web.archive.org/web/20200906050634/https://github.com/brianc/node-postgres/pull/2284/
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

type(pg-connection-string): fix typings in ConnectionOptions #2284

Open
wants to merge 1 commit into
base: master
from

Conversation

@ssut
Copy link

ssut commented Jul 17, 2020

  • The ssl option can be either boolean or its own type consisting of cert, key, and ca.
  • Because as the documentation here says, any other query params should be preserved intact as well as types so I added [key: string]: any in ConnectionOptions.

Because without this I had to cast the result parsed as any type, or add { schema?: string } in everywhere with such conditions that require schema, I think this PR will help those in need.

…onOptions, and add support for any keys as other query params are preserved intact
@ssut ssut changed the title type(pg-connection-string): fix typings for ConnectionOptions type(pg-connection-string): fix typings in ConnectionOptions Jul 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.