Merge pull request #1003 from dv/make-native-auth-code-a-param

Make native redirect code a query param

NEWS.md

@@ -4,6 +4,7 @@ User-visible changes worth mentioning.
 
 ## master
 
+- [#1003] Use URL query param to pass through native redirect auth code so automated apps can find it.
 - [#868] `Scopes#&` and `Scopes#+` now take an array or any other enumerable
  object.
 - [#1019] Remove translation not in use: `invalid_resource_owner`.

lib/doorkeeper/rails/routes.rb

@@ -49,7 +49,7 @@ def authorization_routes(mapping)
  as: mapping[:as],
  controller: mapping[:controllers]
  ) do
- routes.get '/:code', action: :show, on: :member
+ routes.get '/native', action: :show, on: :member
  routes.get '/', action: :new, on: :member
  end
  end

spec/controllers/authorizations_controller_spec.rb

@@ -154,7 +154,7 @@ def translated_error_message(key)
 
  it 'should redirect immediately' do
  expect(response).to be_redirect
- expect(response.location).to match(/oauth\/authorize\//)
+ expect(response.location).to match(/oauth\/authorize\/native\?code=#{Doorkeeper::AccessGrant.first.token}/)
  end
 
  it 'should issue a grant' do

spec/requests/flows/authorization_code_spec.rb

@@ -29,6 +29,7 @@
 
  access_grant_should_exist_for(@client, @resource_owner)
 
+ url_should_have_param('code', Doorkeeper::AccessGrant.first.token)
  i_should_see 'Authorization code:'
  i_should_see Doorkeeper::AccessGrant.first.token
  end