sentry-sidekiq 4.2.1 → 4.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 383b3c28fc5583dadfc497ae0c3b0dc052e7909f8e71e8d8eca728f68b060aea
-  data.tar.gz: b867a8389fbd6e44b06a7256c6dafdc68f04acf2fe4dc2e99c84f647c010e614
+  metadata.gz: 58a2fdb9ebaf4e197fba06d2cae41179a91de332d5a6e4e65d27fa9173aa7a62
+  data.tar.gz: f13820fdaa0c2bc33e03a9301de7dd3f183ea199746842e626bbf9de4d948718
 SHA512:
-  metadata.gz: 71cd8d00dc973f0521b387ea75b2d2aa52da2078b5bbb7cd1b14533f0db944b1f387dd02a6ed0272ea7958c2fe18981c5350b24121e68e846851ae1c8b3e21d0
-  data.tar.gz: c1c3c647df576a8ed592d90a104dd171fe99c4b6cd775ee7364b9f9cbc84489b2614d20d085abf0c7ca1b5c3e71f3f2b50e0bdaf854321fa9b192fb153127b81
+  metadata.gz: 502d5c38a8652c16e099022d19eaaa6bd312e742996bd6039a34c30953b1b498938b9927a15d42a07f11478595566dd39f96b09bf06fc39c68bfa8271caade28
+  data.tar.gz: f72b90e8bb33e1edb00a805e1628057012756da73697c6148331e5668b16772b6000d45f35057ade6b4fcd9f759c10f619278da8f06c9463eee7ebe30911a0ee

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 4.3.0
+
+### Features
+
+- Support performance monitoring on Sidekiq workers [#1311](https://github.com/getsentry/sentry-ruby/pull/1311)
+
 ## 4.2.1
 
 - Use ::Rails::Railtie for checking Rails definition [#1284](https://github.com/getsentry/sentry-ruby/pull/1284)

data/Makefile

@@ -1,3 +1,7 @@
 build:
 	bundle install
 	gem build sentry-sidekiq.gemspec
+
+test:
+	bundle exec rspec
+	WITH_SENTRY_RAILS=1 bundle exec rspec spec/sentry/rails_spec.rb

data/lib/sentry/sidekiq/sentry_context_middleware.rb

@@ -13,13 +13,28 @@ module Sentry
         scope.set_tags(queue: queue, jid: job["jid"])
         scope.set_extras(sidekiq: job.merge("queue" => queue))
         scope.set_transaction_name(context_filter.transaction_name)
+        transaction = Sentry.start_transaction(name: scope.transaction_name, op: "sidekiq")
+        scope.set_span(transaction) if transaction
 
-        yield
+        begin
+          yield
+        rescue => e
+          finish_transaction(transaction, 500)
+          raise
+        end
 
+        finish_transaction(transaction, 200)
         # don't need to use ensure here
         # if the job failed, we need to keep the scope for error handler. and the scope will be cleared there
         scope.clear
       end
+
+      def finish_transaction(transaction, status)
+        return unless transaction
+
+        transaction.set_http_status(status)
+        transaction.finish
+      end
     end
   end
 end

data/lib/sentry/sidekiq/version.rb

@@ -1,5 +1,5 @@
 module Sentry
   module Sidekiq
-    VERSION = "4.2.1"
+    VERSION = "4.3.0"
   end
 end

data/sentry-sidekiq.gemspec

@@ -22,5 +22,5 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "sentry-ruby-core", "~> 4.2.0"
+  spec.add_dependency "sentry-ruby-core", "~> 4.3.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sentry-sidekiq
 version: !ruby/object:Gem::Version
-  version: 4.2.1
+  version: 4.3.0
 platform: ruby
 authors:
 - Sentry Team
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-02-18 00:00:00.000000000 Z
+date: 2021-03-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sentry-ruby-core
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.2.0
+        version: 4.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.2.0
+        version: 4.3.0
 description: A gem that provides Sidekiq integration for the Sentry error logger
 email: accounts@sentry.io
 executables: []