Yubico/yubioath-flutter
1
1
Fork 0
mirror of https://github.com/Yubico/yubioath-flutter.git synced 2024-11-25 23:14:18 +03:00
Code Issues Packages Projects Releases Wiki Activity

updates to logging information

Browse Source
This commit is contained in:
Adam Velebil 2022-03-04 16:51:56 +01:00
parent 4c0bfe1bc4
commit d2b97dc4e1
No known key found for this signature in database
GPG Key ID: AC6D6B9D715FC084
4 changed files with 11 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/android/api/flutter_management_api_impl.dart
View File

@ -1,11 +1,8 @@
import 'package:flutter_riverpod/flutter_riverpod.dart';
import 'package:logging/logging.dart';
import '../command_providers.dart';
import 'impl.dart';
final _log = Logger('android.FManagementApiImpl');
class FManagementApiImpl extends FManagementApi {
final StateNotifierProviderRef _ref;
@ -13,7 +10,6 @@ class FManagementApiImpl extends FManagementApi {
@override
Future<void> updateDeviceInfo(String deviceInfoJson) async {
_log.info('Received: $deviceInfoJson');
_ref.read(androidYubikeyProvider.notifier).set(deviceInfoJson);
}
}

5
lib/android/api/flutter_oath_api_impl.dart
View File

@ -1,11 +1,8 @@
import 'package:flutter_riverpod/flutter_riverpod.dart';
import 'package:logging/logging.dart';
import '../oath/command_providers.dart';
import 'impl.dart';
final _log = Logger('android.FOathApiImpl');
class FOathApiImpl extends FOathApi {
final StateNotifierProviderRef _ref;
@ -13,13 +10,11 @@ class FOathApiImpl extends FOathApi {
@override
Future<void> updateOathCredentials(String credentialListJson) async {
_log.info('Received: $credentialListJson');
_ref.read(androidCredentialsProvider.notifier).set(credentialListJson);
}
@override
Future<void> updateSession(String sessionJson) async {
_log.info('Received: $sessionJson');
_ref.read(androidStateProvider.notifier).set(sessionJson);
}
}

4
lib/android/command_providers.dart
View File

@ -20,7 +20,7 @@ class _YubikeyProvider extends StateNotifier<YubiKeyData?> {
void set(String input) {
try {
if (input.isEmpty) {
_log.info('Yubikey was detached.');
_log.config('Yubikey was detached.');
state = null;
return;
}
@ -50,7 +50,7 @@ class _YubikeyProvider extends StateNotifier<YubiKeyData?> {
: DeviceNode.usbYubiKey(DevicePath([]), name, -1, deviceInfo);
state = YubiKeyData(deviceNode, name, deviceInfo);
} on Exception catch (e) {
_log.info('Invalid data for yubikey: $input. $e');
_log.config('Invalid data for yubikey: $input. $e');
state = null;
}
}

25
lib/android/oath/state.dart
View File

@ -38,7 +38,7 @@ class _AndroidOathStateNotifier extends OathStateNotifier {
try {
await _api.reset();
} catch (e) {
_log.info('Calling reset failed with exception: $e');
_log.config('Calling reset failed with exception: $e');
}
}
@ -54,7 +54,7 @@ class _AndroidOathStateNotifier extends OathStateNotifier {
}
return Pair(unlockSuccess, false); // TODO: provide correct second param
} on PlatformException catch (e) {
_log.info('Calling unlock failed with exception: $e');
_log.config('Calling unlock failed with exception: $e');
return Pair(false, false);
}
}
@ -69,7 +69,7 @@ class _AndroidOathStateNotifier extends OathStateNotifier {
}
return true;
} on PlatformException catch (e) {
_log.info('Calling set password failed with exception: $e');
_log.config('Calling set password failed with exception: $e');
return false;
}
}
@ -80,7 +80,7 @@ class _AndroidOathStateNotifier extends OathStateNotifier {
await _api.unsetPassword(current);
return true;
} on PlatformException catch (e) {
_log.info('Calling unset password failed with exception: $e');
_log.config('Calling unset password failed with exception: $e');
return false;
}
}
@ -90,7 +90,7 @@ class _AndroidOathStateNotifier extends OathStateNotifier {
try {
await _api.forgetPassword();
} on PlatformException catch (e) {
_log.info('Calling forgetPassword failed with exception: $e');
_log.config('Calling forgetPassword failed with exception: $e');
}
}
}
@ -165,14 +165,10 @@ class _AndroidCredentialListNotifier extends OathCredentialListNotifier {
@override
Future<OathCredential> addAccount(Uri credentialUri,
{bool requireTouch = false, bool update = true}) async {
_log.info('About to add new cred: $credentialUri, $requireTouch, $update');
String resultString =
await _api.addAccount(credentialUri.toString(), requireTouch);
var result = jsonDecode(resultString);
_log.info('Received credential: $resultString');
final credential = OathCredential.fromJson(result);
if (update && mounted) {
@ -216,7 +212,7 @@ class _AndroidCredentialListNotifier extends OathCredentialListNotifier {
return renamedCredential;
} on PlatformException catch (e) {
_log.info('Failed to execute renameOathCredential: ${e.message}');
_log.config('Failed to execute renameOathCredential: ${e.message}');
}
return credential;
@ -224,8 +220,6 @@ class _AndroidCredentialListNotifier extends OathCredentialListNotifier {
@override
Future<void> deleteAccount(OathCredential credential) async {
_log.info('About to delete cred: $credential');
try {
await _api.deleteAccount(credential.id);
@ -233,19 +227,18 @@ class _AndroidCredentialListNotifier extends OathCredentialListNotifier {
state = state!.toList()..removeWhere((e) => e.credential == credential);
}
} catch (e) {
_log.info('Call to delete credential failed: $e');
_log.config('Call to delete credential failed: $e');
}
}
refresh() async {
if (_locked) return;
_log.info('refreshing credentials...');
_log.config('refreshing credentials...');
final pairs = [];
try {
var resultString = await _api.refreshCodes();
_log.info('Entries', resultString);
var result = jsonDecode(resultString);
for (var e in result['entries']) {
@ -273,7 +266,7 @@ class _AndroidCredentialListNotifier extends OathCredentialListNotifier {
_scheduleRefresh();
}
} catch (e) {
_log.info('Failure refreshing codes: $e');
_log.config('Failure refreshing codes: $e');
}
}