Skip to content

Conversation

@hannahramadan
Copy link
Member

@hannahramadan hannahramadan commented Sep 15, 2025

Once sql-processor is published, this PR replaces references from the formerly named sql-obfuscation gem to the new sql-processor gem.

This should be merged 3rd in a series of 3 PRs:

  1. feat: introduce sql-processor gem #1673
  2. feat!: deprecate sql-obfuscation gem #1674
  3. feat: replace references sql-obfuscation -> sql-processor #1675

@github-actions
Copy link
Contributor

github-actions bot commented Nov 9, 2025

👋 This pull request has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the keep label to hold stale off permanently, or do nothing. If you do nothing this pull request will be closed eventually by the stale bot

@github-actions github-actions bot added the stale Marks an issue/PR stale label Nov 9, 2025
@arielvalentin
Copy link
Contributor

@hannahramadan are you going to resurrect this? I want to add some features to the processor gem #1818

@github-actions github-actions bot removed the stale Marks an issue/PR stale label Nov 28, 2025
@miry
Copy link
Contributor

miry commented Nov 28, 2025

Gem producess the message:

WARN -- : [DEPRECATION] The 'opentelemetry-helpers-sql-obfuscation' gem has been renamed to 'opentelemetry-helpers-sql-processor'. No action is needed unless you use this gem directly.

It would nice to merge those changes.

@hannahramadan
Copy link
Member Author

@arielvalentin, yes! Apologies - I got bogged down with some other work, but I will prioritize this. Good to get that deprecation warning out of people's logs.

@hannahramadan hannahramadan requested review from a team as code owners November 28, 2025 16:52
@arielvalentin arielvalentin merged commit 73877a3 into open-telemetry:main Nov 28, 2025
66 checks passed
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.

4 participants