Remove builder as a dependency #2445
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add UPGRADING.md entry
Generated by 🚫 Danger |
dblock
reviewed
May 20, 2024
UPGRADING.md
Outdated
| @@ -3,6 +3,10 @@ Upgrading Grape | |||
|
|
|||
| ### Upgrading to >= 2.1.0 | |||
|
|
|||
| #### Builder => XML | |||
|
|
|||
| From now on, if you want to generate XML, you'll need to add `builder` in your Gemfile. | |||
There was a problem hiding this comment.
Let's add some details. Maybe this.
The `builder` gem dependency has been made optional as it's only used when generating XML. If your code does XYZ, add `builder` to `Gemfile`.
See [#2445](https://github.com/ruby-grape/grape/pull/2445) for more information.
dblock
requested changes
May 20, 2024
It's in the our test Gemfile |
dblock
approved these changes
May 21, 2024
Does it make sense to remove it from there and move XML tests to integration? Probably does, can be done in a separate PR. |
arielvalentin
added a commit
to arielvalentin/opentelemetry-ruby-contrib
that referenced
this pull request
Jun 16, 2024
`builder` was removed as a dependency to `grape` but is used by active support in some versions. See ruby-grape/grape#2443 See ruby-grape/grape#2445 Fixes open-telemetry#1015
arielvalentin
added a commit
to open-telemetry/opentelemetry-ruby-contrib
that referenced
this pull request
Jun 16, 2024
`builder` was removed as a dependency to `grape` but is used by active support in some versions. See ruby-grape/grape#2443 See ruby-grape/grape#2445 Fixes #1015
Ongman
added a commit
to Asana/opentelemetry-ruby-contrib
that referenced
this pull request
Jul 24, 2024
* fix: Remove dependence on activesupport (#687)
Instrumentations must not use transitive dependencies used by the library.
In this case, relying on the gruf library to load specific ActiveSupport extensions leaves the instrumentation vulnerable to bugs.
For this reason I have changed the code to use Enumerable methods instead of ActiveSupport extensions.
See https://github.com/open-telemetry/opentelemetry-ruby-contrib/issues/686
* fix!: Drop DelayedJob ActiveRecord in Tests (#685)
* fix: Add Rails 7.1 compatability (#684)
* chore: Add tests for Rails 7.1
* fix: Rails 7.1 incompatabilities
* feat!: obfuscation for mysql2, dalli and postgresql as default option for db_statement (#682)
* feat!: fuscation for mysql2, dalli and pg
* feat!: update readme
* feat!: set db.statement option to obfuscate by default for mysql2, pg and dalli
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* ci: Update test reset code for GraphQL-Ruby 2.1.3 (#691)
Fixes https://github.com/open-telemetry/opentelemetry-ruby-contrib/issues/689
* fix: Omit `nil` `net.peer.name` attributes (#693)
* ci: upgrade to latest stable version of toys (#694)
Fixes errors with incompatible versions defined in the configs:
https://github.com/open-telemetry/opentelemetry-ruby-contrib/actions/runs/6534421626/job/17741560442#step:5:10
* release: Release 12 gems (#695)
* release: Release 12 gems
* opentelemetry-instrumentation-gruf 0.1.1 (was 0.1.0)
* opentelemetry-instrumentation-active_support 0.4.3 (was 0.4.2)
* opentelemetry-instrumentation-action_view 0.6.1 (was 0.6.0)
* opentelemetry-instrumentation-action_pack 0.7.1 (was 0.7.0)
* opentelemetry-instrumentation-active_job 0.6.1 (was 0.6.0)
* opentelemetry-instrumentation-active_record 0.6.3 (was 0.6.2)
* opentelemetry-instrumentation-dalli 0.25.0 (was 0.24.2)
* opentelemetry-instrumentation-delayed_job 0.22.0 (was 0.21.0)
* opentelemetry-instrumentation-faraday 0.23.3 (was 0.23.2)
* opentelemetry-instrumentation-mysql2 0.25.0 (was 0.24.3)
* opentelemetry-instrumentation-pg 0.26.0 (was 0.25.3)
* opentelemetry-instrumentation-rails 0.28.1 (was 0.28.0)
* ci: Trigger builds
* fix: Apply suggestions from code review
Co-authored-by: Josef Šimánek <josef.simanek@gmail.com>
* fix: bump gem versions
* fix: bump gems
---------
Co-authored-by: Ariel Valentin <ariel@arielvalentin.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
Co-authored-by: Josef Šimánek <josef.simanek@gmail.com>
* chore: Update CODEOWNERS (#692)
* chore: Update CODEOWNERS
Add @simi and @kaylareopelle and approvers to the list
* release: Release opentelemetry-instrumentation-all 0.51.0 (was 0.50.1) (#699)
* release: Release opentelemetry-instrumentation-all 0.51.0 (was 0.50.1)
* docs: Update all gem Changelog
---------
Co-authored-by: Ariel Valentin <ariel@arielvalentin.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* feat(trilogy): instrument connect and ping (#704)
These can be just as slow as a query if not slower.
Co-authored-by: Jean Boussier <jean.boussier@gmail.com>
* fix: Remove dependency on ActiveSupport core extensions from Grape instrumentation (#706)
* release: Release opentelemetry-instrumentation-trilogy 0.57.0 (was 0.56.3) (#708)
* release: Release opentelemetry-instrumentation-trilogy 0.57.0 (was 0.56.3)
* Empty commit
* Release instrumentation-all as well.
---------
Co-authored-by: Ariel Valentin <ariel@arielvalentin.com>
Co-authored-by: Francis Bogsanyi <francis.bogsanyi@shopify.com>
* chore(deps): update rubocop requirement from ~> 1.56.2 to ~> 1.57.1 (#702)
Updates the requirements on [rubocop](https://github.com/rubocop/rubocop) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop/releases)
- [Changelog](https://github.com/rubocop/rubocop/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop/rubocop/compare/v1.56.2...v1.57.1)
---
updated-dependencies:
- dependency-name: rubocop
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore(deps-dev): update rubocop requirement from ~> 1.56.1 to ~> 1.57.1 in /instrumentation/base (#701)
chore(deps-dev): update rubocop requirement from ~> 1.56.1 to ~> 1.57.1
Updates the requirements on [rubocop](https://github.com/rubocop/rubocop) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop/releases)
- [Changelog](https://github.com/rubocop/rubocop/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop/rubocop/compare/v1.56.1...v1.57.1)
---
updated-dependencies:
- dependency-name: rubocop
dependency-type: direct:development
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore(deps-dev): update rubocop requirement from ~> 1.56.1 to ~> 1.57.1 in /resource_detectors (#700)
chore(deps-dev): update rubocop requirement from ~> 1.56.1 to ~> 1.57.1
Updates the requirements on [rubocop](https://github.com/rubocop/rubocop) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop/releases)
- [Changelog](https://github.com/rubocop/rubocop/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop/rubocop/compare/v1.56.1...v1.57.1)
---
updated-dependencies:
- dependency-name: rubocop
dependency-type: direct:development
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore(deps-dev): update rubocop requirement from ~> 1.56.1 to ~> 1.57.1 in /propagator/ottrace (#698)
chore(deps-dev): update rubocop requirement from ~> 1.56.1 to ~> 1.57.1
Updates the requirements on [rubocop](https://github.com/rubocop/rubocop) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop/releases)
- [Changelog](https://github.com/rubocop/rubocop/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop/rubocop/compare/v1.56.1...v1.57.1)
---
updated-dependencies:
- dependency-name: rubocop
dependency-type: direct:development
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore(deps-dev): update rubocop requirement from ~> 1.56.1 to ~> 1.57.1 in /propagator/xray (#697)
chore(deps-dev): update rubocop requirement from ~> 1.56.1 to ~> 1.57.1
Updates the requirements on [rubocop](https://github.com/rubocop/rubocop) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop/releases)
- [Changelog](https://github.com/rubocop/rubocop/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop/rubocop/compare/v1.56.1...v1.57.1)
---
updated-dependencies:
- dependency-name: rubocop
dependency-type: direct:development
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Remove opentelemetry-resource_detectors all-in-one gem (#659)
* fix: remove call to ActiveSupport::Notifications.notifier#synchronize deprecated in Rails 7.2 (#707)
* wrap call to depcrecated private API behind version conditional
* convert Rails version string to Gem::Version
* fix module access?
* check for synchronize method instead of Rails version
* add comment
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore: Fix linter issue
* release: Release 2 gems (#710)
* release: Release 2 gems
* opentelemetry-instrumentation-grape 0.1.5 (was 0.1.4)
* opentelemetry-instrumentation-active_support 0.4.4 (was 0.4.3)
* ci: Force
---------
Co-authored-by: Ariel Valentin <ariel@arielvalentin.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* added httpx opentelemetry adapter (#681)
* added httpx opentelemetry adapter
* Update instrumentation/httpx/CHANGELOG.md
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* Update instrumentation/httpx/README.md
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* Update instrumentation/httpx/README.md
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
Co-authored-by: Josef Šimánek <josef.simanek@gmail.com>
* chore: Fix HTTPx attribute and changelog (#711)
* chore: Change release restrictions
The toys gem checks that all actions are passing before allowing
a release request to be opened, however only the CI builds are actually required.
This change limits the release request requirements to look specifically for CI builds
* chore: Fix httpx version
* chore: bump toys
* release: Release opentelemetry-instrumentation-httpx 0.1.0 (initial release) (#713)
* release: Release opentelemetry-instrumentation-httpx 0.1.0 (initial release)
* Update instrumentation/httpx/CHANGELOG.md
* feat!: Drop Rails 6.0 EOL (#680)
* feat!: Drop Rails 6.0 EOL
6.0 is no longer receiving maintenance, security, or feature updates as of 01 Jun 2023
Users who want to continue instrumentating Rails applications should pin to earlier versions of the instrumentation.
* squash: Fix test
* squash: gem version object instead of string
* Update README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
---------
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Inline gemspec dev constraints with Appraisals. (#716)
* Removal Sinatra 1.x Appraisal (#715)
fix: Removal Sinatra 1.x Appraisal
Rack 1.x is not directly supported anymore. Sinatra 1.x in turn
"is not". Removal appraisal and add a compatibility note to both
Sinatra and Rack for proper instrumentation version usage.
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* feat!(active_job): Use ActiveSupport instead of patches (#677)
* feat!(active_job): Use ActiveSupport instead of patches
* refactor: Register discard job
* refactor: independent registration
* refactor: Use template methods for children
* refactor: some more small fixes
* refactor: a little clean up
* refactor: a little more clarity
* refactor: try to make sense of a few things
* refactor: fix test directory structure
* refactor: extract to files
* add some docs
* refactor: clean up and renames
* refactor: split tests out per handler
* refactor... maybe a little overengineering
* fix: add examples of in-line instrumentation
* Update README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update default.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update perform.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* squash: Use Singleton Tracer
* fix: Use top level namespaces
When used in conjunction with the `OpenTelemetry::Instrumenation::ActiveSupport`,
the classloader would mistakenly use the wrong namespace and raises a `NameError`.
This change updates references to ensure we use top level namespaces to load the appropriate classes.
* squash: Update instrumentation/active_job/README.md
Co-authored-by: Robb Kidd <robb@thekidds.org>
* squash: Update docs
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* squash: Reduce Local Variable Allocation
Co-authored-by: Robb Kidd <robb@thekidds.org>
* squash: Safe navigation no longer required
* squash: fix bad commit
* squash: Remove possible PII from status message
* squash: PR Feedback
* squash: Use up to date semconv and remove unused attrs
* squash: remove unused file
* squash: fix test
* squash: fix messaging id
* squash: More semconv fixes
* squash: fix ids
* squash: feedback from Ruby SIG
---------
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
Co-authored-by: Robb Kidd <robb@thekidds.org>
* release: Release 6 gems (#724)
* release: Release 6 gems
* opentelemetry-instrumentation-active_support 0.5.0 (was 0.4.4)
* opentelemetry-instrumentation-action_view 0.7.0 (was 0.6.1)
* opentelemetry-instrumentation-action_pack 0.8.0 (was 0.7.1)
* opentelemetry-instrumentation-active_job 0.7.0 (was 0.6.1)
* opentelemetry-instrumentation-active_record 0.7.0 (was 0.6.3)
* opentelemetry-instrumentation-rails 0.29.0 (was 0.28.1)
* docs: Update instrumentation/active_job/CHANGELOG.md
* chore: bump all dependencies
* chore: bump version file
* chore: bump rails dependencies
---------
Co-authored-by: Ariel Valentin <ariel@arielvalentin.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* fix: Get Rdkafka version from VERSION contant (#726)
* fix: Update `Net::HTTP` instrumentation to no-op on untraced contexts (#722)
* chore: Add rspec-mocks development dependency
* fix: Net::HTTP instrumentation no-op on untraced context
Update Net::HTTP instrumentation to no-op on HTTP requests within untraced
contexts. Prevents the logging of "called finish on an ended Span" messages.
Example scenario:
The Resque instrumentation can be configured to call
`OpenTelemetry::SDK::TraceProvider#force_flush` at the end of each job. That
causes the batch span processor to immediately send everything it has to the
OTLP exporter. The exporter then marks the HTTP request used to send the data
to the APM server as "untraced", by adding an `untraced` key to the context and
setting it to `true`. Before this commit, the `Net::HTTP` instrumentation did
not check for the `untraced` flag. So it would try set attributes and call
`finish` on a span that was not recording, creating one "called finish on an
ended Span" log entry for each batch of exported spans. On a large application,
that results in millions of those log entries per minute.
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* release: Release 2 gems (#728)
* release: Release 2 gems
* opentelemetry-instrumentation-rdkafka 0.4.1 (was 0.4.0)
* opentelemetry-instrumentation-net_http 0.22.3 (was 0.22.2)
* ci: Force
---------
Co-authored-by: Ariel Valentin <ariel@arielvalentin.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* chore(dev-deps): Add rubocop-performance (#727)
chore(dev-deps): Add and enable rubocop-performance
* release: Release opentelemetry-instrumentation-rdkafka 0.4.2 (#731)
* release: Release 23 gems (#732)
* release: Release 23 gems
* opentelemetry-propagator-ottrace 0.21.2 (was 0.21.1)
* opentelemetry-propagator-xray 0.22.1 (was 0.22.0)
* opentelemetry-instrumentation-base 0.22.3 (was 0.22.2)
* opentelemetry-instrumentation-active_job 0.7.1 (was 0.7.0)
* opentelemetry-instrumentation-active_support 0.5.1 (was 0.5.0)
* opentelemetry-instrumentation-concurrent_ruby 0.21.2 (was 0.21.1)
* opentelemetry-instrumentation-delayed_job 0.22.1 (was 0.22.0)
* opentelemetry-instrumentation-ethon 0.21.3 (was 0.21.2)
* opentelemetry-instrumentation-excon 0.21.3 (was 0.21.2)
* opentelemetry-instrumentation-faraday 0.23.4 (was 0.23.3)
* opentelemetry-instrumentation-grape 0.1.6 (was 0.1.5)
* opentelemetry-instrumentation-graphql 0.26.8 (was 0.26.7)
* opentelemetry-instrumentation-http 0.23.2 (was 0.23.1)
* opentelemetry-instrumentation-http_client 0.22.3 (was 0.22.2)
* opentelemetry-instrumentation-httpx 0.1.1 (was 0.1.0)
* opentelemetry-instrumentation-mongo 0.22.2 (was 0.22.1)
* opentelemetry-instrumentation-net_http 0.22.4 (was 0.22.3)
* opentelemetry-instrumentation-pg 0.26.1 (was 0.26.0)
* opentelemetry-instrumentation-que 0.7.1 (was 0.7.0)
* opentelemetry-instrumentation-rack 0.23.5 (was 0.23.4)
* opentelemetry-instrumentation-restclient 0.22.3 (was 0.22.2)
* opentelemetry-instrumentation-rspec 0.3.2 (was 0.3.1)
* opentelemetry-instrumentation-rails 0.29.1 (was 0.29.0)
* chore: Apply suggestions from code review
---------
Co-authored-by: Ariel Valentin <ariel@arielvalentin.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* perf(graphql): cache attribute hashes (#723)
* GraphQL: cache attribute hashes
* Use caches for GraphQLTracer attributes
* Use the same attribute hashes for resolve_type and authorized
* revert changes to legacy trace
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* release: Release 2 gems (#741)
* feat!: Add a connect span to excon (#712)
* Add a connect span to excon and add more span attributes to the tracer middleware
* Skip matching on the error message as it varies by platform
* Rescue the IOError that can occur on accept
* Switch to must_be_empty assertion instead of size
* Move allowing and disallowing connect to setup and after respectively.
* use dig when getting hostname and port of proxy
* Call untraced when we hit an untraced host.
* Switch to using recording? to test whether we should finish a span.
* Switch to handle_error instead of debug logging.
* Record the exception on error
* Perform the next step in the middleware stack in the context of the current span.
* Add assertions on http spans to connect tests.
* Fix rubocop lints
* Remove interpolation from status message on span now that we capture exceptions as an event.
* Switch to attach and detach instead of with_span
* Include untraced context into untraced? check for the middleware and patch.
* Add test for untraced.
* Add a module that centralizes the untraced hosts concern.
* Expand doc comment.
* Move module to the excon gem.
* Add doc comment for untraced? in the concern
* Update instrumentation/excon/lib/opentelemetry/instrumentation/excon/middlewares/tracer_middleware.rb
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* release: Release 2 gems (#744)
* chore: Fix Rails 7.2 secrets deprecation warning (#747)
* feat!(action_pack): Use ActiveSupport instead of patches (#703)
* feat!(action_pack): Use ActiveSupport instead of patches
* feat!(action_pack): remove comments
* feat: drop 6.0 on action_pack to verify the test case
* feat: add rails 6.0 back
* feat!(action_pack): remove rails 6.0 support + update readme on error handling
* Update instrumentation/action_pack/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_pack/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_pack/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/handlers/action_controller.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/handlers.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/handlers/action_controller.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/handlers/action_controller.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/handlers/action_controller.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/handlers/action_controller.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/handlers.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* update Approvers list in README (#755)
Kayla and Josef are already approvers! This brings the docs up-to-date with reality.
* chore(deps-dev): update rubocop to ~> 1.58.0 in multiple gems (#756)
* chore(deps-dev): update rubocop to ~> 1.58.0 in multiple gems
* appease Rubocop: Lint/RedundantSafeNavigation
Redundant safe navigation with default literal detected.
prefix = endpoint.routes.first.options[:prefix]&.to_s || ''
^^^^^^^^^^^^
OK. We can stop at safely navigating the string coersion without
supplying a default because paths.reject{} below filters nils.
* appease Rubocop: Style/RedundantReturn
Redundant return detected.
return "HTTP #{request_method} #{request_path} miniswan"
^^^^^^
Sure.
* appease Rubocop: Style/RedundantParentheses
Don't use parentheses around a logical expression.
(!gem_version.nil? && gem_version >= MINIMUM_VERSION)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
OK.
* chore(deps): bump actions/stale from 8 to 9 (#757)
* chore: let stalebot close issues after 30 days of being stale. (#759)
* feat!: change db.mysql.instance.address to db.instance.id (#758)
* refactor(trilogy): change db.mysql.instance.address to db.instance.id
fixes #745
* feat!(trilogy): change db.mysql.instance.address to db.instance.id
fixes #745
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* chore(deps-dev): update rubocop to ~> 1.59.0 in all gems (#760)
* chore(deps): update rubocop requirement from ~> 1.58.0 to ~> 1.59.0
Updates the requirements on [rubocop](https://github.com/rubocop/rubocop) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop/releases)
- [Changelog](https://github.com/rubocop/rubocop/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop/rubocop/compare/v1.58.0...v1.59.0)
---
updated-dependencies:
- dependency-name: rubocop
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
* chore(deps-dev): also update rubocop to ~> 1.59.0 in all the gemspecs
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Robb Kidd <robbkidd@honeycomb.io>
* fix: Backport Rack proxy event to middleware (#764)
* chore: update codeowners
* chore: Add Tests for GraphQL C Parser (#773)
Graphql 2.2 introduces an optional C implementation of the Parser as well as optimizations to the pure Ruby parser that result in changes to what is captured during instrumentation.
Users of the pure Ruby implementation will no longer see graphql.lex spans emitted as a part of their traces and will instead need to switch to using the C Parser in order to explicitly see time spent in the lexer.
This PR introduces updates to the test suite to add coverage for the C Parser and changes to the Ruby implementation.
There are no underlying changes in the instrumentation itself.
* chore: Experiment with GCP Release Please (#729)
* chore: Experiment with GCP Release Please
An experiment to replace toys.
https://github.com/google-github-actions/release-please-action/tree/main
* squash: fixes from PR feedback
* squash: Give otel bot the credit
* squash: pin to v4.0.2
* squash: add steps to execute the release
* squash: fix output
* squash: release one at a time
* chore: fix actions definition
* chore: Fixes for release-please
* chore: use objects not arrays
* chore: Remove invalid config
* chore: Remove nested objects in config
* chore: Prerelease config
* chore: Fix release version paths
* chore: try again
* chore: Block matrix from running if no release were created
* chore: try https://github.com/orgs/community/discussions/27096
* chore: remove superflous action config
* feat(rack)! Use Rack Events By Default (#709)
* feat(rack)! Use Rack Events By Default
I have been running the `Rack::Events` based instrumentation in our production workloads since 2023-10-23.
There is no noticeable difference in performance, however applications using `Rack::BodyProxy`
to stream responses will notice a slight increase in latency since it will include timings
writing response output to the socket.
* squash: fix invalid HTTP responses in tests
* squash: Remove redundant test
* squash: Update rack error description
* Update instrumentation.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
---------
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* ci: Use Org OTel Bot Token (#789)
The Org Level OTel Bot Token likely a PAT, which will allow PRs opened by GH actions to trigger
subsequent actions workflows.
This will fix the problem where CI jobs do not run when the bot opens a PR for release requests.
* Revert "ci: Use Org OTel Bot Token (#789)" (#790)
This reverts commit 586b0fb137d430e2b826f3814723dd61c8d82ff1.
* Trilogy: only set `db.instance.id` attribute if there is a value (#792)
* fix: Trilogy only set db.instance.id attribute if there is a value (#792)
* ci: OTel Bot Token Redux
* feat!: Use Rack Events By Default
* ci: Revert token again
* chore: release main (#793)
* chore: release main
* feat: Update Trilogy and Rack
---------
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* ci: force latest bundler version
* ci: invalid option
* ci: Fix toys config
* ci: Try again
* Revert "chore: release main (#793)"
This reverts commit 9d8788b8b27f4492eba9d6f60cafcee95ecad148.
* ci: Give Credit to OTel Bot
* release: Release 2 gems (#796)
* release: Release 2 gems
* opentelemetry-instrumentation-trilogy 0.58.0 (was 0.57.0)
* opentelemetry-instrumentation-rack 0.24.0 (was 0.23.5)
* feat!: All Upgrade Trilogy:0.58 Rack:0.24
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* release: Release opentelemetry-instrumentation-all 0.55.0 (was 0.54.0) (#799)
* release: Release opentelemetry-instrumentation-all 0.55.0 (was 0.54.0)
* ci: Update instrumentation/all/CHANGELOG.md
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* release: Release 3 gems (#808)
* release: Release 3 gems
* opentelemetry-instrumentation-all 056.0 (was 0.55.0)
* opentelemetry-instrumentation-action_pack 0.9.0 (was 0.8.0)
* opentelemetry-instrumentation-rails 0.30.0 (was 0.29.1)
* Apply suggestions from code review
* squash: Bump dependencies
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* chore: Bump rubocop-performance 1.20.0 (#780)
```console
$> bin/update-dependencies rubocop-performance 1.20.0
```
* ci: Add Tests For Ruby 3.3 (#797)
* ci: Add Tests For Ruby 3.3
Ruby 3.3 is out so this PR adds actions to test against it.
One this to note is that bigcommerce/gruf does not yet support Ruby 3.3. Once https://github.com/bigcommerce/gruf/pull/197 is merged I will update the test suite to include gruf again.
* squash: fix test
* squash: fix test
* chore(deps): bump ruby/setup-ruby from 1.162.0 to 1.165.1 (#809)
* Add xuan-cao-swi to approvers (#810)
* ci: Update Release Please Configs
* ci: Use appropriate config
* ci: fix job names
* ci: Use correct job names
* ci: update manifest versions
* ci: bump rack manifest
* ci: Fix Job Refs
* ci: remove hyphens
* ci: fix reference
* chore(deps): update rubocop-performance requirement from ~> 1.19.1 to ~> 1.20.2 (#807)
Updates the requirements on [rubocop-performance](https://github.com/rubocop/rubocop-performance) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop-performance/releases)
- [Changelog](https://github.com/rubocop/rubocop-performance/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop/rubocop-performance/compare/v1.19.1...v1.20.2)
---
updated-dependencies:
- dependency-name: rubocop-performance
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* docs: Add note about running update-dependencies in ubuntu-linux (#819)
* chore: Update rubocop version to 1.60.0 (#818)
* chore: Update rubocop version to 1.60.0
* chore: Update rubocop in main Gemfile
* chore: Address Rubocop offenses
Incorporate Style/ArgumentsForwarding - Use shorthand syntax ... for arguments forwarding.
* chore(deps): bump ruby/setup-ruby from 1.165.1 to 1.167.0 (#820)
* fix: "Getting Started" link at README (#822)
* docs: ✏️ Sidekiq instrumentation options (#824)
* chore: add appraisal for Sinatra 4. (#827)
* chore: add appraisal for Sinatra 4.
* chore: allow for latest minor version of rack-test.
* chore: add rackup to sinatra example Gemfile (#828)
Co-authored-by: Sam <370182+plantfansam@users.noreply.github.com>
* chore: Update rubocop to 1.60.1 (#823)
* chore(deps): bump ruby/setup-ruby from 1.167.0 to 1.169.0 (#829)
Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.167.0 to 1.169.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Commits](https://github.com/ruby/setup-ruby/compare/v1.167.0...v1.169.0)
---
updated-dependencies:
- dependency-name: ruby/setup-ruby
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* fix: Remove disable directives leftover from older version of Rubocop (#835)
* docs: relocate Resque config option comments to render in Yard docs (#834)
docs: move config option docs up to the Instrumentation class
As formatted markdown comments on the class, the information is made visible
in the Yard-generated docs without making the options appear to be class or
instance variables.
* chore(deps): bump ruby/setup-ruby from 1.169.0 to 1.170.0 (#836)
Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.169.0 to 1.170.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Commits](https://github.com/ruby/setup-ruby/compare/v1.169.0...v1.170.0)
---
updated-dependencies:
- dependency-name: ruby/setup-ruby
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore(deps): bump ruby/setup-ruby from 1.170.0 to 1.171.0 (#838)
Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.170.0 to 1.171.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Commits](https://github.com/ruby/setup-ruby/compare/v1.170.0...v1.171.0)
---
updated-dependencies:
- dependency-name: ruby/setup-ruby
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* refactor!: Move shared sql behavior to helper gems (#529)
* chore(deps): bump actions/checkout from 3 to 4 (#844)
* chore: Update releases path
* chore: force initial release version (#851)
* chore: force initial release version
* squash: temporarily disable version until helpers are released
* feat: Propagate context to Vitess (#850)
* feat: Propagate context to Vitess
* Require base64
* Refactor to extract Vitess propagator
* Use in-repo vitess propagator
* The actual tests
* Remove unnecessary require
* Robust against frozen string carrier
* Update README
* Fix test
* Test with mutable strings
* Update instrumentation/trilogy/test/opentelemetry/instrumentation/trilogy/instrumentation_test.rb
Co-authored-by: Robert <robertlaurin@users.noreply.github.com>
* Appease the cop
---------
Co-authored-by: Robert <robertlaurin@users.noreply.github.com>
* chore: Update toys and toys release script pins to 0.15.5 (#854)
* release: Release 3 gems (#856)
* release: Release 3 gems
* opentelemetry-helpers-mysql 0.1.0 (initial release)
* opentelemetry-helpers-sql-obfuscation 0.1.0 (initial release)
* opentelemetry-propagator-vitess 0.1.0 (initial release)
* ci: Force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* release: Release 8 gems (#857)
* release: Release 8 gems
* opentelemetry-instrumentation-trilogy 0.59.0 (was 0.58.0)
* opentelemetry-instrumentation-resque 0.5.1 (was 0.5.0)
* opentelemetry-instrumentation-bunny 0.21.2 (was 0.21.1)
* opentelemetry-instrumentation-mysql2 0.26.0 (was 0.25.0)
* opentelemetry-instrumentation-pg 0.27.0 (was 0.26.1)
* opentelemetry-instrumentation-que 0.8.0 (was 0.7.1)
* opentelemetry-instrumentation-sidekiq 0.25.1 (was 0.25.0)
* opentelemetry-instrumentation-all 0.57.0 (was 0.56.0)
* ci: Force
* fix!: Upgrade dependencies
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* docs: Fix doc for sidekiq options (#825)
* docs: ✏️ fix yard for sidekiq instrumentation options
* docs: Refactor Sidekiq instrumentation and add configuration options
* Update instrumentation/sidekiq/lib/opentelemetry/instrumentation/sidekiq.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/sidekiq/lib/opentelemetry/instrumentation/sidekiq/instrumentation.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* ci: Add Conventional Commit Linter (#800)
* ci: Add Conventional Commit Linter
* squash: remove unused action
* squash: Delete commitlint.config.js
* squash: PR feedback
* squash: Update contributing guide
* squash: PR Feedback
* fix: add missing requires for sql-helpers to mysql, pg, and trilogy instrumentation (#859)
* release: Release 4 gems (#860)
* release: Release 4 gems
* opentelemetry-instrumentation-trilogy 0.59.1 (was 0.59.0)
* opentelemetry-instrumentation-mysql2 0.26.1 (was 0.26.0)
* opentelemetry-instrumentation-pg 0.27.1 (was 0.27.0)
* opentelemetry-instrumentation-sidekiq 0.25.2 (was 0.25.1)
* ci: force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* fix: return nil for non-existant key in AwsSdk::MessageAttributeGetter (#853)
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* release: Release opentelemetry-instrumentation-aws_sdk 0.5.1 (was 0.5.0) (#861)
* release: Release opentelemetry-instrumentation-aws_sdk 0.5.1 (was 0.5.0)
* ci: Force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* feat: instrument mysql2 prepare statement (#862)
* feat: instrument prepare for mysql2 lib
* revision
* release: Release 2 gems (#868)
* release: Release 2 gems
* opentelemetry-instrumentation-mysql2 0.27.0 (was 0.26.1)
* opentelemetry-instrumentation-all 0.58.0 (was 0.57.0)
* Update instrumentation/all/CHANGELOG.md
* squash: Bump mysql2 gem
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* feat!: Cache GraphQL attributes (#867)
* release: Release 2 gems (#871)
* release: Release 2 gems
* opentelemetry-instrumentation-graphql 0.28.0 (was 0.27.0)
* opentelemetry-instrumentation-all 0.59.0 (was 0.58.0)
* squash: Apply suggestions from code review
* squash: Bump GraphQL
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* feat: Add support gruf 2.19 (#872)
* fix: dup string if frozen in trilogy query (#863)
* fix: propagate context from trilogy instrumentation for frozen strings
* feat: faraday add support for internal spans (#873)
* release: Release opentelemetry-instrumentation-trilogy 0.59.2 (was 0.59.1) (#875)
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
* release: Release 3 gems (#881)
* release: Release 3 gems
* opentelemetry-instrumentation-gruf 0.2.0 (was 0.1.1)
* opentelemetry-instrumentation-faraday 0.24.0 (was 0.23.4)
* opentelemetry-instrumentation-all 0.60.0 (was 0.59.0)
* docs: Update instrumentation/all/CHANGELOG.md
* chore: bump dependencies
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore: Update rubocop and rubocop-performance in helper gems (#878)
* chore: drop dependency on ruby2_keywords (#885)
Now that Ruby below 2.7 is not supported anymore (#389, https://github.com/open-telemetry/opentelemetry-ruby-contrib/commit/38b083aa41b93bcc8f900cd65bc922c97fe26e5d) this is not needed anymore
* chore: Update stalebot (#889)
* chore(deps): bump ruby/setup-ruby from 1.171.0 to 1.172.0 (#883)
* chore(deps): bump ruby/setup-ruby from 1.172.0 to 1.173.0 (#911)
* fix: Propagate response attributes on Faraday::Error (#912)
feat: Propagate response attributes on Faraday::Error
The current `Faraday` middleware won't report `http.status_code` in
cases where an error is raised via another middleware (such as
`:raise_error` or `:json`).
This adds a special case for subclasses of `Faraday::Error` which have
response information available, so that `http.status_code` the span
status are still added to the span.
* release: Release opentelemetry-instrumentation-faraday 0.24.1 (was 0.24.0) (#913)
* release: Release opentelemetry-instrumentation-faraday 0.24.1 (was 0.24.0)
* ci: force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore: Fix typo in event_handler_resiliency_test.rb filename (#914)
* chore: Update rubocop to 1.62 (#910)
* chore: Update rubocop to 1.62
* Rubocop updates
* add an empty line after a magic comment
* remove redundant line continuation
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore(deps): bump google-github-actions/release-please-action from 4.0.2 to 4.1.0 (#908)
chore(deps): bump google-github-actions/release-please-action
Bumps [google-github-actions/release-please-action](https://github.com/google-github-actions/release-please-action) from 4.0.2 to 4.1.0.
- [Release notes](https://github.com/google-github-actions/release-please-action/releases)
- [Changelog](https://github.com/google-github-actions/release-please-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/google-github-actions/release-please-action/compare/v4.0.2...v4.1.0)
---
updated-dependencies:
- dependency-name: google-github-actions/release-please-action
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* docs: fix typo where Rake is mentioned instead of Rack (#922)
* chore: Run release-please only on open-telemetry (#917)
Currently, release-please attempts to run on forks of
opentelemetry-ruby-contrib. The workflow fails with the message
"GitHub Actions is not permitted to create or approve pull requests."
release-please is not needed on forks.
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* fix: instrumentation/active_record: add `:allow_retry` option to `find_by_sql` patch (#915)
fix: instrumentation/active_record add :allow_retry option to find_by_sql patch
Rails 7.2 (rails/rails@eabcff2) introduces the :allow_retry option for this method, so we need to add it to this patch to maintain compatibility.
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore(deps-ci): upgrade conventional commit check (#923)
commit-me@v1.4.0 includes an update[1] that recognizes scopes that
contain hyphens.
[1] https://github.com/dev-build-deploy/commit-me/commit/38d4cb09a1bf468012516a3bfbfad325f3435915
* chore(deps-dev): update rubocop-performance requirement from ~> 1.20.1 to ~> 1.21.0 in /helpers/sql-obfuscation (#921)
chore(deps-dev): update rubocop-performance requirement from ~> 1.20.1 to ~> 1.21.0
Updates the requirements on [rubocop-performance](https://github.com/rubocop/rubocop-performance) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop-performance/releases)
- [Changelog](https://github.com/rubocop/rubocop-performance/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop/rubocop-performance/compare/v1.20.1...v1.21.0)
---
updated-dependencies:
- dependency-name: rubocop-performance
dependency-type: direct:development
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore(deps-dev): update rubocop-performance requirement from ~> 1.20.1 to ~> 1.21.0 in /helpers/mysql (#919)
chore(deps-dev): update rubocop-performance requirement from ~> 1.20.1 to ~> 1.21.0
Updates the requirements on [rubocop-performance](https://github.com/rubocop/rubocop-performance) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop-performance/releases)
- [Changelog](https://github.com/rubocop/rubocop-performance/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop/rubocop-performance/compare/v1.20.1...v1.21.0)
---
updated-dependencies:
- dependency-name: rubocop-performance
dependency-type: direct:development
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore(deps-dev): update webmock requirement from ~> 3.19.1 to ~> 3.23.0 in /resources/azure (#888)
chore(deps-dev): update webmock requirement from ~> 3.19.1 to ~> 3.23.0
Updates the requirements on [webmock](https://github.com/bblimke/webmock) to permit the latest version.
- [Changelog](https://github.com/bblimke/webmock/blob/master/CHANGELOG.md)
- [Commits](https://github.com/bblimke/webmock/compare/v3.19.1...v3.23.0)
---
updated-dependencies:
- dependency-name: webmock
dependency-type: direct:development
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore(deps): update rubocop-performance requirement from ~> 1.20.2 to ~> 1.21.0 (#920)
Updates the requirements on [rubocop-performance](https://github.com/rubocop/rubocop-performance) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop-performance/releases)
- [Changelog](https://github.com/rubocop/rubocop-performance/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop/rubocop-performance/compare/v1.20.2...v1.21.0)
---
updated-dependencies:
- dependency-name: rubocop-performance
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore: Move @kaylareopelle to maintainers list (#926)
* fix: fix markdown header (#925)
* fix: suppress deprecation warning in Rdkafka Instrumentation (#884)
* fix: suppress deprecation warning in Rdkafka Instrumentation
* memoize getter
* Use constant
* rubocop fix
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore: update webmock requirement from ~> 3.19.1 to ~> 3.20.0 in /resources/google_cloud_platform (#847)
chore(deps-dev): update webmock requirement from ~> 3.19.1 to ~> 3.20.0
Updates the requirements on [webmock](https://github.com/bblimke/webmock) to permit the latest version.
- [Changelog](https://github.com/bblimke/webmock/blob/master/CHANGELOG.md)
- [Commits](https://github.com/bblimke/webmock/compare/v3.19.1...v3.20.0)
---
updated-dependencies:
- dependency-name: webmock
dependency-type: direct:development
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* release: Release 4 gems (#928)
* release: Release 4 gems
* opentelemetry-instrumentation-racecar 0.3.1 (was 0.3.0)
* opentelemetry-instrumentation-rdkafka 0.4.3 (was 0.4.2)
* opentelemetry-instrumentation-active_record 0.7.1 (was 0.7.0)
* opentelemetry-instrumentation-rack 0.24.1 (was 0.24.0)
* ci: Force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* fix: analyze span names in GraphQL instrumentation (#930)
* release: Release opentelemetry-instrumentation-graphql 0.28.1 (was 0.28.0) (#931)
* chore: bump dev-build-deploy/commit-me from 1.4.0 to 1.4.1 (#932)
* chore: bump ruby/setup-ruby from 1.173.0 to 1.174.0 (#936)
* fix: resolve active_record testing issue (#944)
* fix: resolve active_record issue
* Update instrumentation/active_record/Gemfile
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
---------
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore: bump ruby/setup-ruby from 1.174.0 to 1.175.0 (#943)
Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.174.0 to 1.175.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Commits](https://github.com/ruby/setup-ruby/compare/v1.174.0...v1.175.0)
---
updated-dependencies:
- dependency-name: ruby/setup-ruby
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore: update webmock requirement from ~> 3.20.0 to ~> 3.23.0 in /resources/google_cloud_platform (#929)
chore: update webmock requirement from ~> 3.20.0 to ~> 3.23.0
Updates the requirements on [webmock](https://github.com/bblimke/webmock) to permit the latest version.
- [Changelog](https://github.com/bblimke/webmock/blob/master/CHANGELOG.md)
- [Commits](https://github.com/bblimke/webmock/compare/v3.20.0...v3.23.0)
---
updated-dependencies:
- dependency-name: webmock
dependency-type: direct:development
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* fix: revert modifications to Apache license (#935)
See open-telemetry/community#2056 for background
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* chore: bump ruby/setup-ruby from 1.175.0 to 1.175.1 (#945)
* ci: Remove unused jobs (#947)
* chore: JRuby 9.4.6.0 - Ruby 3.1 (#952)
* chore: JRuby 9.4.6.0 - Ruby 3.1
Update CI to run JRuby under Ruby 3.1
* ci: Update Ruby Action
* feat: Add baggage span processor (#937)
* fix: Cancel duplicate builds (#954)
This change cancels any pending builds when someone pushes changes before the tests are complete
* ci: Setup Weekly Releases (#949)
This job will create a release request every week at least one hour before or during the Ruby SIG.
This will ensure are aware of any pending releases during the SIG meeting and choose to submit a release.
* chore: include new baggage span processor in release automation (#959)
tell toys about the new baggage span processor
Follow up to #937 to make the gem releasable with our automation.
* fix: Bundler conflict warnings (#951)
Fixes https://github.com/open-telemetry/opentelemetry-ruby-contrib/issues/938
* release: Release 35 gems (#960)
* ci: Add nightly builds (#948)
* ci: Add nightly builds
This should allow us to get ahead of any gem incompatabilities
* squash: remove unused action
* chore: Release on Tuesdays
* chore: bump ruby/setup-ruby from 1.175.1 to 1.176.0 (#963)
* fix: rack event baggage handling (#965)
* release: Release opentelemetry-instrumentation-rack 0.24.3 (was 0.24.2) (#967)
* release: Release opentelemetry-instrumentation-rack 0.24.3 (was 0.24.2)
* ci: Force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* fix: Untrace entire request (#968)
* fix: untraced in rack event middleware
* chore: bump common across the board
* release: Release 13 gems (#970)
* release: Release 13 gems
* opentelemetry-instrumentation-rdkafka 0.4.5 (was 0.4.4)
* opentelemetry-instrumentation-http_client 0.22.5 (was 0.22.4)
* opentelemetry-instrumentation-koala 0.20.4 (was 0.20.3)
* opentelemetry-instrumentation-dalli 0.25.2 (was 0.25.1)
* opentelemetry-instrumentation-ethon 0.21.5 (was 0.21.4)
* opentelemetry-instrumentation-excon 0.22.2 (was 0.22.1)
* opentelemetry-instrumentation-faraday 0.24.3 (was 0.24.2)
* opentelemetry-instrumentation-net_http 0.22.5 (was 0.22.4)
* opentelemetry-instrumentation-rack 0.24.4 (was 0.24.3)
* opentelemetry-instrumentation-redis 0.25.5 (was 0.25.4)
* opentelemetry-instrumentation-restclient 0.22.5 (was 0.22.4)
* opentelemetry-instrumentation-sidekiq 0.25.4 (was 0.25.3)
* opentelemetry-instrumentation-sinatra 0.23.4 (was 0.23.3)
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Robert Laurin <robert.laurin@shopify.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* feat: add aws Lambda instrumentation (#721)
* feat: add aws lambda instrumentation
* feat: return original handler response and require the handler file
* Update instrumentation/aws_lambda/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/example/Gemfile
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* feat: revision on fix sample test, update readme and add rubocop-performance
* feat: update readme
* Update instrumentation/aws_lambda/CHANGELOG.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/version.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* feat: add codeowner for aws_lambda
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update CODEOWNERS
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* feat: lambda - update semantic convention and test case
* feat: add lambda to workflow
* feat: lambda - run test with rake test directly
* feat: lambda - add rescue to avoid break user function
* feat: aws lambda - make faas.trigger static to http as following spec. add cloud account id
* Update instrumentation/aws_lambda/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* feat: lambda - use symbol and str for statusCode
* feat: lambda - refactor on status code
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* feat: lambda - update span name calculation period. add non-http trigger and test case
* feat: aws_lambda update
* feat: aws lambda - update attributes
* feat: aws lambda - revision
* Update instrumentation/aws_lambda/README.md
Co-authored-by: Lin Lin <lin.lin@solarwinds.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Lin Lin <lin.lin@solarwinds.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Lin Lin <lin.lin@solarwinds.com>
* feat: aws lambda - revision
* Update instrumentation/aws_lambda/example/sample.rb
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* Update instrumentation/aws_lambda/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/lib/opentelemetry/instrumentation/aws_lambda/handler.rb
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/aws_lambda/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* feat: lambda - update release file
* feat: remove ci-instrumentation-canary.yml
---------
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
Co-authored-by: Lin Lin <lin.lin@solarwinds.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* feat: Support prepend SQL comment for PG instrumentation (#690)
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* release: Release 2 gems (#975)
* release: Release 2 gems
* opentelemetry-instrumentation-pg 0.27.3 (was 0.27.2)
* opentelemetry-instrumentation-aws_lambda 0.1.0 (initial release)
* ci: Force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* feat: add instrumentation support for action mailer (#887)
* feat: add notification support for action mailer
* feat: action mailer typo
* Update instrumentation/action_mailer/example/app/views/test_mailer/welcome_email.html.erb
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* feat: action mailer: omit email address
* feat: actionmailer updated readme
* feat: actionmailer exclude mail attributes
* feat: force email out of attr
* feat: actionmailer use ecs email convention
* feat: revision
* feat: restrict on modifying original payload
* feat: revision
* Update instrumentation/action_mailer/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_mailer/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* feat: revision
* Update instrumentation/action_mailer/example/trace_request_demonstration.ru
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* Update instrumentation/action_mailer/README.md
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
---------
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@users.noreply.github.com>
* ci: Add ActionMailer CI (#976)
* ci: Add ActionMailer CI
* ci: Linter fixes
* squash: drop pry-byebug
* ci: fix action_mailer version
Initial versions must be 0.0.0
* release: Release opentelemetry-instrumentation-action_mailer 0.1.0 (initial release) (#979)
* release: Release opentelemetry-instrumentation-action_mailer 0.1.0 (initial release)
* ci: force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* docs: Instrumentation Authors Guide (#946)
* release: Release opentelemetry-instrumentation-grape 0.1.8 (was 0.1.7) (#985)
* release: Release opentelemetry-instrumentation-grape 0.1.8 (was 0.1.7)
* ci: Force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* chore: Update release please action to googleapis (#989)
The previous location, google-github-actions, is deprecated.
All future work will be completed in googleapis.
* fix: add action_mailer to rails and all (#984)
* fix: add action_mailer to rails and all
* fix: lint
---------
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* feat: add aws lambda to instrumentation-all (#986)
fix: add aws lambda to instrumentation-all
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
* chore: Update Rubocop to 1.64.0 (#997)
* chore: Rubocop Style/SuperArguments
* chore: Rubocop Style/RedundantParentheses
* chore: Rubocop
Style/RedundantFreeze
Lint/SuppressedException
* chore: Rubocop Layout/SpaceInsideHash
* chore: Update Rubocop to 1.64.0
* chore: Rubocop Style/SuperArguments
* chore: bump ruby/setup-ruby from 1.176.0 to 1.178.0 (#996)
* chore: bump ruby/setup-ruby from 1.176.0 to 1.178.0
Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.176.0 to 1.178.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Commits](https://github.com/ruby/setup-ruby/compare/v1.176.0...v1.178.0)
---
updated-dependencies:
- dependency-name: ruby/setup-ruby
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
* squash: bump reusable test action
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* test: Add vitess to the propagators matrix (#998)
* release: Release 2 gems (#1000)
* release: Release 2 gems
* opentelemetry-instrumentation-rails 0.30.2 (was 0.30.1)
* opentelemetry-instrumentation-all 0.61.0 (was 0.60.0)
* ci: Force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* fix: Add net.peer.name to ethon (#1004)
The ethon instrumentation did not include the `net.peer.name` attribute like other HTTP instrumentations and was not ported from ddtrace-rb: <https://github.com/DataDog/dd-trace-rb/blob/master/lib/datadog/tracing/contrib/ethon/easy_patch.rb#L159>
This attribute is important to GitHub because we base the `peer.service` extracting the subdomain and host information from `net.peer.name`.
In order to avoid parsing the URL multiple times, I inlined the common gem `cleanse_url` function and plan to open a separate PR adding a feature to support returing URI objects instead of Strings.
* ci: Updates grape appraisals (#1006)
Fixes https://github.com/open-telemetry/opentelemetry-ruby-contrib/issues/1005
* release: Release opentelemetry-instrumentation-ethon 0.21.6 (was 0.21.5) (#1008)
* release: Release opentelemetry-instrumentation-ethon 0.21.6 (was 0.21.5)
* ci: Force
---------
Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com>
Co-authored-by: Ariel Valentin <arielvalentin@github.com>
* test: Fix grape test suite (#1017)
`builder` was removed as a dependency to `grape` but is used by active support in some versions.
See ruby-grape/grape#2443
See ruby-grape/grape#2445
Fixes https://github.com/open-telemetry/opentelemetry-ruby-contrib/issues/1015
* chore: bump ruby/setup-ruby from 1.178.0 to 1.180.0 (#1009)
* chore: bump ruby/setup-ruby from 1.178.0 to 1.180.0
Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.178.0 to 1.180.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Commits](https://github.com/ruby/setup-ruby/compare/v1.178.0...v1.180.0)
---
updated-dependencies:
- dependency-name: ruby/setup-ruby
dependency-type: direct:production
update-type: version-update:semver-minor
...
…
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix #2443