Merge pull request #8 from membraneframework/bugfixes-0.1.0

Fix bugs and bump version

Author: FelonEkonom

README.md

@@ -15,7 +15,7 @@ The package can be installed by adding `ex_hls` to your list of dependencies in
 ```elixir
 def deps do
   [
-    {:ex_hls, "~> 0.1.0"}
+    {:ex_hls, "~> 0.1.1"}
   ]
 end
 ```

lib/ex_hls/client.ex

@@ -133,6 +133,8 @@ defmodule ExHLS.Client do
     end
   end
 
+  defp resolve_hls_mode(%{hls_mode: mode} = client) when mode in [:live, :vod], do: client
+
   defp ensure_hls_mode_resolved!(%__MODULE__{hls_mode: nil} = client) do
     # the error message is about choosing a variant, while the function name is
     # about resolving the HLS mode, but it is done this way because the HLS mode

lib/ex_hls/client/live/reader.ex

@@ -102,10 +102,14 @@ defmodule ExHLS.Client.Live.Reader do
   end
 
   defp should_start_playing?(state) do
-    %{media_sequence: media_sequence, target_duration: target_duration} =
-      state.media_playlist.info
-
-    (is_number(media_sequence) and media_sequence >= 1) or
+    %ExM3U8.MediaPlaylist.Info{
+      media_sequence: media_sequence,
+      target_duration: target_duration,
+      end_list?: end_list?
+    } = state.media_playlist.info
+
+    end_list? or
+      (is_number(media_sequence) and media_sequence >= 1) or
       segments_duration_sum(state) >= 2 * target_duration
   end
 

lib/ex_hls/client/vod.ex

@@ -237,6 +237,7 @@ defmodule ExHLS.Client.VOD do
 
   defp media_type_with_lower_ts(client, mode) when mode in [:read, :returned] do
     client.media_types
+    |> Enum.reject(&client.stream_ended_by_media_type[&1])
     |> Enum.max_by(fn media_type ->
       case client.last_timestamps[media_type][mode] do
         nil -> :infinity

mix.exs

@@ -1,7 +1,7 @@
 defmodule ExHLS.Mixfile do
   use Mix.Project
 
-  @version "0.1.0"
+  @version "0.1.1"
   @github_url "https://github.com/membraneframework/ex_hls"
 
   def project do