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

migrate to a newer ruby_pg #10234

Closed
blshkv opened this issue Jul 2, 2018 · 4 comments
Closed

migrate to a newer ruby_pg #10234

blshkv opened this issue Jul 2, 2018 · 4 comments
Labels

Comments

@blshkv
Copy link

blshkv commented Jul 2, 2018

https://github.com/rapid7/metasploit-framework/blob/master/metasploit-framework.gemspec#L91

  # Used by the Metasploit data model, etc.
  # bound to 0.20 for Activerecord 4.2.8 deprecation warnings:
  # https://github.com/ged/ruby-pg/commit/c90ac644e861857ae75638eb6954b1cb49617090
  spec.add_runtime_dependency 'pg', '0.20.0'

many distros have moved to 0.21.0 or the latest 1.0.0.

Please consider migrating msf as well.

@blshkv
Copy link
Author

blshkv commented Jul 2, 2018

FYI, I have tried to run msf with 0.21.0 and got the following warning:

The PGconn, PGresult, and PGError constants are deprecated, and will be
removed as of version 1.0.

You should use PG::Connection, PG::Result, and PG::Error instead, respectively.

@busterb busterb added the bug label Jul 2, 2018
@kkirsche
Copy link
Contributor

kkirsche commented Aug 16, 2018

Does it make sense instead if we're going to do this work to just move to 1.0.0 which is the default gem install pg version?

I know that potentially means upgrading the Rails related ActiveRecord type things, but may be worth the extra work up front rather than putting it off till later.

@busterb
Copy link
Member

busterb commented Feb 27, 2020

We updated to 0.21 as of #11387 . I believe 1.0+ is underway as well as part of a RoR 5.x upgrade.

@busterb busterb closed this as completed Feb 27, 2020
@blshkv
Copy link
Author

blshkv commented Apr 1, 2020

well, yeah. RoR 4.x is dead, so you should have really pushed 1.0+ instead.
It took 2 years to fix this bug report, by the time you migrate to RoR 5.x (other 2 years probably?.. ;) it will reach EOL too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants