search

fuCtor / grape-doorkeeper

Integration Grape with Doorkeeper
MIT License
52 stars 28 forks source link

Doorkeeper 2.0 & Grape 0.10 support #15

Open 23tux opened 9 years ago

23tux commented 9 years ago

Hey,

I just wanted to upgrade to doorkeeper 2.0 and grape 0.10 and noticed, that this gem doesn't work anymore. It seems that Doorkeeper::DoorkeeperForBuilder.create_doorkeeper_for(*args) has been removed from doorkeeper.

So, any idea on how to make this work with doorkeeper 2.0? If you have some hints on how to do this, I'll try to transfer it to doorkeeper 2.0 and make a PR

fuCtor commented 9 years ago

Hi, When I was writing this code, I looked at the source of the rails helpers and copy the necessary parts. You can check how it works now and rewrite this code.

fuCtor commented 9 years ago

May be this https://github.com/doorkeeper-gem/doorkeeper/wiki/ActionController::Metal-with-doorkeeper wil be usefull.

sdbondi commented 9 years ago

Hey @fuCtor - I've made a fork which works with doorkeeper 2.0. It works but it is not thoroughly tested and I'm not using doorkeeper for my app any more. Hopefully this code can help with a proper pull request.

https://github.com/fixate/grape-doorkeeper/tree/feature/doorkeeper_2.0_support https://github.com/fixate/grape-doorkeeper/commit/e950db89abf39aef6bde8c8a4b6b36a0ce3337c1