Skip to content

Commit eccad09

Browse files
committed
Fix linter warnings
1 parent f8faa30 commit eccad09

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

lib/datadog/appsec/api_security/route_extractor.rb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ def self.route_pattern(request)
5656
pattern = request.env[SINATRA_ROUTE_KEY].split(SINATRA_ROUTE_SEPARATOR, 2)[1]
5757
"#{request.script_name}#{pattern}"
5858
elsif request.env.key?(RAILS_ROUTE_KEY)
59-
route = request.env[RAILS_ROUTE_KEY]
6059
request.env[RAILS_ROUTE_KEY].path.spec.to_s.delete_suffix(RAILS_FORMAT_SUFFIX)
6160
elsif request.env.key?(RAILS_ROUTE_URI_PATTERN_KEY)
6261
request.env[RAILS_ROUTE_URI_PATTERN_KEY].delete_suffix(RAILS_FORMAT_SUFFIX)
@@ -79,7 +78,7 @@ def self.route_pattern(request)
7978
else
8079
Tracing::Contrib::Rack::RouteInference.read_or_infer(request.env)
8180
end
82-
rescue => e
81+
rescue
8382
AppSec.telemetry&.error('AppSec: Could not extract route pattern for APISecurity sampler')
8483

8584
nil

spec/datadog/appsec/api_security/route_extractor_spec.rb

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,11 @@
203203

204204
allow(request).to receive(:env).and_return({
205205
'action_dispatch.routes' => route_set,
206-
'action_dispatch.request.path_parameters' => { 'controller' => 'users', 'action' => 'show', 'id' => '1' }
206+
'action_dispatch.request.path_parameters' => {
207+
'controller' => 'users',
208+
'action' => 'show',
209+
'id' => '1'
210+
}
207211
})
208212

209213
expect(route_set).to receive(:request_class).and_raise(StandardError)

0 commit comments

Comments
 (0)