alexbrainman / odbc

odbc driver written in go
BSD 3-Clause "New" or "Revised" License
359 stars 141 forks source link

MSSQL tests fail when connecting to modern MSSQL Server #38

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
More details here:

http://stackoverflow.com/a/19904973/385979

FreeTDS apparently defaults to an old version of the TDS protocol. It might be 
good to add a "TDS_Version" command line parameter to the mssql_test.go since 
the cryptic error message doesn't explain what is wrong with the connection.

For a short-term fix the attached patch seems to fix the problem by letting the 
server decide the protocol version.

Original issue reported on code.google.com by jeffv...@gmail.com on 28 Mar 2014 at 1:24

Attachments:

GoogleCodeExporter commented 9 years ago
jeffvanb,

Thank you for suggestion, but the accepted solution at 
http://stackoverflow.com/questions/19898932/golang-macports-freetdsunixodbc-odbc
-driver-manager-data-source-name-not-fo/19904973#19904973 looks fine to me as 
is. Do you still have any issues / error messages after you've configured your 
system as described? I don't see how your suggested change will be helpful for 
anyone. Please, explain, if you think I am wrong about it.

Thank you.

Alex

Original comment by alex.bra...@gmail.com on 28 Mar 2014 at 4:15

GoogleCodeExporter commented 9 years ago
This suggested change is to improve support for SQLServer 2012 in the tests.

After installing the ODBC driver with the instructions in the Wiki it suggests 
running the mssql tests to verify that your connection works. As the tests are 
written right now they will fail on MS SQLServer 2012. The error message does 
not give any clue that the problem is related to the TDS version, so it may be 
difficult for developers to figure out why the driver is not working.

Specifying "TDS_Version=auto" in the connection string improves compatibility 
with SQLServer 2012 and I don't believe it will hurt compatibility with older 
servers.

Original comment by jeffv...@gmail.com on 28 Mar 2014 at 6:04

GoogleCodeExporter commented 9 years ago
> ... Specifying "TDS_Version=auto" in the connection string improves 
compatibility with SQLServer 2012 ...

It is difficult for me to see the problem without seeing error messages. 
Please, provide error messages before and after, to make your point.

Thank you.

Alex

Original comment by alex.bra...@gmail.com on 29 Mar 2014 at 1:19

GoogleCodeExporter commented 9 years ago
Closing. Please comment, if you think I should do something here. Thank you.

Alex

Original comment by alex.bra...@gmail.com on 11 Apr 2014 at 6:55