Skip to content

Conversation

jeffcarbs
Copy link

In an application where I'm using this gem, I'm also using the
rack-timeout gem (https://github.com/kch/rack-timeout), so use of
Timeout inside the Rack::OpenID class was finding Rack::Timeout,
which is not the same. This commit updates references to Timeout to
use the root namespace (::Timeout).

In an application where I'm using this gem, I'm also using the
rack-timeout gem (https://github.com/kch/rack-timeout), so use of
Timeout inside the Rack::OpenID class was finding Rack::Timeout,
which is not the same. This commit updates references to Timeout to
use the root namespace (::Timeout).
@teliosdev
Copy link

👍

@brondavies
Copy link

When is this going to get merged?

@kirley
Copy link

kirley commented Sep 22, 2014

Any update on when this might be merged?

simeonwillbanks pushed a commit to stitchfix/rack-openid that referenced this pull request Sep 24, 2014
ruby-openid#5

In an application where I'm using this gem, I'm also using the
rack-timeout gem (https://github.com/kch/rack-timeout), so use of
Timeout inside the Rack::OpenID class was finding Rack::Timeout,
which is not the same. This commit updates references to Timeout to
use the root namespace (::Timeout).

Conflicts:
	lib/rack/openid.rb
@bdarfler
Copy link

👍

Copy link
Member

@pboling pboling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

pboling added a commit to ruby-openid/rack-openid2 that referenced this pull request Sep 4, 2024
@pboling
Copy link
Member

pboling commented Sep 14, 2025

Ah, crap I failed to merge this first. I'll cherry-pick.

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.

6 participants