cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Allow GET as well as POST for oauth#request_token and oauth#access_token

Browse source
This commit is contained in:
Tom Hughes 2013-09-23 13:47:05 +01:00
parent 5052ecab7c
commit 038c341918
2 changed files with 10 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
config/routes.rb
View file

@ -247,8 +247,8 @@ OpenStreetMap::Application.routes.draw do
match '/oauth/revoke' => 'oauth#revoke', :via => [:get, :post] match '/oauth/revoke' => 'oauth#revoke', :via => [:get, :post]
match '/oauth/authorize' => 'oauth#authorize', :via => [:get, :post], :as => :authorize match '/oauth/authorize' => 'oauth#authorize', :via => [:get, :post], :as => :authorize
match '/oauth/token' => 'oauth#token', :via => :get, :as => :token match '/oauth/token' => 'oauth#token', :via => :get, :as => :token
match '/oauth/request_token' => 'oauth#request_token', :via => :get, :as => :request_token match '/oauth/request_token' => 'oauth#request_token', :via => [:get, :post], :as => :request_token
match '/oauth/access_token' => 'oauth#access_token', :via => :get, :as => :access_token match '/oauth/access_token' => 'oauth#access_token', :via => [:get, :post], :as => :access_token
match '/oauth/test_request' => 'oauth#test_request', :via => :get, :as => :test_request match '/oauth/test_request' => 'oauth#test_request', :via => :get, :as => :test_request
# roles and banning pages # roles and banning pages

8
test/functional/oauth_controller_test.rb
View file

@ -28,10 +28,18 @@ class OauthControllerTest < ActionController::TestCase
{ :path => "/oauth/request_token", :method => :get }, { :path => "/oauth/request_token", :method => :get },
{ :controller => "oauth", :action => "request_token" } { :controller => "oauth", :action => "request_token" }
) )
assert_routing(
{ :path => "/oauth/request_token", :method => :post },
{ :controller => "oauth", :action => "request_token" }
)
assert_routing( assert_routing(
{ :path => "/oauth/access_token", :method => :get }, { :path => "/oauth/access_token", :method => :get },
{ :controller => "oauth", :action => "access_token" } { :controller => "oauth", :action => "access_token" }
) )
assert_routing(
{ :path => "/oauth/access_token", :method => :post },
{ :controller => "oauth", :action => "access_token" }
)
assert_routing( assert_routing(
{ :path => "/oauth/test_request", :method => :get }, { :path => "/oauth/test_request", :method => :get },
{ :controller => "oauth", :action => "test_request" } { :controller => "oauth", :action => "test_request" }