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

Upgrade to RSpec 2, Ruby 1.9.2p180 and Rails 3.0.5 #1

Closed
wants to merge 3 commits into from
Closed

Upgrade to RSpec 2, Ruby 1.9.2p180 and Rails 3.0.5 #1

wants to merge 3 commits into from

Conversation

eldavido
Copy link

@eldavido eldavido commented Jun 7, 2011

Gentlemen,

I need to use this gem in a production system that interfaces with an OAuth-protected resource. Can I trouble you to review this for inclusion in the original oauth-active-resource project? One of the RSpec tests is still failing (one succeeds after all my changes!) and I can't quite figure out why, any help would be appreciated.

Thanks,
DA

@eldavido
Copy link
Author

Any updates? jwagener, are you still maintaining this?

@bkabrda
Copy link

bkabrda commented Oct 19, 2011

Hi, does this pull request work for you? For example the OAuthActiveResource::Connection.get_without_decoding doesn't work for me, as the signature of build_request_headers has been changed. I'm currently trying to test the OAuthActiveResource::Resource, but nothing just seems to work with activeresource >= 3.0.0.

@eldavido
Copy link
Author

I wouldn't recommend using this gem unless you're willing to do a pretty serious overhaul. It's creaky and parts of it have started to break for us, and I've spent more time than I'd like fixing it.

That said, it does work for the most part, but I don't think we're using OAuthActiveResource::Resource. I tried to bug someone in Rails core to add OAuth support to ActiveResource, maybe if you try it'll help?

@ghost ghost closed this by deleting the head repository Jun 18, 2024
This pull request was closed.
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.

3 participants