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

Undo changes to library name and gemspec #1

Open
wants to merge 60 commits into
base: master
Choose a base branch
from

Conversation

joe1chen
Copy link

Thanks for merging all the pull requests and updating the gem. One problem with the most recent changes to rename the gem is that it makes the gem no longer a drop-in replacement for the original gem. Libraries such as openauth which have gem requirements on ruby-openid cannot be used with your gem.

mpd and others added 30 commits August 6, 2010 19:26
…g. ActiveSupport's) versions if they are already in-place
it should be "true" when required.
auto-submitting javascript off body attribute and into a timeout to
verify availability of form in the DOM.
auto-submitting javascript off body attribute and into a timeout to
verify availability of form in the DOM.
…setup_needed tests to verify setup_url in response.
…on OSX Lion.


See the upstream branch's pull request openid#22 for more details.
openid#22
…id transaction in progress form to be sure we are autosubmitting the correct form
Joseph Chen added 30 commits February 29, 2012 09:28
…ng URL fragments for Identifier Recycling.

The local_id is a canonical_id at the OP when the local_id is an iname at the RP. Matching claimed_ids instead of local_ids resolves the incompatibility. (reverse-merged from commit 4a17c08)
Conflicts:
	lib/openid.rb
	lib/openid/store/memcache.rb
	lib/openid/util.rb
	ruby-openid.gemspec
	test/test_accept.rb
	test/test_association.rb
	test/test_associationmanager.rb

Reverted Mike Mell's changes to memcached store in favor of agoragames memcached Dalli changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants