Merge pull request #2 from logtail/th/fix-rack-values-encoding

Fix request Rack values encoding

Author: gyfis

lib/logtail-rack/http_events.rb

@@ -182,13 +182,13 @@ def call(env)
                 body: event_body,
                 content_length: request.content_length,
                 headers: request.headers,
-                host: request.host,
+                host: force_encoding(request.host),
                 method: request.request_method,
                 path: request.path,
                 port: request.port,
-                query_string: request.query_string,
+                query_string: force_encoding(request.query_string),
                 request_id: request.request_id,
-                scheme: request.scheme,
+                scheme: force_encoding(request.scheme),
                 body_limit: self.class.http_body_limit,
                 headers_to_sanitize: self.class.http_header_filters,
               )
@@ -271,6 +271,14 @@ def silenced?(env, request)
               false
             end
           end
+
+          def force_encoding(value)
+            if value.respond_to?(:force_encoding)
+              value.dup.force_encoding('UTF-8')
+            else
+              value
+            end
+          end
       end
     end
   end

lib/logtail-rack/version.rb

@@ -1,7 +1,7 @@
 module Logtail
   module Integrations
     module Rack
-      VERSION = "0.1.2"
+      VERSION = "0.1.3"
     end
   end
 end