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

Added Kerberos support (depending on Authen::Krb5) #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

darksoul42
Copy link

It adds a dependency on Authen::Krb5, and if "--kerberos" is added, switches the driver from Sybase to ODBC so it can use the SQL native client, which actually has support for the trusted connection functionality.

The Kerberos authentication bit creates the credential cache that is to be used by the SQL client. (Which other modules such as Authen::Krb5::Simple do not create)

This is the code as we are using it in my organization. It has certainly room for some improvement (the driver string for the native client, or allowing use of keytabs), but I wanted to bring to attention that there was a need for such a feature.

It makes the script usable in this fashion :
$ /tmp/check_mssql_health --server .MYORG.REALM --username --password '...' --database <...> --kerberos MYORG.REALM --mode transactions
OK - Surveillance has 6.0221 transactions / sec | Surveillance_transactions_per_sec=6.0221;10000;50000

@FoneBoneDE
Copy link

This is a very old issue. Why is it not integrated in the main product and updatet?
There is mentioned "SQL Server Native Client 11.0" in "Server.pm" actually we have Microsoft ODBC 18 (see https://learn.microsoft.com/en-us/sql/connect/odbc/linux-mac/installing-the-microsoft-odbc-driver-for-sql-server?view=sql-server-ver16&tabs=alpine18-install%2Calpine17-install%2Cdebian8-install%2Credhat7-13-install%2Crhel7-offline#11).
Could anyone check this?

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.

2 participants