Skip to content

Commit 0c8673d

Browse files
committed
Fix validation error messages
1 parent 33a4ef4 commit 0c8673d

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

lib/doorkeeper/oauth/authorization_code_request.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ class AuthorizationCodeRequest
77
validate :attributes, error: :invalid_request
88
validate :client, error: :invalid_client
99
validate :grant, error: :invalid_grant
10-
validate :redirect_uri, error: :invalid_grant
10+
validate :redirect_uri, error: :invalid_redirect_uri
1111

1212
attr_accessor :server, :grant, :client, :redirect_uri, :access_token
1313

lib/doorkeeper/oauth/password_access_token_request.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ class PasswordAccessTokenRequest
66
include OAuth::Helpers
77

88
validate :client, error: :invalid_client
9-
validate :resource_owner, error: :invalid_grant
9+
validate :resource_owner, error: :invalid_resource_owner
1010
validate :scopes, error: :invalid_scope
1111

1212
attr_accessor :server, :resource_owner, :credentials, :access_token

spec/lib/oauth/authorization_code_request_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ module Doorkeeper::OAuth
5959
it "matches the redirect_uri with grant's one" do
6060
subject.redirect_uri = 'http://other.com'
6161
subject.validate
62-
expect(subject.error).to eq(:invalid_grant)
62+
expect(subject.error).to eq(:invalid_redirect_uri)
6363
end
6464

6565
it "matches the client with grant's one" do

spec/lib/oauth/password_access_token_request_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ module Doorkeeper::OAuth
4444
it 'requires the owner' do
4545
subject.resource_owner = nil
4646
subject.validate
47-
expect(subject.error).to eq(:invalid_grant)
47+
expect(subject.error).to eq(:invalid_resource_owner)
4848
end
4949

5050
it 'optionally accepts the client' do

0 commit comments

Comments
 (0)