Skip to content

Commit

Permalink
Merge pull request #132 from appsignal/ruby-instrument-ownership-conf…
Browse files Browse the repository at this point in the history
…ig-option

Expect `instrument_ownership` config option for Ruby
  • Loading branch information
unflxw authored Jan 20, 2025
2 parents 3059410 + f78ba9b commit bc36448
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 3 deletions.
3 changes: 2 additions & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,5 @@ gem "rack"
gem "rspec"
gem "rubocop"
gem "sinatra"
gem "webrick"
gem "rackup"
gem "puma"
9 changes: 7 additions & 2 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -9,19 +9,24 @@ GEM
method_source (1.0.0)
mustermann (3.0.3)
ruby2_keywords (~> 0.0.1)
nio4r (2.7.4)
parallel (1.20.1)
parser (3.0.2.0)
ast (~> 2.4.1)
pry (0.14.1)
coderay (~> 1.1)
method_source (~> 1.0)
puma (6.5.0)
nio4r (~> 2.0)
rack (3.1.8)
rack-protection (4.1.0)
base64 (>= 0.1.0)
logger (>= 1.6.0)
rack (>= 3.0.0, < 4)
rack-session (2.0.0)
rack (>= 3.0.0)
rackup (2.2.1)
rack (>= 3)
rainbow (3.0.0)
regexp_parser (2.1.1)
rexml (3.3.9)
Expand Down Expand Up @@ -60,18 +65,18 @@ GEM
tilt (~> 2.0)
tilt (2.4.0)
unicode-display_width (2.0.0)
webrick (1.8.2)

PLATFORMS
ruby

DEPENDENCIES
pry
puma
rack
rackup
rspec
rubocop
sinatra
webrick

BUNDLED WITH
2.3.12
6 changes: 6 additions & 0 deletions spec/diagnose_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -405,11 +405,13 @@
/ ignore_namespaces: \[\]/,
/ instrument_http_rb: true/,
/ instrument_net_http: true/,
/ instrument_ownership: true/,
/ instrument_redis: true/,
/ instrument_sequel: true/,
/ log: #{quoted("file")}/,
/ logging_endpoint: #{quoted("https://appsignal-endpoint.net")}/,
/ name: #{quoted "DiagnoseTests"} \(Loaded from: file\)/,
/ ownership_set_namespace: false/,
/ push_api_key: "test" \(Loaded from: env\)/,
/ request_headers: \["HTTP_ACCEPT", "HTTP_ACCEPT_CHARSET", "HTTP_ACCEPT_ENCODING", "HTTP_ACCEPT_LANGUAGE", "HTTP_CACHE_CONTROL", "HTTP_CONNECTION", "CONTENT_LENGTH", "PATH_INFO", "HTTP_RANGE", "REQUEST_METHOD", "REQUEST_PATH", "SERVER_NAME", "SERVER_PORT", "SERVER_PROTOCOL"\]/, # rubocop:disable Layout/LineLength
/ send_environment_metadata: true/,
Expand Down Expand Up @@ -572,11 +574,13 @@
"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",
"name" => "DiagnoseTests",
"ownership_set_namespace" => false,
"push_api_key" => "test",
"request_headers" => [
"HTTP_ACCEPT",
Expand Down Expand Up @@ -754,10 +758,12 @@
"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" => [
"HTTP_ACCEPT",
"HTTP_ACCEPT_CHARSET",
Expand Down

0 comments on commit bc36448

Please sign in to comment.