Switch to built-in JSON module

Refs GH-480, GH-481
getsentry · Jun 10, 2016 · 29a7374 · 29a7374
1 parent 944e72b
commit 29a7374
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 667 deletions.
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -3,7 +3,6 @@ inherit_from: .rubocop_todo.yml
 AllCops:
   Exclude:
     - 'examples/**/*'
-    - 'lib/raven/okjson.rb'
     - 'lib/raven/utils/deep_merge.rb'
 
 Style/SignalException:

diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
@@ -363,7 +363,6 @@ Style/SpaceAroundOperators:
     - 'lib/raven/event.rb'
     - 'lib/raven/integrations/sidekiq.rb'
     - 'lib/raven/interfaces/stack_trace.rb'
-    - 'spec/raven/okjson_spec.rb'
     - 'spec/raven/processors/sanitizedata_processor_spec.rb'
 
 # Offense count: 6

diff --git a/lib/raven/client.rb b/lib/raven/client.rb
@@ -1,9 +1,9 @@
 # frozen_string_literal: true
-require 'zlib'
 require 'base64'
+require 'json'
+require 'zlib'
 
 require 'raven/version'
-require 'raven/okjson'
 require 'raven/transports/http'
 
 module Raven
@@ -73,7 +73,7 @@ def configuration_allows_sending
 
     def encode(event)
       hash = @processors.reduce(event.to_hash) { |memo, p| p.process(memo) }
-      encoded = OkJson.encode(hash)
+      encoded = JSON.generate(hash)
 
       case configuration.encoding
       when 'gzip'