From 867a899d2f4ee7097e3ae480e25bdcbc5ef1e9ae Mon Sep 17 00:00:00 2001 From: Tobe O Date: Fri, 14 Dec 2018 01:55:36 -0500 Subject: [PATCH] patch --- lib/src/server.dart | 32 +++++++++++++++++++++++--------- test/password_test.dart | 2 +- 2 files changed, 24 insertions(+), 10 deletions(-) diff --git a/lib/src/server.dart b/lib/src/server.dart index 0804da44..730ad9f1 100644 --- a/lib/src/server.dart +++ b/lib/src/server.dart @@ -11,8 +11,15 @@ typedef Future ExtensionGrant( Future _getParam(RequestContext req, String name, String state, {bool body: false}) async { - var map = body == true ? await req.parseBody() : await req.parseQuery(); - var value = map.containsKey(name) ? map[name]?.toString() : null; + Map data; + + if (body == true) { + data = await req.parseBody().then((_) => req.bodyAsMap); + } else { + data = req.queryParameters; + } + + var value = data.containsKey(name) ? data[name]?.toString() : null; if (value?.isNotEmpty != true) { throw new AuthorizationException( @@ -30,8 +37,15 @@ Future _getParam(RequestContext req, String name, String state, Future> _getScopes(RequestContext req, {bool body: false}) async { - var map = body == true ? await req.parseBody() : await req.parseQuery(); - return map['scope']?.toString()?.split(' ') ?? []; + Map data; + + if (body == true) { + data = await req.parseBody().then((_) => req.bodyAsMap); + } else { + data = req.queryParameters; + } + + return data['scope']?.toString()?.split(' ') ?? []; } /// An OAuth2 authorization server, which issues access tokens to third parties. @@ -114,7 +128,7 @@ abstract class AuthorizationServer { Iterable scopes, RequestContext req, ResponseContext res) async { - var body = await req.parseBody(); + var body = await req.parseBody().then((_) => req.bodyAsMap); throw new AuthorizationException( new ErrorResponse( ErrorResponse.unsupportedResponseType, @@ -133,7 +147,7 @@ abstract class AuthorizationServer { Iterable scopes, RequestContext req, ResponseContext res) async { - var body = await req.parseBody(); + var body = await req.parseBody().then((_) => req.bodyAsMap); throw new AuthorizationException( new ErrorResponse( ErrorResponse.unsupportedResponseType, @@ -147,7 +161,7 @@ abstract class AuthorizationServer { /// Performs a client credentials grant. Only use this in situations where the client is 100% trusted. Future clientCredentialsGrant( Client client, RequestContext req, ResponseContext res) async { - var body = await req.parseBody(); + var body = await req.parseBody().then((_) => req.bodyAsMap); throw new AuthorizationException( new ErrorResponse( ErrorResponse.unsupportedResponseType, @@ -164,7 +178,7 @@ abstract class AuthorizationServer { String state = ''; try { - var query = await req.parseQuery(); + var query = req.queryParameters; state = query['state']?.toString() ?? ''; var responseType = await _getParam(req, 'response_type', state); @@ -288,7 +302,7 @@ abstract class AuthorizationServer { try { AuthorizationTokenResponse response; - var body = await req.parseBody(); + var body = await req.parseBody().then((_) => req.bodyAsMap); state = body['state']?.toString() ?? ''; diff --git a/test/password_test.dart b/test/password_test.dart index b96ecfa8..b980ab15 100644 --- a/test/password_test.dart +++ b/test/password_test.dart @@ -121,7 +121,7 @@ class _AuthorizationServer orElse: () => null); if (user == null) { - var body = await req.parseBody(); + var body = await req.parseBody().then((_) => req.bodyAsMap); throw new AuthorizationException( new ErrorResponse( ErrorResponse.accessDenied,