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

setupTLSCommonName redeclared #706

Closed
wonsikin opened this issue Dec 12, 2021 · 1 comment
Closed

setupTLSCommonName redeclared #706

wonsikin opened this issue Dec 12, 2021 · 1 comment

Comments

@wonsikin
Copy link

wonsikin commented Dec 12, 2021

Describe the bug
Our Services are using this library and we have a static analysis in the CI. today we found the CI build failed.

# github.com/denisenkom/go-mssqldb/msdsn
/go/src/github.com/denisenkom/go-mssqldb/msdsn/conn_str_go115pre.go:7:6: setupTLSCommonName redeclared in this block
	previous declaration at /go/src/github.com/denisenkom/go-mssqldb/msdsn/conn_str_go115.go:11:57

It looks like this commit add an extra setupTLSCommonName

d99d7d5

cc: @kardianos

@kardianos
Copy link
Collaborator

I feel like an idiot. I corrected by adding in the older style build flags. It should be fixed now.

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

No branches or pull requests

2 participants