react-native-compressor 1.19.0 → 1.19.2

package/android/src/main/java/com/reactnativecompressor/Video/VideoCompressor/compressor/Compressor.kt

@@ -788,32 +788,21 @@ object Compressor {
         inputFormat: MediaFormat,
         outputSurface: OutputSurface,
     ): MediaCodec {
-        val originalMime = inputFormat.getString(MediaFormat.KEY_MIME)!!
-
-        // Dolby Vision (video/dolby-vision) has no standalone decoder on most Android
-        // devices and throws NAME_NOT_FOUND. Profiles 8.1/8.4 carry an HEVC base layer
-        // that the standard HEVC decoder can render, so we remap them to HEVC. Profile 5
-        // has no compatible base layer; it is rejected by isUnsupportedDolbyVision() in
-        // start() before any codec/surface is created, so it never reaches here.
-        val resolvedMime = if (originalMime.equals("video/dolby-vision", ignoreCase = true)) {
-            inputFormat.setString(MediaFormat.KEY_MIME, MediaFormat.MIMETYPE_VIDEO_HEVC)
-            MediaFormat.MIMETYPE_VIDEO_HEVC
-        } else {
-            originalMime
+        // Some inputs (e.g. iPhone .MOV files) report a "video/dolby-vision" MIME
+        // type that many devices cannot decode. Remap to a decodable base-layer
+        // codec, or fail with a clear error, before creating the decoder (#398).
+        ensureDecodableVideoFormat(inputFormat)
+
+        // Clear Dolby Vision specific profile and level to prevent configuration failures
+        // when the MIME type has been remapped to AVC/HEVC.
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
+            inputFormat.removeKey(MediaFormat.KEY_PROFILE)
+            inputFormat.removeKey(MediaFormat.KEY_LEVEL)
         }
 
-        val decoder = MediaCodec.createDecoderByType(resolvedMime)
-
-        try {
-            decoder.configure(inputFormat, outputSurface.getSurface(), null, 0)
-        } catch (e: Exception) {
-            // A codec that throws from configure() is unusable; release it so a
-            // configure failure here doesn't leak the decoder handle.
-            runCatching { decoder.release() }
-            throw e
-        }
+        val decoder = MediaCodec.createDecoderByType(inputFormat.getString(MediaFormat.KEY_MIME)!!)
 
-        Log.i("Compressor", "decoder selected: ${decoder.name} mime=$resolvedMime")
+        decoder.configure(inputFormat, outputSurface.getSurface(), null, 0)
 
         return decoder
     }

package/ios/Video/NextLevelSessionExporter.swift

@@ -13,7 +13,9 @@ public enum NextLevelSessionExporterError: Error, CustomStringConvertible {
     case readingFailure
     case writingFailure
     case cancelled
-    
+    case unsupportedVideoOutputConfiguration
+    case missingVideoTrackInOutput
+
     public var description: String {
         get {
             switch self {
@@ -25,6 +27,10 @@ public enum NextLevelSessionExporterError: Error, CustomStringConvertible {
                 return "Writing failure"
             case .cancelled:
                 return "Cancelled"
+            case .unsupportedVideoOutputConfiguration:
+                return "The writer rejected the video output configuration"
+            case .missingVideoTrackInOutput:
+                return "Export finished without a video track in the output"
             }
         }
     }
@@ -249,7 +255,16 @@ extension NextLevelSessionExporter {
             }
         }
         
-        self.setupVideoOutput(withAsset: asset)
+        // Fail loudly when the writer rejects the video output configuration.
+        // Continuing here used to export only the audio track and still resolve
+        // as a success (issue #400).
+        guard self.setupVideoOutput(withAsset: asset) else {
+            DispatchQueue.main.async {
+                self._completionHandler?(.failure(NextLevelSessionExporterError.unsupportedVideoOutputConfiguration))
+                self._completionHandler = nil
+            }
+            return
+        }
         if !self.stripAudio {
             self.setupAudioOutput(withAsset: asset)
             self.setupAudioInput()
@@ -316,11 +331,14 @@ extension NextLevelSessionExporter {
 
 extension NextLevelSessionExporter {
     
-    private func setupVideoOutput(withAsset asset: AVAsset) {
+    /// Returns `false` when the asset has video tracks but a video writer input
+    /// could not be created — exporting would silently produce an audio-only
+    /// file (see numandev1/react-native-compressor#400).
+    private func setupVideoOutput(withAsset asset: AVAsset) -> Bool {
         let videoTracks = asset.tracks(withMediaType: AVMediaType.video)
-        
+
         guard videoTracks.count > 0 else {
-            return
+            return true
         }
         
         self._videoOutput = AVAssetReaderVideoCompositionOutput(videoTracks: videoTracks, videoSettings: self.videoInputConfiguration)
@@ -344,8 +362,8 @@ extension NextLevelSessionExporter {
             self._videoInput = AVAssetWriterInput(mediaType: AVMediaType.video, outputSettings: self.videoOutputConfiguration)
             self._videoInput?.expectsMediaDataInRealTime = self.expectsMediaDataInRealTime
         } else {
-            print("Unsupported output configuration")
-            return
+            print("NextLevelSessionExporter, unsupported video output configuration, failing export instead of writing an audio-only file")
+            return false
         }
         
         if let writer = self._writer,
@@ -367,8 +385,9 @@ extension NextLevelSessionExporter {
             
             self._pixelBufferAdaptor = AVAssetWriterInputPixelBufferAdaptor(assetWriterInput: videoInput, sourcePixelBufferAttributes: pixelBufferAttrib)
         }
+        return true
     }
-    
+
     private func setupAudioOutput(withAsset asset: AVAsset) {
         let audioTracks = asset.tracks(withMediaType: AVMediaType.audio)
         var audioTracksToUse: [AVAssetTrack] = []
@@ -632,6 +651,22 @@ extension NextLevelSessionExporter {
             break
         }
 
+        // Guard against silently returning an audio-only file: when the source
+        // has a video track and a video output was configured, the exported file
+        // must contain a video track as well. A configuration the encoder rejects
+        // at write time can still end with `.completed` while the video track is
+        // dropped (issue #400) — surface that as an error instead of a success.
+        if self.videoOutputConfiguration != nil,
+           let asset = self.asset,
+           let outputURL = self.outputURL,
+           asset.tracks(withMediaType: AVMediaType.video).count > 0,
+           AVAsset(url: outputURL).tracks(withMediaType: AVMediaType.video).count == 0 {
+            try? FileManager.default.removeItem(at: outputURL)
+            self._completionHandler?(.failure(NextLevelSessionExporterError.missingVideoTrackInOutput))
+            self._completionHandler = nil
+            return
+        }
+
         self._completionHandler?(.success(self.status))
         self._completionHandler = nil
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-compressor",
-  "version": "1.19.0",
+  "version": "1.19.2",
   "description": "Compress Image, Video, and Audio same like Whatsapp & Auto/Manual Compression | Background Upload | Download File | Create Video Thumbnail",
   "main": "lib/commonjs/index",
   "module": "lib/module/index",