Merge branch 'main' of https://github.com/elixir-image/image

Author: kipcole9

CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## Image 0.62.1
+
+This is the changelog for Image version 0.62.1 released on October 23rd, 2025.  For older changelogs please consult the release tag on [GitHub](https://github.com/elixir-image/image/tags)
+
+### Bug Fixes
+
+* Fix decoding EXIF component configuration if the value is invalid. Fixes #194. Thanks to @ethangunderson for the report.
+
+* Fix `Image.minimize_metadata/1` when the image does not have an author or copyright field.
+
 ## Image 0.62.0
 
 This is the changelog for Image version 0.62.0 released on August 8th, 2025.  For older changelogs please consult the release tag on [GitHub](https://github.com/elixir-image/image/tags)

lib/image.ex

@@ -1084,18 +1084,18 @@ defmodule Image do
   * `options` is a keyword list of options. See `Image.open/2`
     for the list of applicable options.
 
-  ### Notes
-
-  This function is typically *not* the best way to open
-  an image. It requires that the entire image
-  is already loaded into memory which, for most use cases,
-  doest not scale well and consumes far more memory than
-  necessary.
-
-  Since `libvips` is a streaming on-demand architecture,
-  it is most likely that a simple `Image.open/2` call will
-  deliver better resource utilitsation and equal or better
-  performance.
+  > #### Note {: .info}
+  >
+  > This function is typically *not* the best way to open
+  > an image. It requires that the entire image
+  > is already loaded into memory which, for most use cases,
+  > doest not scale well and consumes far more memory than
+  > necessary.
+  >
+  > Since `libvips` is a streaming on-demand architecture,
+  > it is most likely that a simple `Image.open/2` call will
+  > deliver better resource utilitsation and equal or better
+  > performance.
 
   ### Returns
 
@@ -6337,16 +6337,19 @@ defmodule Image do
   with intellectual property concerns in mind
   this function will keep the artist and
   copyright fields if they exist in the original
-  image.
+  image EXIF. Creator/Author/Artist and Copyright in
+  other fields like IPTC and XMP are not considered
+  in the current implementation.
 
   On a 1000x500px image exported from Adobe Lightroom
   with metadata intact, removing the metadata
   results in am approximately 50% saving in file
   size due to the removal of most EXIF and all
   IPTC and XMP metadata.
 
-  Note that the minimized metadata is only materialized when
-  the minimized image is saved to a file.
+  > #### Note {: .info}
+  >
+  > the minimized metadata is only materialized when the minimized image is saved.
 
   ### Arguments
 
@@ -6363,15 +6366,28 @@ defmodule Image do
 
   @spec minimize_metadata(image :: Vimage.t()) :: {:ok, Vimage.t()} | {:error, error_message()}
   def minimize_metadata(%Vimage{} = image) do
-    with {:ok, exif} <- exif(image),
-         {:ok, image} <- remove_metadata(image) do
-      Vimage.mutate(image, fn mut_img ->
-        :ok = Exif.put_metadata(mut_img, :copyright, exif.copyright)
-        :ok = Exif.put_metadata(mut_img, :artist, exif.artist)
-      end)
+    case exif(image) do
+      {:ok, exif} ->
+        image
+        |> remove_metadata!()
+        |> put_copyright_and_artist(exif)
+
+      {:error, "No such field"} ->
+        remove_metadata(image)
+
+      other ->
+        other
     end
   end
 
+  defp put_copyright_and_artist(image, exif) do
+    Vimage.mutate(image, fn mut_img ->
+      if exif[:copyright], do: Exif.put_metadata(mut_img, :copyright, exif[:copyright])
+      if exif[:artist], do: Exif.put_metadata(mut_img, :artist, exif[:artist])
+      :ok
+    end)
+  end
+
   @doc """
   Minimize metadata by keeping only the artist
   and copyright (if available).
@@ -6437,7 +6453,7 @@ defmodule Image do
     in an image can be returned with a call to
     `Vix.Vips.Image.header_field_names/1`.
 
-  * Errors removing metadata fields is not propagated
+  * Errors removing metadata fields are not propagated
     into the return for this function. Errors might occur
     when attempting to remove metadata fields that
     do not exist in the image.
@@ -6501,7 +6517,7 @@ defmodule Image do
     in an image can be returned with a call to
     `Vix.Vips.Image.header_field_names/1`.
 
-  * Errors removing metadata fields is not propagated
+  * Errors removing metadata fields are not propagated
     into the return for this function. Errors might occur
     when attempting to remove metadata fields that
     do not exist in the image.

lib/image/exif/decode.ex

@@ -201,12 +201,16 @@ defmodule Image.Exif.Decode do
   @comp_conf {"-", "Y", "Cb", "Cr", "R", "G", "B"}
 
   @spec component_configuration([non_neg_integer()]) :: binary()
-  defp component_configuration(list) do
-    list
+  defp component_configuration(components) when is_list(components) do
+    components
     |> Enum.map(&elem(@comp_conf, &1))
     |> Enum.join(",")
   end
 
+  defp component_configuration(components) do
+    "Unknown #{inspect components}"
+  end
+
   @spec metering_mode(non_neg_integer()) :: binary()
   defp metering_mode(1), do: "Average"
   defp metering_mode(2), do: "Center-weighted average"

mix.exs

@@ -1,7 +1,7 @@
 defmodule Image.MixProject do
   use Mix.Project
 
-  @version "0.62.0"
+  @version "0.62.1"
 
   @app_name "image"
 
@@ -21,13 +21,21 @@ defmodule Image.MixProject do
       package: package(),
       aliases: aliases(),
       elixirc_paths: elixirc_paths(Mix.env()),
-      preferred_cli_env: preferred_cli_env(),
       dialyzer: [
         ignore_warnings: ".dialyzer_ignore_warnings",
         plt_add_apps: ~w(mix nx plug evision bumblebee ex_unit)a
       ],
       compilers: Mix.compilers()
     ]
+    |> Keyword.merge(maybe_add_preferred_cli())
+  end
+
+  defp maybe_add_preferred_cli() do
+    if Version.compare(System.version(), "1.19.0-dev") == :lt do
+      [preferred_cli_env: cli()]
+    else
+      []
+    end
   end
 
   defp description do
@@ -207,7 +215,8 @@ defmodule Image.MixProject do
     |> List.to_integer()
   end
 
-  defp preferred_cli_env() do
+  @doc false
+  def cli() do
     []
   end