Skip to content

Sql Obfuscator: Make use of sql obfuscation in aggregator to add db.sql.normalized property - #188

Merged
kodiakhq[bot] merged 18 commits into
mainfrom
jag/pipeline_obfuscate
Jan 6, 2024
Merged

Sql Obfuscator: Make use of sql obfuscation in aggregator to add db.sql.normalized property#188
kodiakhq[bot] merged 18 commits into
mainfrom
jag/pipeline_obfuscate

Conversation

@jaggederest

Copy link
Copy Markdown
Contributor

No description provided.

@changeset-bot

changeset-botBot commented Jan 5, 2024

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 318f5d8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@hyperdx/apiPatch
@hyperdx/appPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Comment threadpackages/api/src/utils/logParser.ts Outdated
Comment threadpackages/api/src/utils/logParser.ts Outdated
@wrn14897
wrn14897 marked this pull request as ready for review January 5, 2024 21:32

@wrn14897wrn14897 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@jaggederestjaggederest changed the title Sql Obfuscator: Make use of sql obfuscation in aggregator to add db.statement.obfuscated propertySql Obfuscator: Make use of sql obfuscation in aggregator to add db.sql.normalized propertyJan 6, 2024
@kodiakhq
kodiakhqBot merged commit deacf95 into mainJan 6, 2024
@kodiakhq
kodiakhqBot deleted the jag/pipeline_obfuscate branch January 6, 2024 01:35
jaggederest added a commit that referenced this pull request Jan 6, 2024
…add `db.sql.normalized` property (#188)"
This reverts commit deacf95.
knudtty pushed a commit that referenced this pull request Apr 16, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@jaggederest@MikeShi42@wrn14897