-
Notifications
You must be signed in to change notification settings - Fork 161
Bring omniauth-github
up-to-date
#61
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
Conversation
Ping @mbleigh & @intridea-shared |
I don't actively maintain this gem anymore. @sferik is this one of the many On Sun, Aug 28, 2016, 3:00 PM Tom Milewski notifications@github.com wrote:
|
@mbleigh All good, I just want to make sure it gets updated so that there's no longer any security concerns through out-of-date dependencies. As time permits, I'm happy to take over some maintenance responsibilities as well. |
Ping @sferik, are you an owner? |
No response in ~4 months would indicate that @sferik is not a maintainer. @mbleigh , any chance you could widen the maintainer pool to include @tmilewski so this could get merged and released to rubygems? |
@tmilewski you've been added. |
Sorry, just seeing this now… @tmilewski Welcome to the team! |
@mbleigh: All: |
@tmilewski the power is yours! |
@mbleigh Perfect, thank you! It's now live! |
@mbleigh - Please consider this PR and update RubyGems accordingly. The current version is out-of-date by over two years and contains security vulnerabilities. Thanks!
This PR does the following:
Security Updates
https
for the RubyGems URL to passbundler audit
Dependencies
omniauth
to~>1.3.2
omniauth-oauth2
to~> 1.4.0
rspec
to~> 3.5
Specs
RSpec 3+
syntaxHousekeeping
1.2.0
Closes #59
Closes #52
Closes #57