Skip to content

Conversation

@paulojmdias
Copy link
Member

Description

Fix for memory leak when using ObfuscateSQLStringWithOptions() in the obfuscate library

Similar fix done in SQL Server receiver and OracleDB Receiver

Link to tracking issue

Fixes #43076

Copy link
Contributor

@yaten2302 yaten2302 left a comment

Choose a reason for hiding this comment

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

LGTM!

@atoulme
Copy link
Contributor

atoulme commented Oct 6, 2025

@ishleenk17 and @antonblock please review as codeowners?

@ChrsMark ChrsMark added the ready to merge Code review completed; ready to merge by maintainers label Oct 6, 2025
@atoulme atoulme merged commit 861d0d8 into open-telemetry:main Oct 6, 2025
201 of 202 checks passed
@github-actions github-actions bot added this to the next release milestone Oct 6, 2025
@paulojmdias paulojmdias deleted the fix/43076 branch October 6, 2025 15:11
graphaelli pushed a commit to graphaelli/opentelemetry-collector-contrib that referenced this pull request Oct 6, 2025
…n-telemetry#43115)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

Fix for memory leak when using `ObfuscateSQLStringWithOptions()` in the
obfuscate library

Similar fix done in [SQL Server
receiver](https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/42303/files
) and [OracleDB
Receiver](open-telemetry#43096)

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#43076

---------

Signed-off-by: Paulo Dias <[email protected]>
Co-authored-by: Christos Markou <[email protected]>
mashhurs pushed a commit to mashhurs/opentelemetry-collector-contrib that referenced this pull request Oct 9, 2025
…n-telemetry#43115)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

Fix for memory leak when using `ObfuscateSQLStringWithOptions()` in the
obfuscate library

Similar fix done in [SQL Server
receiver](https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/42303/files
) and [OracleDB
Receiver](open-telemetry#43096)

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#43076

---------

Signed-off-by: Paulo Dias <[email protected]>
Co-authored-by: Christos Markou <[email protected]>
tommyers-elastic pushed a commit to tommyers-elastic/opentelemetry-collector-contrib that referenced this pull request Oct 10, 2025
…n-telemetry#43115)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

Fix for memory leak when using `ObfuscateSQLStringWithOptions()` in the
obfuscate library

Similar fix done in [SQL Server
receiver](https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/42303/files
) and [OracleDB
Receiver](open-telemetry#43096)

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#43076

---------

Signed-off-by: Paulo Dias <[email protected]>
Co-authored-by: Christos Markou <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Code review completed; ready to merge by maintainers receiver/postgresql waiting-for-code-owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[receiver/postgresql] Memory leak when using top queries

7 participants