Skip to content

Conditionally compile with net.Dialer.KeepAlive option if go >= 1.3#94

Merged
denisenkom merged 1 commit into
denisenkom:masterfrom
linquize:keepalive
Mar 29, 2015
Merged

Conditionally compile with net.Dialer.KeepAlive option if go >= 1.3#94
denisenkom merged 1 commit into
denisenkom:masterfrom
linquize:keepalive

Conversation

@linquize
Copy link
Copy Markdown
Contributor

This make the library compile with go <= 1.2

denisenkom added a commit that referenced this pull request Mar 29, 2015
Conditionally compile with net.Dialer.KeepAlive option if go >= 1.3
@denisenkom denisenkom merged commit 471d1d6 into denisenkom:master Mar 29, 2015
@denisenkom
Copy link
Copy Markdown
Owner

Thanks

gabrielcorado pushed a commit to gravitational/go-mssqldb that referenced this pull request Mar 24, 2023
…om#94)

* Fixed denisenkom#93: servername was not supplied to TLS cfg

A change in fd44003 moved the code for setting p.Host down in the parse
function, resulting in an empty p.Host value at the time of the creation
of p.TLSConfig. I extracted the TLS parsing into a separate function and
moved the TLSConfig creation to the end of the parsing function.

* Added extra test scenario with hostnameincertificate parameter
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