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

Keyword not supported: 'trust server certificate' #2899

Open
cakeezer opened this issue Apr 11, 2024 · 14 comments
Open

Keyword not supported: 'trust server certificate' #2899

cakeezer opened this issue Apr 11, 2024 · 14 comments
Labels
GS Tutorial Bug from customer using Getting Started Tutorial Priority:0 Work that we can't release without

Comments

@cakeezer
Copy link

cakeezer commented Apr 11, 2024

System Information (please complete the following information):

  • Model Builder or CLI Version: 3.0.0
  • Visual Studio Version (if applicable): 2022 17.9.4

Describe the bug

  • On which page of the tutorial did you run into an issue (URL): Data - Connect to SQL Server
  • Clear description of the problem: After selecting a SQL server in the dropdown, we use Windows Auth and Trusted Certs. Enabling Trusted Certs causes this error to be thrown.

To Reproduce
Steps to reproduce the behavior:

  1. Go to Add Machine Learning from right clicking project
  2. Pick your scenario
  3. Pick your Environment
  4. On the Data selection page, select SQL Server, then Choose your data source
  5. Select your data source, and click the Trusted Certifications checkbox. Leaving this option unchecked results in a connection issue with SQL Server.
  6. Click OK and the error is displayed

Expected behavior
I should connect to the SQL server without being presented with an error message

Screenshots
image
image

Log file
demomodel-7NLN9K.txt

@cakeezer cakeezer added the GS Tutorial Bug from customer using Getting Started Tutorial label Apr 11, 2024
@v-Hailishi
Copy link

I can also repro this bug on the latest main build 17.18.2.2421201.
image

@LittleLittleCloud
Copy link
Contributor

@zewditu can you take a look?

@rkpatrick
Copy link

I have this issue as well. As a workaround, I'm exporting some of my data to CSV and building from that.

@LittleLittleCloud
Copy link
Contributor

Thanks I will try reproducing this

@nmqh
Copy link

nmqh commented Apr 23, 2024

I'm having same issue.

@LittleLittleCloud LittleLittleCloud added the Priority:0 Work that we can't release without label Apr 23, 2024
@LittleLittleCloud
Copy link
Contributor

Update

I'm able to reproduce this issue on my end, and when I disable trust server certificate on loading sql server page the error disappear. So maybe @rkpatrick @cakeezer @nmqh this can be a workaround?

@rkpatrick
Copy link

In my environment, if I uncheck that box, I can't see any databases in the dropdown

@LittleLittleCloud
Copy link
Contributor

After some investigation, the root cause seems to be caused by this issue. My assumption is VS Dev17 upgrades to use Microsoft.Data.SqlClient while model builder still uses System.Data.SqlClient

I'll push a fix on model builder side to fix this issue

@cakeezer
Copy link
Author

cakeezer commented Apr 23, 2024 via email

@v-Hailishi
Copy link

The bug has been fixed on the latest main build 17.18.4.2422301.
image
image

@VipinVenugopal1
Copy link

17.18.4.2422301

Link to download this latest version?

@VipinVenugopal1
Copy link

The bug has been fixed on the latest main build 17.18.4.2422301.
when would this version be available for downloading ?

@LittleLittleCloud
Copy link
Contributor

This version has been shipped with latest vs 2022 preview.

@JakeRadMSFT should we make another release to gallery

@ArlenLi
Copy link

ArlenLi commented Jul 30, 2024

hey @LittleLittleCloud , do we know when a non-preview release containing this fix would be published?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GS Tutorial Bug from customer using Getting Started Tutorial Priority:0 Work that we can't release without
Projects
None yet
Development

No branches or pull requests

7 participants