Skip to content

Sql Obfuscation: Add experimental sql obfuscator (binary w/ipc) - #187

Closed
jaggederest wants to merge 6 commits into
mainfrom
jag/sql_obfuscation
Closed

Sql Obfuscation: Add experimental sql obfuscator (binary w/ipc)#187
jaggederest wants to merge 6 commits into
mainfrom
jag/sql_obfuscation

Conversation

@jaggederest

@jaggederestjaggederest commented Jan 5, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@changeset-bot

changeset-botBot commented Jan 5, 2024•
edited
Loading

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: f64b760

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

wrn14897
wrn14897 reviewed Jan 5, 2024
View reviewed changes
Comment threadpackages/api/src/gobin/src/sql_obfuscator.go
wrn14897
wrn14897 reviewed Jan 5, 2024
View reviewed changes
Comment threadpackages/api/src/gobin/src/sql_obfuscator.go
wrn14897
wrn14897 reviewed Jan 5, 2024
View reviewed changes
Comment threadpackages/api/src/utils/__tests__/sqlObfuscator.test.ts
wrn14897
wrn14897 reviewed Jan 5, 2024
View reviewed changes
Comment threadpackages/api/src/utils/sqlObfuscator.ts
wrn14897
wrn14897 reviewed Jan 5, 2024
View reviewed changes
Comment threadpackages/api/src/utils/sqlObfuscator.ts
wrn14897
wrn14897 reviewed Jan 5, 2024
View reviewed changes
Comment threadpackages/api/src/utils/sqlObfuscator.ts
wrn14897
wrn14897 reviewed Jan 5, 2024
View reviewed changes
Comment threadpackages/api/src/utils/sqlObfuscator.ts
@wrn14897
wrn14897 marked this pull request as ready for review January 5, 2024 21:21
@wrn14897

wrn14897 commented Jan 5, 2024

Copy link
Copy Markdown
Member

Looks good. can ship it once comments are resolved

@jaggederest

jaggederest commented Jan 5, 2024

Copy link
Copy Markdown
ContributorAuthor

Closing in favor of #188

@wrn14897
wrn14897 deleted the jag/sql_obfuscation branch April 6, 2024 06:24
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wrn14897wrn14897wrn14897 left review comments
@MikeShi42MikeShi42Awaiting requested review from MikeShi42
@svc-shorposvc-shorpoAwaiting requested review from svc-shorpo

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jaggederest@wrn14897