Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add http.route tag to rack #3345

Merged
merged 39 commits into from
Feb 8, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
f5320cc
Add crude span for Rails routing
lloeki Dec 21, 2023
1a1909c
Move span creation to Routing::RouteSet
lloeki Dec 21, 2023
047e41a
added routes to sinatra/grape and fixes up rails
zarirhamza Dec 27, 2023
52246bc
linter fixes
zarirhamza Dec 27, 2023
00cf5af
Handle all sorts of nesting
lloeki Jan 10, 2024
737b9ec
cleaned up and addressed most todos
zarirhamza Jan 10, 2024
2d9676a
removes unnecessary rails tracing
zarirhamza Jan 11, 2024
db033da
adds tracing to rack requests and basic testing
zarirhamza Jan 11, 2024
d3dfe32
Add crude span for Rails routing
lloeki Dec 21, 2023
2a26d9c
Move span creation to Routing::RouteSet
lloeki Dec 21, 2023
8e39733
added routes to sinatra/grape and fixes up rails
zarirhamza Dec 27, 2023
02761bb
linter fixes
zarirhamza Dec 27, 2023
016df11
Handle all sorts of nesting
lloeki Jan 10, 2024
898e959
cleaned up and addressed most todos
zarirhamza Jan 10, 2024
9353f02
removes unnecessary rails tracing
zarirhamza Jan 11, 2024
118bc82
adds tracing to rack requests and basic testing
zarirhamza Jan 11, 2024
604f07b
fixes grape flaky test
zarirhamza Jan 12, 2024
8159dac
Merge branch 'add-rails-routing-instrumentation' of ssh://github.com/…
zarirhamza Jan 12, 2024
48334db
Update gemfiles/*
zarirhamza Jan 12, 2024
b335dd5
remove kwargs to fix tests
zarirhamza Jan 12, 2024
2bc87c4
Merge branch 'add-rails-routing-instrumentation' of ssh://github.com/…
zarirhamza Jan 12, 2024
e4aa83a
Skip patching for Rails 3
marcotc Jan 13, 2024
78fe38d
Skip patching for Rails 3
marcotc Jan 13, 2024
2f901db
Skip for Rails < 4.2
marcotc Jan 16, 2024
16e3cb3
linter fixes
zarirhamza Jan 16, 2024
82aaa02
Try to fix Spring test failure
marcotc Jan 16, 2024
8419654
removes unnecessary constants
zarirhamza Jan 25, 2024
696a3d0
added comment and fixed nested rack routes
zarirhamza Jan 25, 2024
ce0db69
fixes leak and potential errors
zarirhamza Jan 30, 2024
4ad8a6d
fixed sinatra route when using script name
zarirhamza Feb 5, 2024
1f89257
adds tests
zarirhamza Feb 5, 2024
293e8c4
Update gemfiles/*
zarirhamza Feb 5, 2024
40fd7bb
update appraisal files
zarirhamza Feb 6, 2024
564c0bc
Merge branch 'master' into add-rails-routing-instrumentation
zarirhamza Feb 6, 2024
c7301d3
adds jruby tests and lint
zarirhamza Feb 6, 2024
b640089
Update gemfiles/*
zarirhamza Feb 6, 2024
128ce7d
fixed rakefile
zarirhamza Feb 6, 2024
6d429ea
Merge branch 'master' into add-rails-routing-instrumentation
zarirhamza Feb 7, 2024
6d0f118
rewrites as trace tag
zarirhamza Feb 8, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 10 additions & 1 deletion Rakefile
Original file line number Diff line number Diff line change
Expand Up @@ -261,6 +261,9 @@ TEST_METADATA = {
'redis-4' => '❌ 2.1 / ❌ 2.2 / ❌ 2.3 / ✅ 2.4 / ✅ 2.5 / ✅ 2.6 / ✅ 2.7 / ✅ 3.0 / ✅ 3.1 / ✅ 3.2 / ✅ 3.3 / ✅ jruby',
'redis-5' => '❌ 2.1 / ❌ 2.2 / ❌ 2.3 / ❌ 2.4 / ✅ 2.5 / ✅ 2.6 / ✅ 2.7 / ✅ 3.0 / ✅ 3.1 / ✅ 3.2 / ✅ 3.3 / ✅ jruby'
},
'routetest' => {
'multi-rack-app' => '❌ 2.1 / ❌ 2.2 / ❌ 2.3 / ❌ 2.4 / ✅ 2.5 / ✅ 2.6 / ✅ 2.7 / ✅ 3.0 / ✅ 3.1 / ✅ 3.2 / ✅ 3.3 / ✅ jruby'
},
'appsec:rack' => {
# Non-deprecated form of Regexp.new does not backport to Rack 1.x, see: https://github.com/rack/rack/pull/1998
'rack-1' => '✅ 2.1 / ✅ 2.2 / ✅ 2.3 / ✅ 2.4 / ✅ 2.5 / ✅ 2.6 / ✅ 2.7 / ✅ 3.0 / ✅ 3.1 / ✅ 3.2 / ❌ 3.3 / ✅ jruby',
Expand Down Expand Up @@ -343,7 +346,7 @@ namespace :spec do
task all: [:main, :benchmark,
:rails, :railsredis, :railsredis_activesupport, :railsactivejob,
:elasticsearch, :http, :redis, :sidekiq, :sinatra, :hanami, :hanami_autoinstrument,
:profiling]
:profiling, :routetest]

desc '' # "Explicitly hiding from `rake -T`"
RSpec::Core::RakeTask.new(:main) do |t, args|
Expand Down Expand Up @@ -378,6 +381,12 @@ namespace :spec do
t.rspec_opts = args.to_a.join(' ')
end

desc '' # "Explicitly hiding from `rake -T`"
RSpec::Core::RakeTask.new(:routetest) do |t, args|
t.pattern = 'spec/datadog/tracing/contrib/http_route_spec.rb'
t.rspec_opts = args.to_a.join(' ')
end

desc '' # "Explicitly hiding from `rake -T`"
RSpec::Core::RakeTask.new(:railsredis) do |t, args|
t.pattern = 'spec/datadog/tracing/contrib/rails/**/*redis*_spec.rb'
Expand Down
8 changes: 8 additions & 0 deletions appraisal/jruby-9.2.rb
Original file line number Diff line number Diff line change
Expand Up @@ -289,3 +289,11 @@
appraise 'core-old' do
gem 'dogstatsd-ruby', '~> 4'
end

