intridea/omniauth-openid — Issue #3: initialize calls 'super' with name string instead of hash
Author: e2
Created: 2011-11-03 22:24:06+00:00
State: open

No patch yet, because I can see that omniauth/omniauth-openid are still a work in progress (based on rspec specs).

I''m opening a ticket to know when this is fixed, so I can test/patch/report/etc.

* lib/omniauth/strategies/open_id.rb:

super(app, (options[:name] || :open_id), &block)

Omniauth::Strategy expects a hash as last param, or arg() should be used

I changed it to:

super(app, name: ( options[:name] || :open_id), &block)

The following are related to omniauth instead, but occur when using omniauth-openid:

* omniauth-1.0.0/lib/strategy.rb:

self.camelizations can be nil, so I added:

self.camelizations ||= {}

* omniauth-1.0.0/lib/omniauth.rb:

options seems to be a hash, so I changed name() to:

def name
options[:name]
end

After that, I get 404 errors, but at least it stops crashing. Sorry for the lame report.