Skip to content

[microsoft_sqlserver] change default username value to be properly encoded#16191

Merged
mykola-elastic merged 2 commits intomainfrom
mykola-elastic-patch-2
Dec 3, 2025
Merged

[microsoft_sqlserver] change default username value to be properly encoded#16191
mykola-elastic merged 2 commits intomainfrom
mykola-elastic-patch-2

Conversation

@mykola-elastic
Copy link
Copy Markdown
Contributor

@mykola-elastic mykola-elastic commented Dec 2, 2025

As per field description:
"Domain users: Pre-encode username when passing backslash e.g. {domain}%5C{username} instead of {domain}\\{username}"

Using %5C (encoded backslash) in the username string default value to be consistent with the requirement.

Also, see #15161 (comment)

Proposed commit message

See title.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Related issues

As per field description:
"Domain users: Pre-encode username when passing backslash e.g. {domain}%5C{username} instead of {domain}\\\\{username}" Using %5C (encoded backslash) in the connection string default value
@mykola-elastic mykola-elastic changed the title [microsoft_sqlserver] change default host string value [microsoft_sqlserver] change default username value Dec 2, 2025
@mykola-elastic mykola-elastic self-assigned this Dec 2, 2025
@mykola-elastic mykola-elastic added Integration:microsoft_sqlserver Microsoft SQL Server Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] enhancement New feature or request labels Dec 2, 2025
@mykola-elastic mykola-elastic marked this pull request as ready for review December 2, 2025 09:50
@mykola-elastic mykola-elastic requested a review from a team as a code owner December 2, 2025 09:50
@mykola-elastic mykola-elastic changed the title [microsoft_sqlserver] change default username value [microsoft_sqlserver] change default username value to be properly encoded Dec 2, 2025
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

cc @mykola-elastic

@mykola-elastic mykola-elastic merged commit 71c03a9 into main Dec 3, 2025
7 checks passed
@mykola-elastic mykola-elastic deleted the mykola-elastic-patch-2 branch December 3, 2025 11:29
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

Package microsoft_sqlserver - 2.14.1 containing this change is available at https://epr.elastic.co/package/microsoft_sqlserver/2.14.1/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Integration:microsoft_sqlserver Microsoft SQL Server Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants