From 5cc844ecbd5d783c04ce2e2aa7608baf7d9d54e0 Mon Sep 17 00:00:00 2001
From: Feross Aboukhadijeh <feross@feross.org>
Date: Wed, 9 Mar 2016 02:15:53 -0800
Subject: [PATCH 1/2] filter file formats that <video> tag definitely can't
 play

---
 renderer/index.js | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/renderer/index.js b/renderer/index.js
index 6458274c..fd98eabc 100644
--- a/renderer/index.js
+++ b/renderer/index.js
@@ -431,13 +431,22 @@ function startServer (infoHash, cb) {
 }
 
 function startServerFromReadyTorrent (torrent, cb) {
+  // filter out file formats that the <video> tag definitely can't play
+  var files = torrent.files.filter(function (file) {
+    var extname = path.extname(file.name)
+    return ['.mp4', '.m4v', '.webm', '.mov', '.mkv'].indexOf(extname) !== -1
+  })
+
+  if (files.length === 0) return cb(new Error('cannot play any files in torrent'))
+
   // use largest file
-  state.torrentPlaying = torrent.files.reduce(function (a, b) {
+  state.torrentPlaying = files.reduce(function (a, b) {
     return a.length > b.length ? a : b
   })
-  var index = torrent.files.indexOf(state.torrentPlaying)
 
+  var index = torrent.files.indexOf(state.torrentPlaying)
   var server = torrent.createServer()
+
   server.listen(0, function () {
     var port = server.address().port
     var urlSuffix = ':' + port + '/' + index
@@ -467,7 +476,9 @@ function openPlayer (torrentSummary) {
     update()
   }, 10000) /* give it a few seconds */
 
-  startServer(torrentSummary.infoHash, function () {
+  startServer(torrentSummary.infoHash, function (err) {
+    if (err) return onError(err)
+
     // if we timed out (user clicked play a long time ago), don't autoplay
     clearTimeout(timeout)
     var timedOut = torrentSummary.playStatus === 'timeout'

From f99bbab91c4864353738c62b053d7189fcad2e4b Mon Sep 17 00:00:00 2001
From: Feross Aboukhadijeh <feross@feross.org>
Date: Wed, 9 Mar 2016 02:16:20 -0800
Subject: [PATCH 2/2] airplay/chromecast: remove unneeded startServer() calls

---
 renderer/index.js | 24 ++++++++++--------------
 1 file changed, 10 insertions(+), 14 deletions(-)

diff --git a/renderer/index.js b/renderer/index.js
index fd98eabc..fc44cd1b 100644
--- a/renderer/index.js
+++ b/renderer/index.js
@@ -530,25 +530,21 @@ function deleteTorrent (torrentSummary) {
 
 function openChromecast (infoHash) {
   var torrentSummary = getTorrentSummary(infoHash)
-  startServer(infoHash, function () {
-    state.devices.chromecast.play(state.server.networkURL, {
-      title: config.APP_NAME + ' — ' + torrentSummary.name
-    })
-    state.devices.chromecast.on('error', function (err) {
-      err.message = 'Chromecast: ' + err.message
-      onError(err)
-    })
-    update()
+  state.devices.chromecast.play(state.server.networkURL, {
+    title: config.APP_NAME + ' — ' + torrentSummary.name
   })
+  state.devices.chromecast.on('error', function (err) {
+    err.message = 'Chromecast: ' + err.message
+    onError(err)
+  })
+  update()
 }
 
 function openAirplay (infoHash) {
-  startServer(infoHash, function () {
-    state.devices.airplay.play(state.server.networkURL, 0, function () {
-      // TODO: handle airplay errors
-    })
-    update()
+  state.devices.airplay.play(state.server.networkURL, 0, function () {
+    // TODO: handle airplay errors
   })
+  update()
 }
 
 // Set window dimensions to match video dimensions or fill the screen