Skip to content

Fix ssl=false parsing - #39

Closed
sonicdoe wants to merge 1 commit into
iceddev:masterfrom
kontist:ssl=false
Closed

Fix ssl=false parsing#39
sonicdoe wants to merge 1 commit into
iceddev:masterfrom
kontist:ssl=false

Conversation

@sonicdoe

Copy link
Copy Markdown
Contributor

ssl was mistakenly set to the string "false" instead of the Boolean false.

ssl was mistakenly set to the string "false" instead of the Boolean false.
@coveralls

coveralls commented Sep 24, 2020

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling 7ad6ba4 on kontist:ssl=false into 5233b3e on iceddev:master.

@hjr3

hjr3 commented Nov 29, 2020

Copy link
Copy Markdown
Contributor

Please submit this to https://github.com/brianc/node-postgres

@hjr3 hjr3 closed this Nov 29, 2020
@sonicdoe

Copy link
Copy Markdown
Contributor Author

It’s already submitted: brianc/node-postgres#2423 🙂

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.

3 participants