appsignal 4.3.2 → 4.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 42879f53736b4c57af015ffd56698a8e5e7ac1d3add47169067ae464d6fecf57
-  data.tar.gz: ef13a0ee3e7c64243d89c48e00bb2ea4a43f43869ad5f33f47690918234cb548
+  metadata.gz: d0ba9c95d270a611f4b26314db6d6a62a5ffb59e76bd8b797e4f272557ec70d8
+  data.tar.gz: 2318a7c02f13fe76b5d2c9c054eb900f9e3342c4b5e5d8108285b709a3610a74
 SHA512:
-  metadata.gz: 1a0c12865aea57f0602ca23346deaf74a9cb0e350f45335eb3f8511a14e3e82ab3e17cd57e76bf5b567a0621232b15832cf9c89fb2355195d31db650ed84efd9
-  data.tar.gz: e14293ffbec48adfbd1459138e7d71f8bcbbaebc2cde6684c94b7eebd20973cae4069cc1099fc6742783e013dc41643342e9c68c36cf8841f887cfe886bb19da
+  metadata.gz: 55b25b22f9a2d60c4973c547bd63aa8de67d6fcb83ac5aa47a63b8c0bf19e368bf09c4f09c1362c7552a133de41321e93957f1ed965973fcd818c0ecce0889fa
+  data.tar.gz: cd0c243ee29a67648b3bb397ae2699b16982395501d43b16821e91a063eec6d6f5f3551e190ad7f7ade12c5b264581aede02e83186f69ceeccf89fc29e509e6f

data/CHANGELOG.md

@@ -1,5 +1,40 @@
 # AppSignal for Ruby gem Changelog
 
