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

Resque instrumentation can require resolv-replace which will become a bundled gem in Ruby 3.4 #2396

Open
Earlopain opened this issue Jan 15, 2024 · 3 comments
Labels
community To tag external issues and PRs submitted by the community oct-dec qtr Possible FY Q3 candidate

Comments

@Earlopain
Copy link
Contributor

Description

resolv-replace will get the same treatment as base64 (see #2238/#2378/ruby changelog). This means that for users running Ruby 3.3 that set resque.use_ruby_dns to true a warning will be logged, and for Ruby 3.4 (when it gets released next year) an error will be thrown.

I tried finding this warning in the CI output here but I think this code path isn't being exercised in tests. This is the corresponding code:

Note that I haven't actually tested this myself in a proper environment, I'm just going through gems I currently depend on that make use of these gems without declaring them.

Expected Behavior

No warning is logged.

Steps to Reproduce

Set resque.use_ruby_dns to true with Ruby 3.3

For Maintainers Only or Hero Triaging this bug

Suggested Priority (P1,P2,P3,P4,P5):
Suggested T-Shirt size (S, M, L, XL, Unknown):

@workato-integration
Copy link

@github-actions github-actions bot added the community To tag external issues and PRs submitted by the community label Jan 15, 2024
@tannalynn
Copy link
Contributor

Thank you for bringing this to our attention, we didn't realize this. We'll take a look!

@fallwith
Copy link
Contributor

For this issue to be addressed in the same manner as #2378, we'd want to store a fork of resolve-replace.rb locally and then reference it with a require_relative and remove the existing require.

We welcome community PR submissions for this change and if in the absence of any we'll want to get after this some time in 2024.

@kford-newrelic kford-newrelic added estimate Issue needing estimation apr-jun qtr Represents proposed work item for the Apr-Jun quarter labels Feb 6, 2024
@kford-newrelic kford-newrelic removed the bug label Feb 6, 2024
@kford-newrelic kford-newrelic added 1 Story Point Estimate and removed estimate Issue needing estimation labels Feb 16, 2024
@kford-newrelic kford-newrelic moved this from Triage to In Quarter in Ruby Engineering Board Apr 1, 2024
@kford-newrelic kford-newrelic added jul-sep qtr Represents proposed work item for the Jul-Sep quarter and removed apr-jun qtr Represents proposed work item for the Apr-Jun quarter labels May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community To tag external issues and PRs submitted by the community oct-dec qtr Possible FY Q3 candidate
Projects
Status: In Quarter
Development

No branches or pull requests

5 participants