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.

Sign up for GitHub

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

Jump to bottom

TDS8 - Add "Strict", "mandatory" and "optional" options for encrypt #136

Closed
apoorvdeshmukh opened this issue Jul 20, 2023 · 0 comments · Fixed by #125
Closed

TDS8 - Add "Strict", "mandatory" and "optional" options for encrypt #136

apoorvdeshmukh opened this issue Jul 20, 2023 · 0 comments · Fixed by #125
Assignees
@apoorvdeshmukh
Labels
Area - connections Issues affecting connection security or strings security TDS8
Milestone
TDS 8.0 P1

Comments

@apoorvdeshmukh
Copy link

For using TDS8 support needs to be added for connection property encrypt=strict.
Additional options such as 'mandatory', 'optional' also need to be added.

For strict option, ALPN sequence has to be set to "tds/8.0" in TLS setup.
More details here: TDS8
@apoorvdeshmukh apoorvdeshmukh self-assigned this Jul 20, 2023
@apoorvdeshmukh apoorvdeshmukh added this to the TDS 8.0 P1 milestone Jul 20, 2023
@apoorvdeshmukh apoorvdeshmukh mentioned this issue Jul 27, 2023
Merged
@dlevy-msft dlevy-msft added Area - connections Issues affecting connection security or strings security labels Aug 15, 2023
apoorvdeshmukh added a commit that referenced this issue Aug 31, 2023
@apoorvdeshmukh
Add TDS8 support
2c72afb 
This commit fixes #136.  Adds support for TDS8. TDS8 connection
can now be used by specifying encrypt=strict. TrustServerCertificate=true
will not come into effect when encrypt is set to 'strict'.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@apoorvdeshmukh apoorvdeshmukh

Labels
Area - connections Issues affecting connection security or strings security TDS8
Projects
None yet
Milestone
TDS 8.0 P1
Development

Successfully merging a pull request may close this issue.

TDS8: Accept additional values for encrypt microsoft/go-mssqldb
2 participants
@apoorvdeshmukh @dlevy-msft