Skip to content
This repository

Fix for CVE-2012-6134 #1

Closed
wants to merge 1 commit into from

2 participants

Tom Pesman John Duff
Tom Pesman

I'm not sure everything is still working. I get:
(shopify) Authentication failure! invalid_credentials: OmniAuth::Strategies::OAuth2::CallbackError, OmniAuth::Strategies::OAuth2::CallbackError

Looks like there are some issues with the OAuth2 update: intridea/omniauth-oauth2#33

Maybe someone else has more experience with the OAuth2 gems?

John Duff
Owner
jduff commented

fixed by 1463b04 thanks!

John Duff jduff closed this
Tom Pesman

Thanks! :+1:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. +1 1  omniauth-shopify-oauth2.gemspec
2  omniauth-shopify-oauth2.gemspec
@@ -15,7 +15,7 @@ Gem::Specification.new do |s|
15 15 s.executables = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
16 16 s.require_paths = ['lib']
17 17
18   - s.add_runtime_dependency 'omniauth-oauth2', '~> 1.0.0'
  18 + s.add_runtime_dependency 'omniauth-oauth2', '~> 1.1.1'
19 19
20 20 s.add_development_dependency 'rspec', '~> 2.7.0'
21 21 s.add_development_dependency 'rake'

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.