Conversation

julienschmidt

@julienschmidtjulienschmidt added this to the v1.4.0 milestone May 27, 2018
@julienschmidtjulienschmidt self-assigned this May 27, 2018
@julienschmidtjulienschmidt mentioned this pull request May 27, 2018
CHANGELOG.md Outdated

Bugfixes:

- Use the DSN hostename as TLS default ServerNname if `tls=true` (#564, #718)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hostename ServerNname typos?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, fixed.

CHANGELOG.md Outdated
New Features:

- Multi-Results support (#537)
- `rejectReadOnly` DNS option (#604)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DSN

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM except one comment I left

@julienschmidtjulienschmidt merged commit d523deb into master Jun 3, 2018
@julienschmidtjulienschmidt deleted the changelog branch June 3, 2018 12:45
@methanemethane mentioned this pull request Dec 11, 2019
Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet

Successfully merging this pull request may close these issues.