appraise 'multi-rack-app' do
gem 'sinatra'
gem 'rack-contrib'
gem 'rack-test' # Dev dependencies for testing rack-based code
gem 'grape'
gem 'rails', '~> 5.2.1'
end
8 changes: 8 additions & 0 deletions appraisal/jruby-9.3.rb
Original file line number Diff line number Diff line change
Expand Up @@ -260,3 +260,11 @@
appraise 'core-old' do
gem 'dogstatsd-ruby', '~> 4'
end

appraise 'multi-rack-app' do
gem 'sinatra', '>= 3'
gem 'rack-contrib'
gem 'rack-test' # Dev dependencies for testing rack-based code
gem 'grape'
gem 'rails', '~> 5.2.1'
end
8 changes: 8 additions & 0 deletions appraisal/jruby-9.4.rb
Original file line number Diff line number Diff line change
Expand Up @@ -166,3 +166,11 @@
appraise 'core-old' do
gem 'dogstatsd-ruby', '~> 4'
end

appraise 'multi-rack-app' do
gem 'sinatra', '>= 3'
gem 'rack-contrib'
gem 'rack-test' # Dev dependencies for testing rack-based code
gem 'grape'
gem 'rails', '~> 6.1.0'
end
8 changes: 8 additions & 0 deletions appraisal/ruby-2.5.rb
Original file line number Diff line number Diff line change
Expand Up @@ -262,3 +262,11 @@
appraise 'core-old' do
gem 'dogstatsd-ruby', '~> 4'
end

appraise 'multi-rack-app' do
gem 'sinatra'
gem 'rack-contrib'
gem 'rack-test' # Dev dependencies for testing rack-based code
gem 'grape'
gem 'rails', '~> 5.2.1'
end
8 changes: 8 additions & 0 deletions appraisal/ruby-2.6.rb
Original file line number Diff line number Diff line change
Expand Up @@ -263,3 +263,11 @@
appraise 'core-old' do
gem 'dogstatsd-ruby', '~> 4'
end

appraise 'multi-rack-app' do
gem 'sinatra', '>= 3'
gem 'rack-contrib'
gem 'rack-test' # Dev dependencies for testing rack-based code
gem 'grape'
gem 'rails', '~> 5.2.1'
end
8 changes: 8 additions & 0 deletions appraisal/ruby-2.7.rb
Original file line number Diff line number Diff line change
Expand Up @@ -264,3 +264,11 @@
appraise 'core-old' do
gem 'dogstatsd-ruby', '~> 4'
end

appraise 'multi-rack-app' do
gem 'sinatra', '>= 3'
gem 'rack-contrib'
gem 'rack-test' # Dev dependencies for testing rack-based code
gem 'grape'
gem 'rails', '~> 5.2.1'
end
8 changes: 8 additions & 0 deletions appraisal/ruby-3.0.rb
Original file line number Diff line number Diff line change
Expand Up @@ -177,3 +177,11 @@
appraise 'core-old' do
gem 'dogstatsd-ruby', '~> 4'
end

appraise 'multi-rack-app' do
gem 'sinatra', '>= 3'
gem 'rack-contrib'
gem 'rack-test' # Dev dependencies for testing rack-based code
gem 'grape'
gem 'rails', '~> 6.1.0'
end
8 changes: 8 additions & 0 deletions appraisal/ruby-3.1.rb
Original file line number Diff line number Diff line change
Expand Up @@ -177,3 +177,11 @@
appraise 'core-old' do
gem 'dogstatsd-ruby', '~> 4'
end

appraise 'multi-rack-app' do
gem 'sinatra', '>= 3'
gem 'rack-contrib'
gem 'rack-test' # Dev dependencies for testing rack-based code
gem 'grape'
gem 'rails', '~> 6.1.0'
end
8 changes: 8 additions & 0 deletions appraisal/ruby-3.2.rb
Original file line number Diff line number Diff line change
Expand Up @@ -177,3 +177,11 @@
appraise 'core-old' do
gem 'dogstatsd-ruby', '~> 4'
end

appraise 'multi-rack-app' do
gem 'sinatra', '>= 3'
gem 'rack-contrib'
gem 'rack-test' # Dev dependencies for testing rack-based code
gem 'grape'
gem 'rails', '~> 6.1.0'
end
8 changes: 8 additions & 0 deletions appraisal/ruby-3.3.rb
Original file line number Diff line number Diff line change
Expand Up @@ -179,3 +179,11 @@
appraise 'core-old' do
gem 'dogstatsd-ruby', '~> 4'
end

appraise 'multi-rack-app' do
gem 'sinatra', '>= 3'
gem 'rack-contrib'
gem 'rack-test' # Dev dependencies for testing rack-based code
gem 'grape'
gem 'rails', '~> 6.1.0'
end
40 changes: 40 additions & 0 deletions gemfiles/jruby_9.2_multi_rack_app.gemfile

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading
Loading