brand_logo 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8ca5db19ef68c02172defd8e04ba4a228c5819c32e3f2aa8dd10de412b4889f1
-  data.tar.gz: 6a1bf3a2bda344977f005a0c12ae07c884fa24bc19d5bdabc650ccb57f2e9806
+  metadata.gz: e165515641f5a8bf584699ee81dbdf62040409d00b06ea0052a4270e7c25abc7
+  data.tar.gz: 915f8dd74b6df4eb896328298e5b6859d87fe95ca14e287f2b1f30b80d14372a
 SHA512:
-  metadata.gz: ee0e662864b0ede5a0a78b538109468efc96db273c41bb37ccb20c36f808e3869258e5e29792c8dfcb5a7410fc3c4fe45ffa8e9f6a3453197e421abc80a08173
-  data.tar.gz: 74c9c3d4fe4102fdc86bd2070129dc93b9f7b09da79677278c1ea0b3e4bc1cfdc6779195f4b800e0ea9d410c71b0f2d72f89c3ab59354d40b34385bac48557f3
+  metadata.gz: f03b21ed953c2320a238c39b4dddf1df5d78258f1b98070258831b1c51c2a1116924d01508bfde563e3f5303cb50f197dfa36d5c1d775511b9f6d6f6024eab4b
+  data.tar.gz: 4b1a37cc4dba262c158d920e0879f07943ffc84218d7aa68207aff72539171d4f8391556acafb6ad7f3f5e070df608e5404cbb2f5c7e5d0240ee81202594e8f4

data/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented here.
 Format: [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## [2.0.1] - 2026-10-09
+
+### Fixed
+
+- JSON-LD and web app manifests containing duplicated keys or comments are parsed again with `json` 3, which rejects them by default.
+
+### Changed
+
+- CI tests the minimum supported dependency versions (`gemfiles/minimum.gemfile`, Ruby 3.2) and Dependabot is enabled.
+- Gem metadata: removed the `homepage_uri` duplicate of `source_code_uri`.
+
 ## [2.0.0] - 2026-10-02
 
 Rewrite focused on picking the right icon, speed and safety. Breaking changes: see *Upgrading*.

data/lib/brand_logo/lenient_json.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+# typed: strict
+
+require 'json'
+
+module BrandLogo
+  module LenientJson
+    extend T::Sig
+
+    # json >= 3 rejects duplicate keys and comments by default; json 2.x ignores these options.
+    OPTIONS = T.let({ allow_duplicate_key: true, allow_comments: true }.freeze, T::Hash[Symbol, T::Boolean])
+
+    sig { params(text: String).returns(T.untyped) }
+    def self.parse(text)
+      JSON.parse(text, **OPTIONS)
+    end
+  end
+end

data/lib/brand_logo/strategies/json_ld.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 # typed: strict
 
-require 'json'
-
 module BrandLogo
   module Strategies
     class JsonLd
@@ -25,7 +23,7 @@ module BrandLogo
 
         logos = []
         page.document.css('script[type="application/ld+json"]').each do |script|
-          collect(JSON.parse(script.text), false, logos)
+          collect(LenientJson.parse(script.text), false, logos)
         rescue JSON::ParserError => e
           Logging.logger.warn("JsonLd: invalid JSON at #{page.url}: #{e.message}")
         end

data/lib/brand_logo/strategies/manifest.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 # typed: strict
 
-require 'json'
-
 module BrandLogo
   module Strategies
     class Manifest
@@ -31,7 +29,7 @@ module BrandLogo
 
       sig { params(response: Http::Response).returns(T::Array[T.untyped]) }
       def parse(response)
-        data = JSON.parse(response.body)
+        data = LenientJson.parse(response.body)
         icons = data['icons'] if data.is_a?(::Hash)
         icons.is_a?(::Array) ? icons : []
       rescue JSON::ParserError => e

data/lib/brand_logo/version.rb

@@ -2,5 +2,5 @@
 # typed: strict
 
 module BrandLogo
-  VERSION = '2.0.0'
+  VERSION = '2.0.1'
 end

data/lib/brand_logo.rb

@@ -20,6 +20,7 @@ require_relative 'brand_logo/http/client'
 require_relative 'brand_logo/http/url_guard'
 require_relative 'brand_logo/http/real_client'
 
+require_relative 'brand_logo/lenient_json'
 require_relative 'brand_logo/url_resolver'
 require_relative 'brand_logo/image_probe'
 require_relative 'brand_logo/page'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: brand_logo
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - AdVitam
@@ -105,6 +105,7 @@ files:
 - lib/brand_logo/icon.rb
 - lib/brand_logo/image_format.rb
 - lib/brand_logo/image_probe.rb
+- lib/brand_logo/lenient_json.rb
 - lib/brand_logo/logging.rb
 - lib/brand_logo/lookup.rb
 - lib/brand_logo/page.rb
@@ -126,7 +127,6 @@ licenses:
 - MIT
 metadata:
   rubygems_mfa_required: 'true'
-  homepage_uri: https://github.com/AdVitam/brand_logo
   source_code_uri: https://github.com/AdVitam/brand_logo
   changelog_uri: https://github.com/AdVitam/brand_logo/blob/master/CHANGELOG.md
   bug_tracker_uri: https://github.com/AdVitam/brand_logo/issues