Skip to content

Creating importers for the new SQL Log Scout Connectivity Related collector - #560

Merged
JosephPilov-MSFT (PiJoCoder) merged 2 commits into
masterfrom
ansavio_newconnectivityimporters_logscout
Sep 15, 2026
Merged

JosephPilov-MSFT (PiJoCoder) merged 2 commits into
masterfrom
ansavio_newconnectivityimporters_logscout

Conversation

@asavioliMSFT

@asavioliMSFT Andre Savioli (asavioliMSFT) commented Sep 11, 2026 •

Copy link
Copy Markdown
Contributor

This pull request adds support for importing and testing a new set of SQL LogScout connectivity and security collector rowsets. These rowsets cover TLS protocols, SQL client drivers, SPNs, service accounts, client aliases, and server certificates, and are now defined in both TextRowsets.xml files. Additionally, new regression tests ensure the import engine correctly parses these fixed-width, sqlcmd-style outputs, especially for edge cases like wide trailing columns and renamed headers.

The following tables have been created

select * from [tbl_tls_ssl_protocols]
select * from [tbl_sql_client_drivers]
select * from [tbl_sql_spn_service_accounts]
select * from [tbl_sql_spn_registered]
select * from [tbl_sql_spn_suggested]
select * from [tbl_sql_service_summary]
select * from [tbl_sql_service_account_delegation]
select * from [tbl_kerberos_local_policy]
select * from [tbl_sql_instance_network_security]
select * from [tbl_sql_client_aliases]
select * from [tbl_sql_server_certificates]
select * from [tbl_sql_server_certificate_key_acl]

Rowset Definitions and Import Support:

  • Added definitions for new SQL LogScout connectivity and security collector rowsets in both RowsetImportEngine/TextRowsets.xml and sqlnexus/TextRowsets.xml. These include rowsets for TLS/SSL protocols, SQL client drivers, SPN service accounts and registrations, service summaries, Kerberos policy, client aliases, server certificates, and related key ACLs. Each rowset specifies known columns and their types/widths to support fixed-width import.

Testing and Validation:

  • Introduced a new test class ConnectivityCollectorRowsetTests in ConnectivityCollectorRowsetTests.cs to provide regression coverage for these rowsets. The tests verify that:
    • Column headers are mapped in the correct order.
    • Widened trailing columns (such as the Path field in client drivers) are fully captured, preventing silent truncation.
    • Renamed headers (e.g., ServiceAccountName instead of StartName) are correctly recognized and parsed.

@PiJoCoder

Copy link
Copy Markdown
Collaborator

Tested an import and I got schema for all these tables (some of them were empty due to empty source data)

use sqlnexus
go
select * from tbl_tls_ssl_protocols
select * from tbl_sql_client_drivers
select * from tbl_sql_spn_service_accounts
select * from tbl_sql_spn_registered
select * from tbl_sql_spn_suggested
select * from tbl_sql_service_summary
select * from tbl_sql_service_account_delegation
select * from tbl_kerberos_local_policy
select * from tbl_sql_instance_network_security
select * from tbl_sql_client_aliases
select * from tbl_sql_server_certificates
select * from tbl_sql_server_certificate_key_acl

@PiJoCoder
JosephPilov-MSFT (PiJoCoder) merged commit 7616caa into master Sep 15, 2026
2 of 3 checks passed
@PiJoCoder
JosephPilov-MSFT (PiJoCoder) deleted the ansavio_newconnectivityimporters_logscout branch September 15, 2026 18:54
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