🐛 Fix decode box error

Author: CaiJingLong

CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 1.4.2
+
+- Fix: decode box error
+
 ## 1.4.1
 
 - Fix: `BaseBmffContext` constructor

example/bin/print_async_bmff_boxes_info.dart

@@ -2,13 +2,28 @@ import 'package:bmff/bmff.dart';
 import 'package:http/http.dart' as http;
 
 Future<void> main(List<String> args) async {
-  // final uri = Uri.parse('https://www.w3school.com.cn/i/movie.mp4');
-  final uri = Uri.parse(
-      'http://vfx.mtime.cn/Video/2019/03/18/mp4/190318231014076505.mp4');
+  final urls = [
+    'http://zuoye.free.fr/files/compare_still_1.heic.jpg',
+    // 'http://vfx.mtime.cn/Video/2019/03/18/mp4/190318231014076505.mp4',
+    // 'https://www.w3school.com.cn/i/movie.mp4',
+    // 'https://cdn.jsdelivr.net/gh/ExampleAssets/ExampleAsset@master/preview_0.heic',
+  ];
+
+  for (final url in urls) {
+    await showHttpBoxInfo(url);
+  }
+}
+
+Future<void> showHttpBoxInfo(String url) async {
+  print('------------------ $url ------------------');
+
+  final uri = Uri.parse(url);
   final bmff = await Bmff.asyncContext(AsyncBmffContextHttp(uri));
   for (final box in bmff.childBoxes) {
     showBoxInfo(0, box);
   }
+
+  print('\n');
 }
 
 void showBoxInfo(int level, AsyncBmffBox box) {
@@ -23,30 +38,53 @@ void showBoxInfo(int level, AsyncBmffBox box) {
 // part download http context
 class AsyncBmffContextHttp extends AsyncBmffContext {
   final Uri uri;
+  final int maxCacheLength;
+
+  AsyncBmffContextHttp(
+    this.uri, {
+    this.maxCacheLength = 1024 * 1024 * 1,
+  });
 
-  const AsyncBmffContextHttp(this.uri);
+  List<int>? _bytes;
 
   @override
   Future<List<int>> getRangeData(int start, int end) async {
+    if (_bytes != null) {
+      return _bytes!.sublist(start, end);
+    }
+
     final response = await http.get(uri, headers: {
       'Range': 'bytes=$start-${end - 1}',
     });
     if (response.statusCode != 206) {
       throw Exception(
-          'Current http status code is ${response.statusCode}, not 206, not support range download');
+          'Current http status code is ${response.statusCode}, not 206. '
+          'The server not support range download');
     }
     final bytes = response.bodyBytes;
     return bytes;
   }
 
   @override
   Future<int> lengthAsync() async {
-    final response = await http.head(uri);
-    final contentLength = response.headers['content-length'];
-    if (contentLength != null) {
-      return int.parse(contentLength);
+    final response = await http.head(uri, headers: {
+      'User-Agent': 'curl/7.79.1',
+      'Accept': '*/*',
+      'Host': uri.host,
+    });
+    final contentLengthList = response.headers.entries
+        .where((element) => element.key.toLowerCase() == 'content-length');
+    if (contentLengthList.isNotEmpty) {
+      final contentLength = int.parse(contentLengthList.first.value);
+      if (contentLength < maxCacheLength) {
+        final response = await http.get(uri);
+        _bytes = response.bodyBytes;
+      }
+      return contentLength;
     } else {
-      throw Exception('content-length not found');
+      final response = await http.get(uri);
+      _bytes = response.bodyBytes;
+      return response.bodyBytes.length;
     }
   }
 }

lib/src/bmff/async_bmff.dart

@@ -1,7 +1,7 @@
 import 'dart:collection';
 
 import 'package:bmff/bmff.dart';
-import 'package:bmff/src/box/box_factory.dart';
+import 'package:bmff/src/factory/box_factory.dart';
 
 /// {@template bmff.async_bmff}
 ///

lib/src/bmff/bmff.dart

@@ -1,5 +1,6 @@
 import 'package:bmff/bmff.dart';
-import 'package:bmff/src/box/box_factory.dart';
+import 'package:bmff/src/factory/box_factory.dart';
+import 'package:bmff/src/box/full_box_type.dart';
 import 'package:bmff/src/box/impl/bmff_impl.dart';
 
 import 'stub.dart'
@@ -61,15 +62,24 @@ abstract class Bmff extends BoxContainer {
   /// {@macro bmff.bmff_for_web}
   ///
   /// {@macro bmff.bmff_example}
-  factory Bmff.file(String path) {
+  factory Bmff.file(
+    String path, {
+    List<String> fullBoxtypes = fullBoxType,
+  }) {
     return createBmffFromFile(path);
   }
 
   /// Create [Bmff] from [bytes].
   ///
   /// {@macro bmff.bmff_example}
-  factory Bmff.memory(List<int> bytes) {
-    return ContextBmffImpl(BmffMemoryContext(bytes));
+  factory Bmff.memory(
+    List<int> bytes, {
+    List<String> fullBoxTypes = fullBoxType,
+  }) {
+    return ContextBmffImpl(BmffMemoryContext(
+      bytes,
+      fullBoxTypes: fullBoxTypes,
+    ));
   }
 
   /// Type box

lib/src/bmff/io_impl.dart

@@ -4,8 +4,11 @@ import 'package:bmff/bmff.dart';
 import 'package:bmff/src/box/full_box_type.dart';
 
 /// {@macro bmff.createBmffFromFile}
-Bmff createBmffFromFile(String path) {
-  return BmffIoContext(File(path)).bmff;
+Bmff createBmffFromFile(
+  String path, {
+  List<String> fullBoxTypes = fullBoxType,
+}) {
+  return BmffIoContext(File(path), fullBoxTypes: fullBoxTypes).bmff;
 }
 
 /// {@template bmff.BmffIoContext}

lib/src/bmff/stub.dart

@@ -1,4 +1,5 @@
 import 'package:bmff/bmff.dart';
+import 'package:bmff/src/box/full_box_type.dart';
 
 ///
 /// {@template bmff.createBmffFromFile}
@@ -8,6 +9,7 @@ import 'package:bmff/bmff.dart';
 /// For code compatibility, the web can call this method, but an error will be reported.
 ///
 /// {@endtemplate}
-Bmff createBmffFromFile(String path) {
+Bmff createBmffFromFile(String path,
+    {List<String> fullBoxTypes = fullBoxType}) {
   throw UnimplementedError();
 }

lib/src/bmff/web_impl.dart

@@ -1,6 +1,8 @@
 import 'package:bmff/bmff.dart';
+import 'package:bmff/src/box/full_box_type.dart';
 
 /// {@macro bmff.createBmffFromFile}
-Bmff createBmffFromFile(String path) {
+Bmff createBmffFromFile(String path,
+    {List<String> fullBoxTypes = fullBoxType}) {
   throw UnimplementedError();
 }

lib/src/box/box.dart

@@ -4,7 +4,7 @@ import 'dart:typed_data';
 import 'package:bmff/bmff.dart';
 import 'package:bmff/src/box_base.dart';
 
-import 'box_factory.dart';
+import '../factory/box_factory.dart';
 
 /// {@template bmff.bmff_box}
 ///
@@ -137,6 +137,10 @@ class AsyncBmffBox extends BmffBoxBase {
   Future<void> init() async {
     final children = await AsyncBoxFactory().decodeChildBoxes(this);
     childBoxes.addAll(children);
+
+    for (final child in children) {
+      await child.init();
+    }
   }
 
   Future<void> updateForceFullBox(bool? forceFullBox) async {
@@ -147,9 +151,6 @@ class AsyncBmffBox extends BmffBoxBase {
   Future<void> _updateChildBoxes() async {
     childBoxes.clear();
     await init();
-    for (final child in childBoxes) {
-      await child._updateChildBoxes();
-    }
   }
 
   /// Get data of the box.

lib/src/box/full_box.dart

@@ -0,0 +1 @@
+

lib/src/box/impl/bmff_impl.dart

@@ -1,5 +1,5 @@
 import 'package:bmff/bmff.dart';
-import 'package:bmff/src/box/box_factory.dart';
+import 'package:bmff/src/factory/box_factory.dart';
 
 class ContextBmffImpl extends Bmff {
   /// {@macro bmff.bmff_example}

lib/src/context.dart

@@ -102,10 +102,23 @@ abstract class AsyncBmffContext extends BaseBmffContext {
   Future<List<int>> getRangeData(int start, int end);
 
   /// Create a [AsyncBmffContext] from [bytes].
-  factory AsyncBmffContext.memory(List<int> bytes) {
-    return MemoryAsyncBmffContext(
-      () async => bytes.length,
-      (start, end) async => bytes.sublist(start, end),
+  factory AsyncBmffContext.bytes(
+    List<int> bytes, {
+    List<String> fullBoxTypes = fullBoxType,
+  }) {
+    return MemoryAsyncBmffContext(bytes, fullBoxTypes: fullBoxTypes);
+  }
+
+  /// Create a [AsyncBmffContext] from [lengthAsyncGetter] and [rangeDataGetter].
+  factory AsyncBmffContext.common(
+    LengthGetter lengthAsyncGetter,
+    RangeDataGetter rangeDataGetter, {
+    List<String> fullBoxTypes = fullBoxType,
+  }) {
+    return _CommonAsyncBmffContext(
+      lengthAsyncGetter,
+      rangeDataGetter,
+      fullBoxTypes: fullBoxTypes,
     );
   }
 }
@@ -115,9 +128,9 @@ abstract class AsyncBmffContext extends BaseBmffContext {
 /// The context of a BMFF file in memory.
 ///
 /// {@endtemplate}
-class MemoryAsyncBmffContext extends AsyncBmffContext {
+class _CommonAsyncBmffContext extends AsyncBmffContext {
   /// {@macro bmff.MemoryAsyncBmffContext}
-  const MemoryAsyncBmffContext(
+  const _CommonAsyncBmffContext(
     this.lengthAsyncGetter,
     this.rangeDataGetter, {
     List<String> fullBoxTypes = fullBoxType,
@@ -139,3 +152,32 @@ class MemoryAsyncBmffContext extends AsyncBmffContext {
     return rangeDataGetter.call(start, end);
   }
 }
+
+/// {@template bmff.MemoryAsyncBmffContext}
+/// The context of a BMFF file in memory.
+///
+/// This is a simple implementation of [AsyncBmffContext].
+///
+/// You can use [AsyncBmffContext.bytes] to use this.
+///
+/// {@endtemplate}
+class MemoryAsyncBmffContext extends AsyncBmffContext {
+  /// {@macro bmff.MemoryAsyncBmffContext}
+  const MemoryAsyncBmffContext(
+    this.bytes, {
+    List<String> fullBoxTypes = fullBoxType,
+  }) : super(fullBoxTypes: fullBoxTypes);
+
+  /// The bytes of the context.
+  final List<int> bytes;
+
+  @override
+  Future<int> lengthAsync() {
+    return Future.value(bytes.length);
+  }
+
+  @override
+  Future<List<int>> getRangeData(int start, int end) {
+    return Future.value(bytes.sublist(start, end));
+  }
+}