Use RFC2396_PARSER explicitly to avoid the warning.

[eGreshkov]

Description

The URI library recently changed the URI::DEFAULT_PARSER to URI::RFC3986_PARSER.
https://github.com/ruby/uri/blob/master/lib/uri/common.rb#L22

Additionally, when using the URI::RFC3986_PARSER.escape method a warning is logged, telling that the method is obsolete and that URI::RFC2396_PARSER.escape should be used explicitly instead.
https://github.com/ruby/uri/blob/master/lib/uri/rfc3986_parser.rb#L155-L159

This results in a lot of warnings logged to stderr. This simple fix should do the trick :)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.16%. Comparing base (1cd952b) to head (bf7130d).
Report is 2 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2509   +/-   ##
=======================================
  Coverage   98.16%   98.16%           
=======================================
  Files         128      128           
  Lines        4848     4849    +1     
=======================================
+ Hits         4759     4760    +1     
  Misses         89       89           

Components	Coverage Δ
sentry-ruby	98.54% <100.00%> (+<0.01%)	⬆️
sentry-rails	97.07% <ø> (ø)
sentry-sidekiq	97.09% <ø> (ø)
sentry-resque	92.85% <ø> (ø)
sentry-delayed_job	95.65% <ø> (ø)
sentry-opentelemetry	99.31% <ø> (ø)

Files with missing lines	Coverage Δ
sentry-ruby/lib/sentry/net/http.rb	100.00% <100.00%> (ø)

[solnic]

Thank you for the PR! Please update to latest master though as we had some issues with the spec suite that should be fixed on master now.

[solnic]

Looks like in older rubies the constant does not exist:

uninitialized constant URI::RFC2396_PARSER

This is from 2.5 tests. There are more failing test suites with this error unfortunately, so we gotta have a fallback mechanism to make this work for old rubies too.

[eGreshkov]

Thanks for checking the tests so quickly, great catch!