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

chore(deps): change to github.com/microsoft/go-mssqldb@v1.6.0 #5419

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

Jarnpher553
Copy link
Contributor

@Jarnpher553 Jarnpher553 commented Jul 19, 2024

change to github.com/microsoft/go-mssqldb@v1.6.0

Proposed changes

When my project uses github.com/microsoft/go-mssqldb, it conflicts with github.com/denisenkom/go-mssqldb used by nuclei, resulting in the error "panic: sql: Register called twice for driver mssql". The main reason for this issue is that both go-mssqldb packages register a driver with the same name mssql, so I would like to update denisenkom/go-mssqldb to microsoft/go-mssqldb.

Checklist

  • Pull request is created against the dev branch
  • All checks passed (lint, unit/integration/regression tests etc.) with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@GeorginaReeder
Copy link

Thanks for your contribution @Jarnpher553 , we appreciate it!

We also have a Discord server, which you’re more than welcome to join. It's a great place to connect with fellow contributors and stay updated with the latest developments!

Copy link
Member

@tarunKoyalwar tarunKoyalwar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm !

@Jarnpher553
Copy link
Contributor Author

Something went wrong, Windows testing failed. 😓

@tarunKoyalwar
Copy link
Member

tarunKoyalwar commented Jul 19, 2024

no worries @Jarnpher553 , it seems to be flaky , the integration test passed in debug mode

@ehsandeep ehsandeep merged commit f930e9a into projectdiscovery:dev Jul 24, 2024
11 of 12 checks passed
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

Successfully merging this pull request may close these issues.

4 participants