fixed TTL_MINUTES handling

privacyidea · Dec 9, 2024 · 80e40aa · 80e40aa
1 parent be601cc
commit 80e40aa
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 12 deletions.
diff --git a/lib/model/token_container.dart b/lib/model/token_container.dart
@@ -70,7 +70,7 @@ class TokenContainer with _$TokenContainer {
 
 // Container registration:
   static const String ISSUER = 'issuer';
-  static const String TTL = 'ttl';
+  static const String TTL_MINUTES = 'ttl';
   static const String NONCE = 'nonce';
   static const String TIMESTAMP = 'time';
   static const String FINALIZATION_URL = 'url';
@@ -109,7 +109,7 @@ class TokenContainer with _$TokenContainer {
       map: uriMap,
       validators: {
         ISSUER: const ObjectValidator<String>(),
-        TTL: ObjectValidator<Duration>(transformer: (value) => Duration(minutes: value)),
+        TTL_MINUTES: durationValidator.withDefault(const Duration(minutes: 10)),
         NONCE: const ObjectValidator<String>(),
         TIMESTAMP: ObjectValidator<DateTime>(transformer: (v) => DateTime.parse(v)),
         FINALIZATION_URL: stringToUrivalidator,
@@ -124,7 +124,7 @@ class TokenContainer with _$TokenContainer {
     );
     return TokenContainer.unfinalized(
       issuer: uriMap[ISSUER],
-      ttl: uriMap[TTL],
+      ttl: uriMap[TTL_MINUTES],
       nonce: uriMap[NONCE],
       timestamp: uriMap[TIMESTAMP],
       serverUrl: uriMap[FINALIZATION_URL],

diff --git a/lib/model/tokens/push_token.dart b/lib/model/tokens/push_token.dart
@@ -219,10 +219,7 @@ class PushToken extends Token {
         Token.ISSUER: const ObjectValidatorNullable<String>(defaultValue: ''),
         Token.SERIAL: const ObjectValidator<String>(),
         SSL_VERIFY: boolValidator.withDefault(true),
-        TTL_MINUTES: ObjectValidator<Duration>(
-          transformer: (v) => Duration(minutes: v is int ? v : int.parse(v)),
-          defaultValue: const Duration(minutes: 10),
-        ),
+        TTL_MINUTES: durationValidator.withDefault(const Duration(minutes: 3)),
         ENROLLMENT_CREDENTIAL: const ObjectValidatorNullable<String>(),
         ROLLOUT_URL: stringToUrivalidator,
         Token.IMAGE: stringToUriValidatorNullable,
@@ -287,9 +284,7 @@ class PushToken extends Token {
         Token.ISSUER: const ObjectValidatorNullable<String>(),
         Token.SERIAL: const ObjectValidatorNullable<String>(),
         SSL_VERIFY: boolValidatorNullable,
-        TTL_MINUTES: ObjectValidatorNullable<Duration>(
-          transformer: (v) => Duration(minutes: int.parse(v)),
-        ),
+        TTL_MINUTES: durationValidator.withDefault(const Duration(minutes: 3)),
         ENROLLMENT_CREDENTIAL: const ObjectValidatorNullable<String>(),
         ROLLOUT_URL: stringToUriValidatorNullable,
         Token.IMAGE: stringToUriValidatorNullable,

diff --git a/lib/utils/object_validator.dart b/lib/utils/object_validator.dart
@@ -24,6 +24,12 @@ import '../model/enums/encodings.dart';
 import '../model/exception_errors/localized_argument_error.dart';
 import 'logger.dart';
 
+final durationValidatorNullable = durationValidator.nullable();
+final durationValidator = ObjectValidator<Duration>(
+  transformer: (v) => Duration(seconds: v is int ? v : int.parse(v)),
+  allowedValues: (v) => v.inSeconds > 0,
+);
+
 final otpAutjPeriodSecondsValidatorNullable = otpAuthPeriodSecondsValidator.nullable();
 final otpAuthPeriodSecondsValidator = ObjectValidator<int>(
   transformer: (v) {

diff --git a/test/unit_test/model/processor_result_test.dart b/test/unit_test/model/processor_result_test.dart
@@ -20,12 +20,12 @@ void _testProcessorResult() {
     });
     group('factories', () {
       test('success', () {
-        const ProcessorResult<String> result = ProcessorResult.success('data');
+        final ProcessorResult<String> result = ProcessorResult.success('data');
         expect(result, isA<ProcessorResultSuccess>());
         expect((result as ProcessorResultSuccess).resultData, 'data');
       });
       test('error', () {
-        const ProcessorResult<String> result = ProcessorResult.failed('error');
+        final ProcessorResult<String> result = ProcessorResult.failed('error');
         expect(result, isA<ProcessorResultFailed>());
         expect((result as ProcessorResultFailed).message, 'error');
       });