+## 4.3.3
+
+_Published on 2025-01-17._
+
+### Added
+
+- Add support for the [Ownership](https://github.com/ankane/ownership) gem, which is used to mark different segments of the application as owned by specific teams.
+
+  The AppSignal sample will be tagged with the given owner:
+
+  ```ruby
+  class OrdersController < ApplicationController
+    owner :logistics
+    # Transactions for requests handled by this controller will be tagged
+    # in AppSignal with the "owner" tag set to "logistics"
+  end
+  ```
+
+  If several owners are set within the same transaction, the last owner will take precedence. If an error is reported in the transaction, the owner tag will be set to the owner that was set when the error was raised.
+
+  Set the [`ownership_set_namespace` configuration option](https://docs.appsignal.com/ruby/configuration/options.html#option-ownership_set_namespace) to `true` to also set the AppSignal sample's namespace to the owner. Note that doing so will cause existing performance and error incidents to be re-created with the new namespace.
+
+  Set the [`instrument_ownership` configuration option](https://docs.appsignal.com/ruby/configuration/options.html#option-instrument_ownership) to `false` to disable the integration with the Ownership gem.
+
+  (patch [865b348e](https://github.com/appsignal/appsignal-ruby/commit/865b348e1cf37193e3c31176dad47cbbd5d810bb))
+
+### Changed
+
+- Do not report `SystemExit` errors from our Rake integration. (patch [ad4f1a25](https://github.com/appsignal/appsignal-ruby/commit/ad4f1a250f00f1d1997932a3f108b0cf6b99ea43))
+- Do not report `SignalException` errors from our Rake integration. (patch [7c07a596](https://github.com/appsignal/appsignal-ruby/commit/7c07a59622426ca565c538a0c239270ebbd54f4c))
+
+### Fixed
+
+- Fix deprecation warnings when building the AppSignal gem's native extension on Ruby 3.4, by porting the extension to use the TypedData API. (patch [e2d186c9](https://github.com/appsignal/appsignal-ruby/commit/e2d186c9e79a5d03d3692c6ed84ca085e1383031))
+
 ## 4.3.2
 
 _Published on 2024-12-31._

data/build_matrix.yml

@@ -88,7 +88,7 @@ matrix:
       - "rails-8.0"
 
   ruby:
-    - ruby: "3.4.0-rc1"
+    - ruby: "3.4.1"
     - ruby: "3.3.4"
     - ruby: "3.2.5"
     - ruby: "3.1.6"
@@ -103,7 +103,7 @@ matrix:
     - gem: "dry-monitor"
       only:
         ruby:
-          - "3.4.0-rc1"
+          - "3.4.1"
           - "3.3.4"
           - "3.2.5"
           - "3.1.6"
@@ -112,7 +112,7 @@ matrix:
     - gem: "hanami-2.0"
       only:
         ruby:
-          - "3.4.0-rc1"
+          - "3.4.1"
           - "3.3.4"
           - "3.2.5"
           - "3.1.6"
@@ -120,7 +120,7 @@ matrix:
     - gem: "hanami-2.1"
       only:
         ruby:
-          - "3.4.0-rc1"
+          - "3.4.1"
           - "3.3.4"
           - "3.2.5"
           - "3.1.6"
@@ -128,16 +128,17 @@ matrix:
     - gem: "hanami-2.2"
       only:
         ruby:
-          - "3.4.0-rc1"
+          - "3.4.1"
           - "3.3.4"
           - "3.2.5"
           - "3.1.6"
     - gem: "http5"
+    - gem: "ownership"
     - gem: "padrino"
     - gem: "psych-3"
       only:
         ruby:
-          - "3.4.0-rc1"
+          - "3.4.1"
           - "3.3.4"
           - "3.2.5"
           - "3.1.6"
@@ -146,7 +147,7 @@ matrix:
     - gem: "psych-4"
       only:
         ruby:
-          - "3.4.0-rc1"
+          - "3.4.1"
           - "3.3.4"
           - "3.2.5"
           - "3.1.6"
@@ -172,7 +173,7 @@ matrix:
     - gem: "rails-7.0"
       only:
         ruby:
-          - "3.4.0-rc1"
+          - "3.4.1"
           - "3.3.4"
           - "3.2.5"
           - "3.1.6"
@@ -182,7 +183,7 @@ matrix:
     - gem: "rails-7.1"
       only:
         ruby:
-          - "3.4.0-rc1"
+          - "3.4.1"
           - "3.3.4"
           - "3.2.5"
           - "3.1.6"
@@ -191,7 +192,7 @@ matrix:
     - gem: "rails-7.2"
       only:
         ruby:
-          - "3.4.0-rc1"
+          - "3.4.1"
           - "3.3.4"
           - "3.2.5"
           - "3.1.6"
@@ -199,7 +200,7 @@ matrix:
     - gem: "rails-8.0"
       only:
         ruby:
-          - "3.4.0-rc1"
+          - "3.4.1"
           - "3.3.4"
           - "3.2.5"
     - gem: "sequel"

data/ext/appsignal_extension.c

@@ -15,6 +15,30 @@ static inline VALUE make_ruby_string(appsignal_string_t string) {
   return str;
 }
 
+const rb_data_type_t transaction_data_type = {
+  .wrap_struct_name = "Appsignal::Extension::Transaction",
+  .function = {
+    .dfree = appsignal_free_transaction,
+  },
+  .flags = RUBY_TYPED_FREE_IMMEDIATELY,
+};
+
+const rb_data_type_t data_data_type = {
+  .wrap_struct_name = "Appsignal::Extension::Data",
+  .function = {
+    .dfree = appsignal_free_data,
+  },
+  .flags = RUBY_TYPED_FREE_IMMEDIATELY,
+};
+
+const rb_data_type_t span_data_type = {
+  .wrap_struct_name = "Appsignal::Extension::Span",
+  .function = {
+    .dfree = appsignal_free_span,
+  },
+  .flags = RUBY_TYPED_FREE_IMMEDIATELY,
+};
+
 VALUE Appsignal;
 VALUE Extension;
 VALUE Transaction;
@@ -64,7 +88,7 @@ static VALUE start_transaction(VALUE self, VALUE transaction_id, VALUE namespace
   );
 
   if (transaction) {
-    return Data_Wrap_Struct(Transaction, NULL, appsignal_free_transaction, transaction);
+    return TypedData_Wrap_Struct(Transaction, &transaction_data_type, transaction);
   } else {
     return Qnil;
   }
@@ -75,7 +99,7 @@ static VALUE start_event(VALUE self, VALUE gc_duration_ms) {
 
   Check_Type(gc_duration_ms, T_FIXNUM);
 
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   appsignal_start_event(transaction, NUM2LONG(gc_duration_ms));
 
@@ -91,7 +115,7 @@ static VALUE finish_event(VALUE self, VALUE name, VALUE title, VALUE body, VALUE
   Check_Type(title, T_STRING);
   Check_Type(body_format, T_FIXNUM);
 
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   body_type = TYPE(body);
   if (body_type == T_STRING) {
@@ -103,8 +127,8 @@ static VALUE finish_event(VALUE self, VALUE name, VALUE title, VALUE body, VALUE
         FIX2INT(body_format),
         FIX2LONG(gc_duration_ms)
     );
-  } else if (body_type == RUBY_T_DATA) {
-    Data_Get_Struct(body, appsignal_data_t, body_data);
+  } else if (body_type == T_DATA) {
+    TypedData_Get_Struct(body, appsignal_data_t, &data_data_type, body_data);
     appsignal_finish_event_data(
         transaction,
         make_appsignal_string(name),
@@ -134,7 +158,7 @@ static VALUE record_event(VALUE self, VALUE name, VALUE title, VALUE body, VALUE
   }
   Check_Type(body_format, T_FIXNUM);
 
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   body_type = TYPE(body);
   if (body_type == T_STRING) {
@@ -147,8 +171,8 @@ static VALUE record_event(VALUE self, VALUE name, VALUE title, VALUE body, VALUE
         NUM2LONG(duration),
         NUM2LONG(gc_duration_ms)
     );
-  } else if (body_type == RUBY_T_DATA) {
-    Data_Get_Struct(body, appsignal_data_t, body_data);
+  } else if (body_type == T_DATA) {
+    TypedData_Get_Struct(body, appsignal_data_t, &data_data_type, body_data);
     appsignal_record_event_data(
         transaction,
         make_appsignal_string(name),
@@ -171,10 +195,10 @@ static VALUE set_transaction_error(VALUE self, VALUE name, VALUE message, VALUE
 
   Check_Type(name, T_STRING);
   Check_Type(message, T_STRING);
-  Check_Type(backtrace, RUBY_T_DATA);
 
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
-  Data_Get_Struct(backtrace, appsignal_data_t, backtrace_data);
+  backtrace_data = rb_check_typeddata(backtrace, &data_data_type);
+
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   appsignal_set_transaction_error(
       transaction,
@@ -190,10 +214,10 @@ static VALUE set_transaction_sample_data(VALUE self, VALUE key, VALUE payload) {
   appsignal_data_t* payload_data;
 
   Check_Type(key, T_STRING);
-  Check_Type(payload, RUBY_T_DATA);
 
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
-  Data_Get_Struct(payload, appsignal_data_t, payload_data);
+  payload_data = rb_check_typeddata(payload, &data_data_type);
+
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   appsignal_set_transaction_sample_data(
       transaction,
@@ -207,7 +231,7 @@ static VALUE set_transaction_action(VALUE self, VALUE action) {
   appsignal_transaction_t* transaction;
 
   Check_Type(action, T_STRING);
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   appsignal_set_transaction_action(
       transaction,
@@ -220,7 +244,7 @@ static VALUE set_transaction_namespace(VALUE self, VALUE namespace) {
   appsignal_transaction_t* transaction;
 
   Check_Type(namespace, T_STRING);
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   appsignal_set_transaction_namespace(
       transaction,
@@ -238,7 +262,7 @@ static VALUE set_transaction_queue_start(VALUE self, VALUE queue_start) {
       rb_raise(rb_eTypeError, "queue_start should be an Integer");
   }
 
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   appsignal_set_transaction_queue_start(
       transaction,
@@ -252,7 +276,7 @@ static VALUE set_transaction_metadata(VALUE self, VALUE key, VALUE value) {
 
   Check_Type(key, T_STRING);
   Check_Type(value, T_STRING);
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   appsignal_set_transaction_metadata(
       transaction,
@@ -267,7 +291,7 @@ static VALUE finish_transaction(VALUE self, VALUE gc_duration_ms) {
   int sample;
 
   Check_Type(gc_duration_ms, T_FIXNUM);
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   sample = appsignal_finish_transaction(transaction, NUM2LONG(gc_duration_ms));
   return sample == 1 ? Qtrue : Qfalse;
@@ -278,7 +302,7 @@ static VALUE duplicate_transaction(VALUE self, VALUE new_transaction_id) {
   appsignal_transaction_t* duplicate_transaction;
 
   Check_Type(new_transaction_id, T_STRING);
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   duplicate_transaction = appsignal_duplicate_transaction(
       transaction,
@@ -286,10 +310,9 @@ static VALUE duplicate_transaction(VALUE self, VALUE new_transaction_id) {
   );
 
   if (duplicate_transaction) {
-      return Data_Wrap_Struct(
+      return TypedData_Wrap_Struct(
           Transaction,
-          NULL,
-          appsignal_free_transaction,
+          &transaction_data_type,
           duplicate_transaction
       );
   } else {
@@ -300,7 +323,7 @@ static VALUE duplicate_transaction(VALUE self, VALUE new_transaction_id) {
 static VALUE complete_transaction(VALUE self) {
   appsignal_transaction_t* transaction;
 
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   appsignal_complete_transaction(transaction);
   return Qnil;
@@ -310,7 +333,7 @@ static VALUE transaction_to_json(VALUE self) {
   appsignal_transaction_t* transaction;
   appsignal_string_t json;
 
-  Data_Get_Struct(self, appsignal_transaction_t, transaction);
+  TypedData_Get_Struct(self, appsignal_transaction_t, &transaction_data_type, transaction);
 
   json = appsignal_transaction_to_json(transaction);
 
@@ -327,7 +350,7 @@ static VALUE data_map_new(VALUE self) {
   data = appsignal_data_map_new();
 
   if (data) {
-    return Data_Wrap_Struct(Data, NULL, appsignal_free_data, data);
+    return TypedData_Wrap_Struct(Data, &data_data_type, data);
   } else {
     return Qnil;
   }
@@ -339,7 +362,7 @@ static VALUE data_array_new(VALUE self) {
   data = appsignal_data_array_new();
 
   if (data) {
-    return Data_Wrap_Struct(Data, NULL, appsignal_free_data, data);
+    return TypedData_Wrap_Struct(Data, &data_data_type, data);
   } else {
     return Qnil;
   }
@@ -351,7 +374,7 @@ static VALUE data_set_string(VALUE self, VALUE key, VALUE value) {
   Check_Type(key, T_STRING);
   Check_Type(value, T_STRING);
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_map_set_string(
     data,
@@ -371,7 +394,7 @@ static VALUE data_set_integer(VALUE self, VALUE key, VALUE value) {
     rb_raise(rb_eTypeError, "wrong argument type %s (expected Integer)", rb_obj_classname(value));
   }
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_map_set_integer(
     data,
@@ -388,7 +411,7 @@ static VALUE data_set_float(VALUE self, VALUE key, VALUE value) {
   Check_Type(key, T_STRING);
   Check_Type(value, T_FLOAT);
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_map_set_float(
     data,
@@ -404,7 +427,7 @@ static VALUE data_set_boolean(VALUE self, VALUE key, VALUE value) {
 
   Check_Type(key, T_STRING);
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_map_set_boolean(
     data,
@@ -420,7 +443,7 @@ static VALUE data_set_nil(VALUE self, VALUE key) {
 
   Check_Type(key, T_STRING);
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_map_set_null(
     data,
@@ -435,10 +458,9 @@ static VALUE data_set_data(VALUE self, VALUE key, VALUE value) {
   appsignal_data_t* value_data;
 
   Check_Type(key, T_STRING);
-  Check_Type(value, RUBY_T_DATA);
+  value_data = rb_check_typeddata(value, &data_data_type);
 
-  Data_Get_Struct(self, appsignal_data_t, data);
-  Data_Get_Struct(value, appsignal_data_t, value_data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_map_set_data(
     data,
@@ -454,7 +476,7 @@ static VALUE data_append_string(VALUE self, VALUE value) {
 
   Check_Type(value, T_STRING);
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_array_append_string(
     data,
@@ -472,7 +494,7 @@ static VALUE data_append_integer(VALUE self, VALUE value) {
     rb_raise(rb_eTypeError, "wrong argument type %s (expected Integer)", rb_obj_classname(value));
   }
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_array_append_integer(
     data,
@@ -487,7 +509,7 @@ static VALUE data_append_float(VALUE self, VALUE value) {
 
   Check_Type(value, T_FLOAT);
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_array_append_float(
     data,
@@ -500,7 +522,7 @@ static VALUE data_append_float(VALUE self, VALUE value) {
 static VALUE data_append_boolean(VALUE self, VALUE value) {
   appsignal_data_t* data;
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_array_append_boolean(
     data,
@@ -513,7 +535,7 @@ static VALUE data_append_boolean(VALUE self, VALUE value) {
 static VALUE data_append_nil(VALUE self) {
   appsignal_data_t* data;
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_array_append_null(data);
 
@@ -524,10 +546,9 @@ static VALUE data_append_data(VALUE self, VALUE value) {
   appsignal_data_t* data;
   appsignal_data_t* value_data;
 
-  Check_Type(value, RUBY_T_DATA);
+  value_data = rb_check_typeddata(value, &data_data_type);
 
-  Data_Get_Struct(self, appsignal_data_t, data);
-  Data_Get_Struct(value, appsignal_data_t, value_data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   appsignal_data_array_append_data(
     data,
@@ -541,12 +562,12 @@ static VALUE data_equal(VALUE self, VALUE other) {
   appsignal_data_t* data;
   appsignal_data_t* other_data;
 
-  if (TYPE(other) != RUBY_T_DATA) {
+  if (TYPE(other) != T_DATA) {
     return Qfalse;
   }
 
-  Data_Get_Struct(self, appsignal_data_t, data);
-  Data_Get_Struct(other, appsignal_data_t, other_data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
+  TypedData_Get_Struct(other, appsignal_data_t, &data_data_type, other_data);
 
   if (appsignal_data_equal(data, other_data) == 1) {
     return Qtrue;
@@ -559,7 +580,7 @@ static VALUE data_to_s(VALUE self) {
   appsignal_data_t* data;
   appsignal_string_t json;
 
-  Data_Get_Struct(self, appsignal_data_t, data);
+  TypedData_Get_Struct(self, appsignal_data_t, &data_data_type, data);
 
   json = appsignal_data_to_json(data);
 
@@ -578,7 +599,7 @@ static VALUE root_span_new(VALUE self, VALUE namespace) {
   span = appsignal_create_root_span(make_appsignal_string(namespace));
 
   if (span) {
-    return Data_Wrap_Struct(Span, NULL, appsignal_free_span, span);
+    return TypedData_Wrap_Struct(Span, &span_data_type, span);
   } else {
     return Qnil;
   }
@@ -588,12 +609,12 @@ static VALUE child_span_new(VALUE self) {
   appsignal_span_t* parent;
   appsignal_span_t* span;
 
-  Data_Get_Struct(self, appsignal_span_t, parent);
+  TypedData_Get_Struct(self, appsignal_span_t, &span_data_type, parent);
 
   span = appsignal_create_child_span(parent);
 
   if (span) {
-    return Data_Wrap_Struct(Span, NULL, appsignal_free_span, span);
+    return TypedData_Wrap_Struct(Span, &span_data_type, span);
   } else {
     return Qnil;
   }
@@ -604,7 +625,7 @@ static VALUE set_span_name(VALUE self, VALUE name) {
 
   Check_Type(name, T_STRING);
 
-  Data_Get_Struct(self, appsignal_span_t, span);
+  TypedData_Get_Struct(self, appsignal_span_t, &span_data_type, span);
 
   appsignal_set_span_name(span, make_appsignal_string(name));
   return Qnil;
@@ -616,10 +637,10 @@ static VALUE add_span_error(VALUE self, VALUE name, VALUE message, VALUE backtra
 
   Check_Type(name, T_STRING);
   Check_Type(message, T_STRING);
-  Check_Type(backtrace, RUBY_T_DATA);
 
-  Data_Get_Struct(self, appsignal_span_t, span);
-  Data_Get_Struct(backtrace, appsignal_data_t, backtrace_data);
+  backtrace_data = rb_check_typeddata(backtrace, &data_data_type);
+
+  TypedData_Get_Struct(self, appsignal_span_t, &span_data_type, span);
 
   appsignal_add_span_error(
       span,
@@ -635,10 +656,10 @@ static VALUE set_span_sample_data(VALUE self, VALUE key, VALUE payload) {
   appsignal_data_t* payload_data;
 
   Check_Type(key, T_STRING);
-  Check_Type(payload, RUBY_T_DATA);
 
-  Data_Get_Struct(self, appsignal_span_t, span);
-  Data_Get_Struct(payload, appsignal_data_t, payload_data);
+  payload_data = rb_check_typeddata(payload, &data_data_type);
+
+  TypedData_Get_Struct(self, appsignal_span_t, &span_data_type, span);
 
   appsignal_set_span_sample_data(
       span,
@@ -654,7 +675,7 @@ static VALUE set_span_attribute_string(VALUE self, VALUE key, VALUE value) {
   Check_Type(key, T_STRING);
   Check_Type(value, T_STRING);
 
-  Data_Get_Struct(self, appsignal_span_t, span);
+  TypedData_Get_Struct(self, appsignal_span_t, &span_data_type, span);
 
   appsignal_set_span_attribute_string(
     span,
@@ -675,7 +696,7 @@ static VALUE set_span_attribute_int(VALUE self, VALUE key, VALUE value) {
     rb_raise(rb_eTypeError, "wrong argument type %s (expected Integer)", rb_obj_classname(value));
   }
 
-  Data_Get_Struct(self, appsignal_span_t, span);
+  TypedData_Get_Struct(self, appsignal_span_t, &span_data_type, span);
 
   appsignal_set_span_attribute_int(
     span,
@@ -691,7 +712,7 @@ static VALUE set_span_attribute_bool(VALUE self, VALUE key, VALUE value) {
 
   Check_Type(key, T_STRING);
 
-  Data_Get_Struct(self, appsignal_span_t, span);
+  TypedData_Get_Struct(self, appsignal_span_t, &span_data_type, span);
 
   appsignal_set_span_attribute_bool(
     span,
@@ -708,7 +729,7 @@ static VALUE set_span_attribute_double(VALUE self, VALUE key, VALUE value) {
   Check_Type(key, T_STRING);
   Check_Type(value, T_FLOAT);
 
-  Data_Get_Struct(self, appsignal_span_t, span);
+  TypedData_Get_Struct(self, appsignal_span_t, &span_data_type, span);
 
   appsignal_set_span_attribute_double(
     span,
@@ -723,7 +744,7 @@ static VALUE span_to_json(VALUE self) {
   appsignal_span_t* span;
   appsignal_string_t json;
 
-  Data_Get_Struct(self, appsignal_span_t, span);
+  TypedData_Get_Struct(self, appsignal_span_t, &span_data_type, span);
 
   json = appsignal_span_to_json(span);
 
@@ -737,7 +758,7 @@ static VALUE span_to_json(VALUE self) {
 static VALUE close_span(VALUE self) {
   appsignal_span_t* span;
 
-  Data_Get_Struct(self, appsignal_span_t, span);
+  TypedData_Get_Struct(self, appsignal_span_t, &span_data_type, span);
 
   appsignal_close_span(span);
 
@@ -751,9 +772,8 @@ static VALUE a_log(VALUE self, VALUE group, VALUE severity, VALUE format, VALUE
   Check_Type(severity, T_FIXNUM);
   Check_Type(format, T_FIXNUM);
   Check_Type(message, T_STRING);
-  Check_Type(attributes, RUBY_T_DATA);
 
-  Data_Get_Struct(attributes, appsignal_data_t, attributes_data);
+  attributes_data = rb_check_typeddata(attributes, &data_data_type);
 
   appsignal_log(
       make_appsignal_string(group),
@@ -771,9 +791,8 @@ static VALUE set_gauge(VALUE self, VALUE key, VALUE value, VALUE tags) {
 
   Check_Type(key, T_STRING);
   Check_Type(value, T_FLOAT);
-  Check_Type(tags, RUBY_T_DATA);
 
-  Data_Get_Struct(tags, appsignal_data_t, tags_data);
+  tags_data = rb_check_typeddata(tags, &data_data_type);
 
   appsignal_set_gauge(
       make_appsignal_string(key),
@@ -788,9 +807,8 @@ static VALUE increment_counter(VALUE self, VALUE key, VALUE count, VALUE tags) {
 
   Check_Type(key, T_STRING);
   Check_Type(count, T_FLOAT);
-  Check_Type(tags, RUBY_T_DATA);
 
-  Data_Get_Struct(tags, appsignal_data_t, tags_data);
+  tags_data = rb_check_typeddata(tags, &data_data_type);
 
   appsignal_increment_counter(
       make_appsignal_string(key),
@@ -805,9 +823,8 @@ static VALUE add_distribution_value(VALUE self, VALUE key, VALUE value, VALUE ta
 
   Check_Type(key, T_STRING);
   Check_Type(value, T_FLOAT);
-  Check_Type(tags, RUBY_T_DATA);
 
-  Data_Get_Struct(tags, appsignal_data_t, tags_data);
+  tags_data = rb_check_typeddata(tags, &data_data_type);
 
   appsignal_add_distribution_value(
       make_appsignal_string(key),

data/lib/appsignal/config.rb

@@ -111,10 +111,12 @@ module Appsignal
       :ignore_namespaces => [],
       :instrument_http_rb => true,
       :instrument_net_http => true,
+      :instrument_ownership => true,
       :instrument_redis => true,
       :instrument_sequel => true,
       :log => "file",
       :logging_endpoint => "https://appsignal-endpoint.net",
+      :ownership_set_namespace => false,
       :request_headers => %w[
         HTTP_ACCEPT HTTP_ACCEPT_CHARSET HTTP_ACCEPT_ENCODING
         HTTP_ACCEPT_LANGUAGE HTTP_CACHE_CONTROL HTTP_CONNECTION
@@ -181,8 +183,10 @@ module Appsignal
       :files_world_accessible => "APPSIGNAL_FILES_WORLD_ACCESSIBLE",
       :instrument_http_rb => "APPSIGNAL_INSTRUMENT_HTTP_RB",
       :instrument_net_http => "APPSIGNAL_INSTRUMENT_NET_HTTP",
+      :instrument_ownership => "APPSIGNAL_INSTRUMENT_OWNERSHIP",
       :instrument_redis => "APPSIGNAL_INSTRUMENT_REDIS",
       :instrument_sequel => "APPSIGNAL_INSTRUMENT_SEQUEL",
+      :ownership_set_namespace => "APPSIGNAL_OWNERSHIP_SET_NAMESPACE",
       :running_in_container => "APPSIGNAL_RUNNING_IN_CONTAINER",
       :send_environment_metadata => "APPSIGNAL_SEND_ENVIRONMENT_METADATA",
       :send_params => "APPSIGNAL_SEND_PARAMS",

data/lib/appsignal/environment.rb

@@ -90,6 +90,7 @@ module Appsignal
       hanami
       hiredis
       mongo_ruby_driver
+      ownership
       padrino
       passenger
       puma

data/lib/appsignal/hooks/ownership.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module Appsignal
+  class Hooks
+    # @api private
+    class OwnershipHook < Appsignal::Hooks::Hook
+      register :ownership
+
+      def dependencies_present?
+        defined?(::Ownership) &&
+          Gem::Version.new(::Ownership::VERSION) >= Gem::Version.new("0.2.0") &&
+          Appsignal.config &&
+          Appsignal.config[:instrument_ownership]
+      end
+
+      def install
+        require "appsignal/integrations/ownership"
+
+        # If a transaction is created in a code context that has an owner,
+        # set the namespace of the transaction to the owner.
+        Appsignal::Transaction.after_create <<
+          Appsignal::Integrations::OwnershipIntegrationHelper.method(:after_create)
+
+        # If an error was reported in a code context that has an owner,
+        # set the namespace of the transaction to the owner.
+        # In some circumstances, this will be more accurate than the last owner
+        # that was set for the transaction, which is what would otherwise be
+        # reported.
+        Appsignal::Transaction.before_complete <<
+          Appsignal::Integrations::OwnershipIntegrationHelper.method(:before_complete)
+
+        # If an owner is set in a code context that has an active transaction,
+        # set the namespace of the transaction to the owner.
+        unless ::Ownership.singleton_class.included_modules.include?(
+          Appsignal::Integrations::OwnershipIntegration
+        )
+          ::Ownership.singleton_class.prepend Appsignal::Integrations::OwnershipIntegration
+        end
+
+        Appsignal::Environment.report_enabled("ownership")
+      end
+    end
+  end
+end

data/lib/appsignal/hooks.rb

@@ -84,6 +84,7 @@ require "appsignal/hooks/dry_monitor"
 require "appsignal/hooks/http"
 require "appsignal/hooks/mri"
 require "appsignal/hooks/net_http"
+require "appsignal/hooks/ownership"
 require "appsignal/hooks/passenger"
 require "appsignal/hooks/puma"
 require "appsignal/hooks/rake"

data/lib/appsignal/integrations/ownership.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+module Appsignal
+  module Integrations
+    # @api private
+    module OwnershipIntegration
+      # Implement the `around_change` logic by monkey-patching the reader,
+      # instead of by using the `around_change=` writer. This allows customers
+      # to use the `around_change=` writer in their own code without
+      # accidentally overriding AppSignal's instrumentation.
+      def around_change
+        proc do |owner, block|
+          OwnershipIntegrationHelper.set(Appsignal::Transaction.current, owner)
+
+          original = super
+
+          if original
+            original.call(owner, block)
+          else
+            block.call
+          end
+        end
+      end
+    end
+
+    module OwnershipIntegrationHelper
+      class << self
+        def set(transaction, owner)
+          return if owner.nil?
+
+          transaction.add_tags(:owner => owner)
+          transaction.set_namespace(owner) if set_namespace?
+        end
+
+        def after_create(transaction)
+          set(transaction, ::Ownership.owner)
+        end
+
+        def before_complete(transaction, error)
+          set(transaction, error.owner) if error.respond_to?(:owner)
+        end
+
+        private
+
+        def set_namespace?
+          Appsignal.config && Appsignal.config[:ownership_set_namespace]
+        end
+      end
+    end
+  end
+end

data/lib/appsignal/integrations/rake.rb

@@ -4,6 +4,16 @@ module Appsignal
   module Integrations
     # @api private
     module RakeIntegration
+      IGNORED_ERRORS = [
+        # Normal exits from the application we do not need to report
+        SystemExit,
+        SignalException
+      ].freeze
+
+      def self.ignored_error?(error)
+        IGNORED_ERRORS.include?(error.class)
+      end
+
       def execute(*args)
         transaction =
           if Appsignal.config[:enable_rake_performance_instrumentation]
@@ -16,8 +26,10 @@ module Appsignal
         end
       rescue Exception => error # rubocop:disable Lint/RescueException
         Appsignal::Integrations::RakeIntegrationHelper.register_at_exit_hook
-        transaction ||= _appsignal_create_transaction
-        transaction.set_error(error)
+        unless RakeIntegration.ignored_error?(error)
+          transaction ||= _appsignal_create_transaction
+          transaction.set_error(error)
+        end
         raise error
       ensure
         if transaction

data/lib/appsignal/transaction.rb

@@ -44,6 +44,40 @@ module Appsignal
         end
       end
 
+      # @api private
+      # @return [Array<Proc>]
+      # Add a block, if given, to be executed after a transaction is created.
+      # The block will be called with the transaction as an argument.
+      # Returns the array of blocks that will be executed after a transaction
+      # is created.
+      def after_create(&block)
+        @after_create ||= Set.new
+
+        return @after_create if block.nil?
+
+        @after_create << block
+      end
+
+      # @api private
+      # @return [Array<Proc>]
+      # Add a block, if given, to be executed before a transaction is completed.
+      # This happens after duplicating the transaction for each error that was
+      # reported in the transaction -- that is, when a transaction with
+      # several errors is completed, the block will be called once for each
+      # error, with the transaction (either the original one or a duplicate of it)
+      # that has each of the errors set.
+      # The block will be called with the transaction as the first argument,
+      # and the error reported by the transaction, if any, as the second argument.
+      # Returns the array of blocks that will be executed before a transaction is
+      # completed.
+      def before_complete(&block)
+        @before_complete ||= Set.new
+
+        return @before_complete if block.nil?
+
+        @before_complete << block
+      end
+
       # @api private
       def set_current_transaction(transaction)
         Thread.current[:appsignal_transaction] = transaction
@@ -137,6 +171,8 @@ module Appsignal
         @namespace,
         0
       ) || Appsignal::Extension::MockTransaction.new
+
+      run_after_create_hooks
     end
 
     # @api private
@@ -192,7 +228,11 @@ module Appsignal
           end
         end
       end
+
+      run_before_complete_hooks
+
       sample_data if should_sample
+
       @ext.complete
     end
 
@@ -596,6 +636,18 @@ module Appsignal
 
     attr_reader :breadcrumbs
 
+    def run_after_create_hooks
+      self.class.after_create.each do |block|
+        block.call(self)
+      end
+    end
+
+    def run_before_complete_hooks
+      self.class.before_complete.each do |block|
+        block.call(self, @error_set)
+      end
+    end
+
     def _set_error(error)
       backtrace = cleaned_backtrace(error.backtrace)
       @ext.set_error(

data/lib/appsignal/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Appsignal
-  VERSION = "4.3.2"
+  VERSION = "4.3.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 4.3.2
+  version: 4.3.3
 platform: ruby
 authors:
 - Robert Beekman
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-31 00:00:00.000000000 Z
+date: 2025-01-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: logger
@@ -210,6 +210,7 @@ files:
 - lib/appsignal/hooks/mongo_ruby_driver.rb
 - lib/appsignal/hooks/mri.rb
 - lib/appsignal/hooks/net_http.rb
+- lib/appsignal/hooks/ownership.rb
 - lib/appsignal/hooks/passenger.rb
 - lib/appsignal/hooks/puma.rb
 - lib/appsignal/hooks/que.rb
@@ -234,6 +235,7 @@ files:
 - lib/appsignal/integrations/mongo_ruby_driver.rb
 - lib/appsignal/integrations/net_http.rb
 - lib/appsignal/integrations/object.rb
+- lib/appsignal/integrations/ownership.rb
 - lib/appsignal/integrations/puma.rb
 - lib/appsignal/integrations/que.rb
 - lib/appsignal/integrations/railtie.rb