refactor(nextcloud)!: use http_parser for http date parsing

Signed-off-by: Nikolas Rimikis <leptopoda@users.noreply.github.com>
nextcloud · Mar 10, 2024 · 5223df7 · 5223df7
1 parent d538b71
commit 5223df7
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 15 deletions.
diff --git a/.github/renovate.json b/.github/renovate.json
@@ -12,11 +12,6 @@
     "automerge": true
   },
   "packageRules": [
-    {
-      "matchFileNames": ["packages/nextcloud/**"],
-      "matchPackageNames": ["intl"],
-      "rangeStrategy": "widen"
-    },
     {
       "matchManagers": ["pub"],
       "matchDatasources": ["dart-version", "flutter-version"],

diff --git a/packages/nextcloud/lib/src/webdav/file.dart b/packages/nextcloud/lib/src/webdav/file.dart
@@ -1,3 +1,4 @@
+import 'package:http_parser/http_parser.dart';
 import 'package:nextcloud/src/webdav/client.dart';
 import 'package:nextcloud/src/webdav/path_uri.dart';
 import 'package:nextcloud/src/webdav/props.dart';
@@ -62,7 +63,7 @@ class WebDavFile {
   /// Last modified date of the file
   late final DateTime? lastModified = () {
     if (props.davgetlastmodified != null) {
-      return webdavDateFormat.parseUtc(props.davgetlastmodified!);
+      return parseHttpDate(props.davgetlastmodified!);
     }
     return null;
   }();

diff --git a/packages/nextcloud/lib/src/webdav/webdav.dart b/packages/nextcloud/lib/src/webdav/webdav.dart
@@ -1,17 +1,13 @@
 // ignore_for_file: public_member_api_docs
 // coverage:ignore-file
 
-import 'package:intl/intl.dart';
 import 'package:meta/meta.dart';
 import 'package:nextcloud/src/webdav/props.dart';
 import 'package:xml/xml.dart';
 import 'package:xml_annotation/xml_annotation.dart' as annotation;
 
 part 'webdav.g.dart';
 
-/// Format used in WebDAV
-final webdavDateFormat = DateFormat('E, d MMM yyyy HH:mm:ss', 'en_US');
-
 const namespaceDav = 'DAV:';
 const namespaceOwncloud = 'http://owncloud.org/ns';
 const namespaceNextcloud = 'http://nextcloud.org/ns';

diff --git a/packages/nextcloud/pubspec.yaml b/packages/nextcloud/pubspec.yaml
@@ -19,7 +19,7 @@ dependencies:
   crypton: ^2.0.0
   dynamite_runtime: ^0.2.0
   http: ^1.2.0
-  intl: '>=0.17.0 <0.20.0'
+  http_parser: ^4.0.0
   json_annotation: ^4.8.1
   meta: ^1.0.0
   universal_io: ^2.0.0

diff --git a/packages/nextcloud/test/webdav_test.dart b/packages/nextcloud/test/webdav_test.dart
@@ -2,6 +2,7 @@ import 'dart:convert';
 import 'dart:math';
 import 'dart:typed_data';
 
+import 'package:http_parser/http_parser.dart';
 import 'package:mocktail/mocktail.dart';
 import 'package:nextcloud/nextcloud.dart';
 import 'package:nextcloud_test/nextcloud_test.dart';
@@ -174,7 +175,7 @@ void main() {
             responses.singleWhere((response) => response.href!.endsWith('/Nextcloud.png')).propstats.first.prop;
         expect(props.nchaspreview, isTrue);
         expect(props.davgetcontenttype, 'image/png');
-        expect(webdavDateFormat.parseUtc(props.davgetlastmodified!).isBefore(DateTime.now()), isTrue);
+        expect(parseHttpDate(props.davgetlastmodified!).isBefore(DateTime.now()), isTrue);
         expect(props.ocsize, 50598);
       });
 
@@ -241,7 +242,7 @@ void main() {
         expect(response.name, 'Nextcloud.png');
         expect(response.isDirectory, isFalse);
 
-        expect(webdavDateFormat.parseUtc(response.props.davgetlastmodified!).isBefore(DateTime.now()), isTrue);
+        expect(parseHttpDate(response.props.davgetlastmodified!).isBefore(DateTime.now()), isTrue);
         expect(response.props.davgetetag, isNotEmpty);
         expect(response.props.davgetcontenttype, 'image/png');
         expect(response.props.davgetcontentlength, 50598);
@@ -301,7 +302,7 @@ void main() {
 
         expect(response.props.davgetcontenttype, isNull);
         expect(
-          webdavDateFormat.parseUtc(response.props.davgetlastmodified!).millisecondsSinceEpoch,
+          parseHttpDate(response.props.davgetlastmodified!).millisecondsSinceEpoch,
           closeTo(DateTime.now().millisecondsSinceEpoch, 10E3),
         );
         expect(response.props.davresourcetype!.collection, isNotNull);
@@ -370,7 +371,7 @@ void main() {
             .first
             .prop;
         expect(props.ocfavorite, 1);
-        expect(webdavDateFormat.parseUtc(props.davgetlastmodified!), lastModifiedDate);
+        expect(parseHttpDate(props.davgetlastmodified!), lastModifiedDate);
         expect(props.nccreationtime! * 1000, createdDate.millisecondsSinceEpoch);
         expect(props.ncuploadtime! * 1000, closeTo(uploadTime.millisecondsSinceEpoch, 10E3));
       });