Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

release: Release 4 gems #1161

Closed
wants to merge 2 commits into from
Closed

Conversation

github-actions[bot]
Copy link
Contributor

This pull request prepares new gem releases for the following gems:

  • opentelemetry-helpers-sql-obfuscation 0.2.0 (was 0.1.1)
  • opentelemetry-instrumentation-pg 0.29.0 (was 0.28.0)
  • opentelemetry-instrumentation-mysql2 0.28.0 (was 0.27.2)
  • opentelemetry-instrumentation-trilogy 0.60.0 (was 0.59.3)

For each gem, this pull request modifies the gem version and provides an initial changelog entry based on conventional commit messages. You can edit these changes before merging, to release a different version or to alter the changelog text.

  • To confirm this release, merge this pull request, ensuring the "release: pending" label is set. The release script will trigger automatically on merge.
  • To abort this release, close this pull request without merging.

The generated changelog entries have been copied below:


opentelemetry-helpers-sql-obfuscation

v0.2.0 / 2024-09-10

  • BREAKING CHANGE: Return message when sql is over the obfuscation limit

  • FIXED: Return message when sql is over the obfuscation limit


opentelemetry-instrumentation-pg

v0.29.0 / 2024-09-10

  • BREAKING CHANGE: Return message when sql is over the obfuscation limit

  • FIXED: Return message when sql is over the obfuscation limit


opentelemetry-instrumentation-mysql2

v0.28.0 / 2024-09-10

  • BREAKING CHANGE: Return message when sql is over the obfuscation limit

  • FIXED: Return message when sql is over the obfuscation limit


opentelemetry-instrumentation-trilogy

v0.60.0 / 2024-09-10

  • BREAKING CHANGE: Return message when sql is over the obfuscation limit

  • FIXED: Return message when sql is over the obfuscation limit

* opentelemetry-helpers-sql-obfuscation 0.2.0 (was 0.1.1)
* opentelemetry-instrumentation-pg 0.29.0 (was 0.28.0)
* opentelemetry-instrumentation-mysql2 0.28.0 (was 0.27.2)
* opentelemetry-instrumentation-trilogy 0.60.0 (was 0.59.3)
Comment on lines +5 to +7
* BREAKING CHANGE: Return message when sql is over the obfuscation limit

* FIXED: Return message when sql is over the obfuscation limit
Copy link
Contributor

Choose a reason for hiding this comment

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

@arielvalentin - Is this too wordy for a changelog entry? I have another option that's a little more concise in another comment below. I think these should be edited regardless, since the same message is listed twice.

Whatever we pick, I'll apply to the other changelogs.

Suggested change
* BREAKING CHANGE: Return message when sql is over the obfuscation limit
* FIXED: Return message when sql is over the obfuscation limit
* BREAKING CHANGE: Return a message when sql is over the obfuscation limit. Fixes a bug that sent un-obfuscated statements when sql was over the obfuscation limit and the statement had prepended comments.

Comment on lines +5 to +7
* BREAKING CHANGE: Return message when sql is over the obfuscation limit

* FIXED: Return message when sql is over the obfuscation limit
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* BREAKING CHANGE: Return message when sql is over the obfuscation limit
* FIXED: Return message when sql is over the obfuscation limit
* BREAKING CHANGE: Return message when sql is over the obfuscation limit
* FIXED: SQL statements with prepended comments that exceeded the obfuscation limit would be sent unsanitized

@kaylareopelle
Copy link
Contributor

Closing, we need to update instrumentation-all as well.

@github-actions github-actions bot deleted the release/multi/20240910202818 branch September 10, 2024 22:39
Copy link
Contributor Author

Release PR closed without merging.

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

Successfully merging this pull request may close these issues.

2 participants