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.

Sign up for GitHub

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

Jump to bottom

Rails 7.1: DEPRECATION WARNING: Calling ActiveRecord::Base.clear_active_connections! is deprecated. Please call the method directly on the connection handler; for example: ActiveRecord::Base.connection_handler.clear_active_connections! #205

Closed
andreavocado opened this issue Feb 14, 2024 · 1 comment · Fixed by #208
Closed

Rails 7.1: DEPRECATION WARNING: Calling ActiveRecord::Base.clear_active_connections! is deprecated. Please call the method directly on the connection handler; for example: ActiveRecord::Base.connection_handler.clear_active_connections! #205

andreavocado opened this issue Feb 14, 2024 · 1 comment · Fixed by #208

Comments

@andreavocado
Copy link

Please describe the issue

When running run_rpc method from gruf-rspec under Rails 7.1 the deprecation warning "DEPRECATION WARNING: Calling ActiveRecord::Base.clear_active_connections! is deprecated. Please call the method directly on the connection handler; for example: ActiveRecord::Base.connection_handler.clear_active_connections!" is shown.

This line should be migrated:

gruf/lib/gruf/interceptors/active_record/connection_reset.rb

Line 36 in 8ea05f6

target_classes.each(&:clear_active_connections!) if enabled?

How to Reproduce

Steps to reproduce the behavior:

Run any test with run_rpc method from gruf-rspec

What should happen?

The connection is cleared on the connection handler of AR and the deprecation warning is gone.

Anything else we should know?

  • gruf-2.19.0
  • gruf-rspec-1.0.0
  • rails-7.1.2
  • Ruby 3.2.3
@Abdullah-l Abdullah-l mentioned this issue Feb 22, 2024
Merged
@Abdullah-l
Copy link
Contributor

It appears in logs while using rails 7.1 with gruf in general not just with gruf-rspec

On an unrelated note, @splittingred is it just you maintaining gruf at the moment?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix rails clear_active_connections! deprecation warning Abdullah-l/gruf
2 participants
@Abdullah-l @andreavocado