mrpastewart/platform
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed auth failed test cases

Browse source
This commit is contained in:
thomashii 2023-06-05 19:00:42 +08:00
parent 8e251e553a
commit a806d7c6db
3 changed files with 24 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
packages/auth/example/client/example_client.http
View file

@ -1,12 +1,17 @@
### Load landing page
GET http://localhost:3000/ HTTP/1.1
### Success redirect
### login (call_back)
POST http://localhost:3000/login HTTP/1.1
Content-Type: application/json
Authorization: Basic jdoe1:password
### Failure redirect
### Success redirect (local)
POST http://localhost:3000/login HTTP/1.1
Content-Type: application/json
Authorization: Basic username:password
### Failure redirect (local)
POST http://localhost:3000/login HTTP/1.1
Content-Type: application/json
Authorization: Basic password:username

6
packages/auth/example/example2.dart
View file

@ -25,10 +25,14 @@ Future<Map<String, String>> verifier(String? username, String? password) async {
Future wireAuth(Angel app) async {
//auth.strategies['local'] = LocalAuthStrategy(verifier);
auth.strategies['local'] = LocalAuthStrategy(verifier, forceBasic: true, realm: 'test');
auth.strategies['local'] =
LocalAuthStrategy(verifier, forceBasic: true, realm: 'test');
await app.configure(auth.configureServer);
}
/*
* Backend for local test cases
*/
void main() async {
Angel app = Angel(reflector: MirrorsReflector());
AngelHttp angelHttp = AngelHttp(app, useZone: false);

17
packages/auth/lib/src/strategies/local.dart
View file

@ -81,15 +81,22 @@ class LocalAuthStrategy<User> extends AuthStrategy<User> {
}
}
// User authentication succeeded
if (verificationResult == true ||
(verificationResult is Map && verificationResult.isNotEmpty)) {
return verificationResult;
// User authentication succeeded can return Map(one element), User(non null) or true
if (verificationResult != null && verificationResult != false) {
if (verificationResult is Map && verificationResult.isNotEmpty) {
return verificationResult;
} else if (verificationResult is! Map) {
return verificationResult;
}
}
// Force basic if set
if (forceBasic) {
res.headers['www-authenticate'] = 'Basic realm="$realm"';
//res.headers['www-authenticate'] = 'Basic realm="$realm"';
res
..statusCode = 401
..headers['www-authenticate'] = 'Basic realm="$realm"';
await res.close();
return null;
}