We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Hi everyone, I found an issue when building the string connection with sql on the module: sqlserver_data_source.py
When the connection is trusted, you dont need user and pass, so the solution could be: (the last two lines were added)
self.connection = pyodbc.connect( ("Trusted_Connection=YES;" if self.trusted_connection else "") + ("TrustServerCertificate=YES;" if self.trust_server_certificate else "") + ("Encrypt=YES;" if self.encrypt else "") + "DRIVER={" + self.driver + "};SERVER=" + self.host + "," + str(self.port) + ";DATABASE=" + self.database + (";UID=" + self.username if not self.trusted_connection else "") + (";PWD=" + self.password if not self.trusted_connection else "") )
Many thanks guys for the job done, i am starting to use it and i like it :)
The text was updated successfully, but these errors were encountered:
SAS-2143
Sorry, something went wrong.
Thank you for the report @rodrigomartinez1302 ! Would you like to open a PR with the fix? I'll be happy to merge it ASAP.
Hi, Is this issue fixed ?
Not yet, would you be able to open a PR for this @rodrigomartinez1302 ?
Fixed this in #2160
No branches or pull requests
Hi everyone, I found an issue when building the string connection with sql on the module: sqlserver_data_source.py
When the connection is trusted, you dont need user and pass, so the solution could be:
(the last two lines were added)
self.connection = pyodbc.connect(
("Trusted_Connection=YES;" if self.trusted_connection else "")
+ ("TrustServerCertificate=YES;" if self.trust_server_certificate else "")
+ ("Encrypt=YES;" if self.encrypt else "")
+ "DRIVER={"
+ self.driver
+ "};SERVER="
+ self.host
+ ","
+ str(self.port)
+ ";DATABASE="
+ self.database
+ (";UID=" + self.username if not self.trusted_connection else "")
+ (";PWD=" + self.password if not self.trusted_connection else "")
)
Many thanks guys for the job done, i am starting to use it and i like it :)
The text was updated successfully, but these errors were encountered: