devise_oauth2_providable 1.0.4 → 1.0.5

data/app/controllers/devise/oauth2_providable/authorizations_controller.rb

@@ -32,7 +32,7 @@ module Devise
       def authorize_endpoint(allow_approval = false)
         Rack::OAuth2::Server::Authorize.new do |req, res|
           @client = Client.find_by_identifier(req.client_id) || req.bad_request!
-          res.redirect_uri, @redirect_uri = req.verify_redirect_uri!(@client.redirect_uri)
+          res.redirect_uri = @redirect_uri = req.verify_redirect_uri!(@client.redirect_uri)
           if allow_approval
             if params[:approve].present?
               case req.response_type

data/devise_oauth2_providable.gemspec

@@ -23,6 +23,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency(%q<pry>, ['0.9.6.2'])
   s.add_development_dependency(%q<factory_girl>, ['2.2.0'])
   s.add_development_dependency(%q<factory_girl_rspec>, ['0.0.1'])
+  s.add_development_dependency(%q<rake>, ['0.9.2.2'])
 
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")

data/lib/devise/oauth2_providable/version.rb

@@ -1,5 +1,5 @@
 module Devise
   module Oauth2Providable
-    VERSION = "1.0.4"
+    VERSION = "1.0.5"
   end
 end

data/spec/controllers/authorizations_controller_spec.rb

@@ -2,15 +2,31 @@ require 'spec_helper'
 
 describe Devise::Oauth2Providable::AuthorizationsController do
   describe 'GET #new' do
-    with :user
-    with :client
-    before do
-      sign_in user
-      get :new, :client_id => client.identifier, :redirect_uri => client.redirect_uri, :response_type => 'code', :use_route => 'devise_oauth2_providable'
+    context 'with valid redirect_uri' do
+      with :user
+      with :client
+      let(:redirect_uri) { client.redirect_uri }
+      before do
+        sign_in user
+        get :new, :client_id => client.identifier, :redirect_uri => redirect_uri, :response_type => 'code', :use_route => 'devise_oauth2_providable'
+      end
+      it { should respond_with :ok }
+      it { should respond_with_content_type :html }
+      it { should assign_to(:redirect_uri).with(redirect_uri) }
+      it { should assign_to(:response_type) }
+      it { should render_template 'devise/oauth2_providable/authorizations/new' }
+      it { should render_with_layout 'application' }
+    end
+    context 'with invalid redirect_uri' do
+      with :user
+      with :client
+      let(:redirect_uri) { 'http://example.com/foo/bar' }
+      before do
+        sign_in user
+        get :new, :client_id => client.identifier, :redirect_uri => redirect_uri, :response_type => 'code', :use_route => 'devise_oauth2_providable'
+      end
+      it { should respond_with :bad_request }
+      it { should respond_with_content_type :html }
     end
-    it { should respond_with :ok }
-    it { should respond_with_content_type :html }
-    it { should assign_to(:redirect_uri) }
-    it { should assign_to(:response_type) }
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: devise_oauth2_providable
 version: !ruby/object:Gem::Version 
-  hash: 31
+  hash: 29
   prerelease: 
   segments: 
   - 1
   - 0
-  - 4
-  version: 1.0.4
+  - 5
+  version: 1.0.5
 platform: ruby
 authors: 
 - Ryan Sonnek
@@ -15,12 +15,10 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-12-05 00:00:00 Z
+date: 2012-01-20 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: rails
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
+  version_requirements: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -31,12 +29,12 @@ dependencies:
         - 1
         - 0
         version: 3.1.0
-  version_requirements: *id001
+  requirement: *id001
+  prerelease: false
+  name: rails
   type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: devise
-  prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  version_requirements: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -47,12 +45,12 @@ dependencies:
         - 4
         - 3
         version: 1.4.3
-  version_requirements: *id002
+  requirement: *id002
+  prerelease: false
+  name: devise
   type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: rack-oauth2
-  prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  version_requirements: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -63,12 +61,12 @@ dependencies:
         - 11
         - 0
         version: 0.11.0
-  version_requirements: *id003
+  requirement: *id003
+  prerelease: false
+  name: rack-oauth2
   type: :runtime
 - !ruby/object:Gem::Dependency 
-  name: rspec-rails
-  prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  version_requirements: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -79,12 +77,12 @@ dependencies:
         - 6
         - 1
         version: 2.6.1
-  version_requirements: *id004
+  requirement: *id004
+  prerelease: false
+  name: rspec-rails
   type: :development
 - !ruby/object:Gem::Dependency 
-  name: sqlite3
-  prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  version_requirements: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -95,12 +93,12 @@ dependencies:
         - 3
         - 4
         version: 1.3.4
-  version_requirements: *id005
+  requirement: *id005
+  prerelease: false
+  name: sqlite3
   type: :development
 - !ruby/object:Gem::Dependency 
-  name: shoulda-matchers
-  prerelease: false
-  requirement: &id006 !ruby/object:Gem::Requirement 
+  version_requirements: &id006 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -113,12 +111,12 @@ dependencies:
         - beta
         - 3
         version: 1.0.0.beta3
-  version_requirements: *id006
+  requirement: *id006
+  prerelease: false
+  name: shoulda-matchers
   type: :development
 - !ruby/object:Gem::Dependency 
-  name: pry
-  prerelease: false
-  requirement: &id007 !ruby/object:Gem::Requirement 
+  version_requirements: &id007 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -130,12 +128,12 @@ dependencies:
         - 6
         - 2
         version: 0.9.6.2
-  version_requirements: *id007
+  requirement: *id007
+  prerelease: false
+  name: pry
   type: :development
 - !ruby/object:Gem::Dependency 
-  name: factory_girl
-  prerelease: false
-  requirement: &id008 !ruby/object:Gem::Requirement 
+  version_requirements: &id008 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -146,12 +144,12 @@ dependencies:
         - 2
         - 0
         version: 2.2.0
-  version_requirements: *id008
+  requirement: *id008
+  prerelease: false
+  name: factory_girl
   type: :development
 - !ruby/object:Gem::Dependency 
-  name: factory_girl_rspec
-  prerelease: false
-  requirement: &id009 !ruby/object:Gem::Requirement 
+  version_requirements: &id009 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
@@ -162,7 +160,26 @@ dependencies:
         - 0
         - 1
         version: 0.0.1
-  version_requirements: *id009
+  requirement: *id009
+  prerelease: false
+  name: factory_girl_rspec
+  type: :development
+- !ruby/object:Gem::Dependency 
+  version_requirements: &id010 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - "="
+      - !ruby/object:Gem::Version 
+        hash: 11
+        segments: 
+        - 0
+        - 9
+        - 2
+        - 2
+        version: 0.9.2.2
+  requirement: *id010
+  prerelease: false
+  name: rake
   type: :development
 description: Rails3 engine that adds OAuth2 Provider support to any application built with Devise authentication
 email: 
@@ -292,7 +309,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: devise_oauth2_providable
-rubygems_version: 1.8.5
+rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: OAuth2 Provider for Rails3 applications