bugsnag 6.26.1 → 6.26.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 60383f9662a63f219295a1554c0643092d201039d57ec4b93f7542761662105f
-  data.tar.gz: 04076a3736b8ff9ee5b575b083e328429f149cd4a531f5ac062581d9472feea9
+  metadata.gz: 51f147f279fd5a2a7bbe26663246348f4d4bbccf9d901febeda2620d533341d3
+  data.tar.gz: 3648e3a84b41ff738318c4ea8b0db7218a6c3dd149cd084604c9cbe102cfc593
 SHA512:
-  metadata.gz: '0826f29f80c34ec5e785dff331b185498b2d2862455f5011dc6405eedfdbe23188a811ba93f8673bc79834e3a5e8c46d63d0cfa5ca541c4864ef5367ec83e034'
-  data.tar.gz: 9a5bdf81e2ddd1b7ad7c0b2ec48463c288c9e8f55887454530c016a95a6f38f593e696af69adc558bd1a54df0f1fa104954c0d40612d70ac309c88765c6d6fb7
+  metadata.gz: d3942fed5bf6c50cdaa12d25e39c7e7218badf7ef1d9481baeacd220fe5a14271200e47d552eb2e35008fdb83c3a1eac11eaaaae72ea1228879ad5be2c191e57
+  data.tar.gz: 10a2182a9002b5d0f664ba8a442717c5dafbcb32cc96561c179189c5a3be0661dc824070eaa3c28d32f927a732549962e13d556ef8fc2b454a9f44a177b683af

data/CHANGELOG.md

@@ -1,6 +1,18 @@
 Changelog
 =========
 
+## v6.26.3 (24 January 2024)
+
+* Handle mailto links in `Cleaner#clean_url`
+  | [#813](https://github.com/bugsnag/bugsnag-ruby/pull/813)
+
+## v6.26.2 (17 January 2024)
+
+### Fixes
+
+* Fix unhandled `URI::InvalidURIError` in `Cleaner#clean_url`
+  | [#811](https://github.com/bugsnag/bugsnag-ruby/pull/811)
+
 ## v6.26.1 (9 January 2024)
 
 ### Fixes

data/VERSION

@@ -1 +1 @@
-6.26.1
+6.26.3

data/lib/bugsnag/cleaner.rb

@@ -26,21 +26,23 @@ module Bugsnag
     # @return [String]
     def clean_url(url)
       return url if @configuration.meta_data_filters.empty? && @configuration.redacted_keys.empty?
+      return url unless url.include?('?')
 
-      uri = URI(url)
-      return url unless uri.query
+      begin
+        uri = URI(url)
 
-      query_params = uri.query.split('&').map { |pair| pair.split('=') }
-      query_params.map! do |key, val|
-        if filters_match?(key)
-          "#{key}=#{FILTERED}"
+        if uri.is_a?(URI::MailTo)
+          clean_mailto_url(url, uri)
         else
-          "#{key}=#{val}"
+          clean_generic_url(url, uri)
         end
-      end
+      rescue URI::InvalidURIError
+        pre_query_string, _query_string = url.split('?', 2)
 
-      uri.query = query_params.join('&')
-      uri.to_s
+        "#{pre_query_string}?#{FILTERED}"
+      rescue StandardError
+        FILTERED
+      end
     end
 
     ##
@@ -201,5 +203,33 @@ module Bugsnag
         scope.start_with?("#{scope_to_filter}.")
       end
     end
+
+    def clean_generic_url(original_url, uri)
+      return original_url unless uri.query
+
+      query_params = uri.query.split('&').map { |pair| pair.split('=') }
+
+      uri.query = filter_uri_parameter_array(query_params).join('&')
+      uri.to_s
+    end
+
+    def clean_mailto_url(original_url, uri)
+      return original_url unless uri.headers
+
+      # headers in mailto links can't contain square brackets so we replace
+      # filtered parameters with 'FILTERED' instead of '[FILTERED]'
+      uri.headers = filter_uri_parameter_array(uri.headers, 'FILTERED').join('&')
+      uri.to_s
+    end
+
+    def filter_uri_parameter_array(parameters, replacement = FILTERED)
+      parameters.map do |key, value|
+        if filters_match?(key)
+          "#{key}=#{replacement}"
+        else
+          "#{key}=#{value}"
+        end
+      end
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bugsnag
 version: !ruby/object:Gem::Version
-  version: 6.26.1
+  version: 6.26.3
 platform: ruby
 authors:
 - James Smith
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-09 00:00:00.000000000 Z
+date: 2024-01-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -107,7 +107,7 @@ homepage: https://github.com/bugsnag/bugsnag-ruby
 licenses:
 - MIT
 metadata:
-  changelog_uri: https://github.com/bugsnag/bugsnag-ruby/blob/v6.26.1/CHANGELOG.md
+  changelog_uri: https://github.com/bugsnag/bugsnag-ruby/blob/v6.26.3/CHANGELOG.md
   documentation_uri: https://docs.bugsnag.com/platforms/ruby/
   source_code_uri: https://github.com/bugsnag/bugsnag-ruby/
   rubygems_mfa_required: 'true'