pliny 0.26.1 → 0.29.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: e4adaaedbebc3ae5352103ddadbf1f85ee0ea368
-  data.tar.gz: d0c837f014f54d02eb7cc28903fdec073502564b
+SHA256:
+  metadata.gz: e47d761bd19ec78a4558b81ab7018a268bea25545895a67c07f21623699c21ba
+  data.tar.gz: ee9b112e0a31a4e9646d3e60b02f8c8538440c770dae8c6376031bbad285b7dd
 SHA512:
-  metadata.gz: f50ff132e89a39ca8d28c9e873dd3f85a9d4fb2210fda66eaa7634af7a4a4647db0e5e158554add566dbdea69400c70332317153e45aa945315466a9b64ea58a
-  data.tar.gz: 88082d4d62b2b4b6e59011dfa22420f942242990fe32256be729f91400b5b5ab6d92035353c710053a82977db99dfc5929eccc7f0eca4bea1fe9488e05b21ac8
+  metadata.gz: 3e5682954ac33fa5b7231c50cc59101c4fe811611e21cdc9ffc0eb06d586281692d5bc6c9231f0d553cc4b3cc275640b953cd239d219c46576d2425725f92f72
+  data.tar.gz: 0be3d40932b255ca70c0101cfd20a5680d5bc01fd61d8d448d27ca9f4c52043d25ec48af172122f30ab28ef4a37315c37f28358feec36edebcf7bf068a24560d

data/lib/pliny.rb

@@ -8,6 +8,7 @@ require_relative "pliny/errors"
 require_relative "pliny/helpers/encode"
 require_relative "pliny/helpers/params"
 require_relative "pliny/helpers/serialize"
+require_relative "pliny/helpers/zulu_time"
 require_relative "pliny/log"
 require_relative "pliny/metrics/backends/logger"
 require_relative "pliny/metrics"

data/lib/pliny/commands/updater.rb

@@ -1,7 +1,7 @@
-require 'fileutils'
-require 'pathname'
-require 'pliny/version'
-require 'uri'
+require "fileutils"
+require "pathname"
+require "pliny/version"
+require "uri"
 
 module Pliny::Commands
   class Updater
@@ -37,7 +37,7 @@ module Pliny::Commands
 
     # we need a local copy of the pliny repo to produce a diff
     def ensure_repo_available
-      if File.exists?(repo_dir)
+      if File.exist?(repo_dir)
         unless system("cd #{repo_dir} && git fetch --tags")
           abort("Could not update Pliny repo at #{repo_dir}")
         end

data/lib/pliny/db_support.rb

@@ -1,6 +1,7 @@
 require "logger"
 require "sequel"
 require "sequel/extensions/migration"
+require "stringio"
 
 module Pliny
   class DbSupport

data/lib/pliny/error_reporters/rollbar.rb

@@ -21,6 +21,7 @@ module Pliny
         scope = { custom: context }
         unless rack_env.empty?
           scope[:request] = proc { extract_request_data_from_rack(rack_env) }
+          scope[:person] = proc { extract_person_data_from_controller(rack_env) }
         end
         scope
       rescue Exception => e

data/lib/pliny/helpers/params.rb

@@ -21,10 +21,21 @@ module Pliny::Helpers
     def load_params(data)
       # Sinatra 1.x only supports the method. Sinatra 2.x only supports the class
       if defined?(Sinatra::IndifferentHash)
-        Sinatra::IndifferentHash[data]
+        indifferent_params_v2(data)
       else
         indifferent_params(data)
       end
     end
+
+    def indifferent_params_v2(data)
+      case data
+      when Hash
+        Sinatra::IndifferentHash[data]
+      when Array
+        data.map { |item| indifferent_params_v2(item) }
+      else
+        data
+      end
+    end
   end
 end

data/lib/pliny/helpers/serialize.rb

@@ -1,42 +1,41 @@
 module Pliny::Helpers
   module Serialize
-    def self.included(base)
-      base.send :extend, ClassMethods
+    def self.registered(base)
+      base.helpers Helpers
+      base.set :serializer_class, nil
     end
 
-    def serialize(data, structure = :default)
-      serializer_class = self.class.serializer_class
+    module Helpers
+      def serialize(data, structure = :default)
+        serializer_class = settings.serializer_class
 
-      if serializer_class.nil?
-        raise <<-eos.strip
-No serializer has been specified for this endpoint. Please specify one with
-`serializer Serializers::ModelName` in the endpoint.
-        eos
-      end
+        if serializer_class.nil?
+          raise <<~eos.strip
+            No serializer has been specified for this endpoint. Please specify one with
+            `serializer Serializers::ModelName` in the endpoint.
+          eos
+        end
 
-      env['pliny.serializer_arity'] = data.respond_to?(:size) ? data.size : 1
+        env['pliny.serializer_arity'] = data.respond_to?(:size) ? data.size : 1
 
-      start = Time.now
-      serializer_class.new(structure).serialize(data).tap do
-        env['pliny.serializer_timing'] = (Time.now - start).to_f
+        start = Time.now
+        serializer_class.new(structure).serialize(data).tap do
+          env['pliny.serializer_timing'] = (Time.now - start).to_f
+        end
       end
     end
 
-    module ClassMethods
-      # Provide a way to specify endpoint serializer class.
-      #
-      # class Endpoints::User < Base
-      #   serializer Serializers::User
-      #
-      #   get do
-      #     encode serialize(User.all)
-      #   end
-      # end
-      def serializer(serializer_class)
-        @serializer_class = serializer_class
-      end
-
-      attr_reader :serializer_class
+    # Provide a way to specify endpoint serializer class.
+    #
+    # class Endpoints::User < Base
+    #   serializer Serializers::User
+    #
+    #   get do
+    #     encode serialize(User.all)
+    #   end
+    # end
+    def serializer(serializer_class)
+      set :serializer_class, serializer_class
     end
   end
 end

data/lib/pliny/helpers/zulu_time.rb

@@ -0,0 +1,7 @@
+module Pliny::Helpers
+  module ZuluTime
+    def zulu_time(time)
+      time ? time.getutc.strftime("%Y-%m-%dT%H:%M:%SZ") : nil
+    end
+  end
+end

data/lib/pliny/log.rb

@@ -124,7 +124,6 @@ module Pliny
     end
 
     def quote_string(k, v)
-      # try to find a quote style that fits
       if !v.include?('"')
         %{#{k}="#{v}"}
       elsif !v.include?("'")
@@ -140,19 +139,22 @@ module Pliny
 
     def unparse_pair(k, v)
       v = v.call if v.is_a?(Proc)
-      # only quote strings if they include whitespace
+
       if v == nil
         nil
       elsif v == true
         k
       elsif v.is_a?(Float)
         "#{k}=#{format("%.3f", v)}"
-      elsif v.is_a?(String) && v =~ /\s/
-        quote_string(k, v)
       elsif v.is_a?(Time)
         "#{k}=#{v.iso8601}"
       else
-        "#{k}=#{v}"
+        v = "#{v}"
+        if v =~ /\s/
+          quote_string(k, v)
+        else
+          "#{k}=#{v}"
+        end
       end
     end
   end

data/lib/pliny/tasks/db.rake

@@ -32,12 +32,12 @@ begin
 
     desc "Seed the database with data"
     task :seed do
-      if File.exist?('./db/seeds.rb')
+      if File.exist?("./db/seeds.rb")
         database_urls.each do |database_url|
           # make a DB instance available to the seeds file
-          self.class.send(:remove_const, 'DB') if self.class.const_defined?('DB')
-          self.class.const_set('DB', Sequel.connect(database_url))
-          load 'db/seeds.rb'
+          self.class.send(:remove_const, "DB") if self.class.const_defined?("DB")
+          self.class.const_set("DB", Sequel.connect(database_url))
+          load "db/seeds.rb"
         end
         disconnect
       end
@@ -66,7 +66,7 @@ begin
         admin_url = Pliny::DbSupport.admin_url(database_url)
         db = Sequel.connect(admin_url)
         name = name_from_uri(database_url)
-        db.run(%{DROP DATABASE IF EXISTS "#{name}"})
+        db.run(%(DROP DATABASE IF EXISTS "#{name}"))
         puts "Dropped `#{name}`"
       end
       disconnect
@@ -75,7 +75,7 @@ begin
     namespace :schema do
       desc "Load the database schema"
       task :load do
-        if File.exists?("./db/schema.sql")
+        if File.exist?("./db/schema.sql")
           schema = File.read("./db/schema.sql")
           database_urls.each do |database_url|
             db = Sequel.connect(database_url)
@@ -107,7 +107,7 @@ begin
           search_path = db.dataset.with_sql("SHOW search_path").single_value
           schema << "SET search_path = #{search_path};\n\n"
 
-          db[:schema_migrations].each do |migration|
+          db[:schema_migrations].order_by(:filename).each do |migration|
             schema << db[:schema_migrations].insert_sql(migration) + ";\n"
           end
         end
@@ -118,14 +118,14 @@ begin
       end
 
       desc "Merges migrations into schema and removes them"
-      task :merge => ["db:setup", "db:schema:dump"] do
+      task merge: ["db:setup", "db:schema:dump"] do
         FileUtils.rm Dir["./db/migrate/*.rb"]
         puts "Removed migrations"
       end
     end
 
     desc "Setup the database"
-    task :setup => [:drop, :create, "schema:load", :migrate, :seed]
+    task setup: [:drop, :create, "schema:load", :migrate, :seed]
 
     private
 
@@ -138,12 +138,10 @@ begin
       if ENV["DATABASE_URL"]
         [ENV["DATABASE_URL"]]
       else
-        %w(.env .env.test).map { |env_file|
+        %w[.env .env.test].map { |env_file|
           env_path = "./#{env_file}"
-          if File.exists?(env_path)
+          if File.exist?(env_path)
             Pliny::Utils.parse_env(env_path)["DATABASE_URL"]
-          else
-            nil
           end
         }.compact
       end
@@ -153,7 +151,6 @@ begin
       URI.parse(uri).path[1..-1]
     end
   end
-
 rescue LoadError
   puts "Couldn't load sequel. Skipping database tasks"
 end

data/lib/pliny/templates/serializer.erb

@@ -10,9 +10,9 @@
 <%=  ident %>class <%= modules.last %> < Serializers::Base
 <%=  ident %>  structure(:default) do |arg|
 <%=  ident %>    {
-<%=  ident %>      created_at: arg.created_at.try(:iso8601),
+<%=  ident %>      created_at: zulu_time(arg.created_at),
 <%=  ident %>      id:         arg.id,
-<%=  ident %>      updated_at: arg.updated_at.try(:iso8601),
+<%=  ident %>      updated_at: zulu_time(arg.updated_at)
 <%=  ident %>    }
 <%=  ident %>  end
 <%=  ident %>end

data/lib/pliny/version.rb

@@ -1,3 +1,3 @@
 module Pliny
-  VERSION = "0.26.1"
+  VERSION = "0.29.0"
 end

data/lib/template/Gemfile

@@ -4,7 +4,7 @@ ruby "2.4.0"
 gem "multi_json"
 gem "oj"
 gem "pg"
-gem "pliny", "~> 0.26"
+gem "pliny", "~> 0.28"
 gem "pry"
 gem "puma", "~> 3"
 gem "rack-ssl"
@@ -21,7 +21,7 @@ gem "sucker_punch"
 
 group :development, :test do
   gem "pry-byebug"
-  gem "rubocop", "~> 0.46.0", require: false
+  gem "rubocop", "~> 0.52.1", require: false
   gem "rubocop-rspec", require: false
 end
 

data/lib/template/lib/endpoints/base.rb

@@ -5,7 +5,7 @@ module Endpoints
 
     helpers Pliny::Helpers::Encode
     helpers Pliny::Helpers::Params
-    helpers Pliny::Helpers::Serialize
+    register Pliny::Helpers::Serialize
 
     set :dump_errors, false
     set :raise_errors, true

data/lib/template/lib/serializers/base.rb

@@ -1,5 +1,7 @@
 module Serializers
   class Base
+    extend Pliny::Helpers::ZuluTime
+
     @@structures = {}
 
     def self.structure(type, &blk)

data/spec/commands/creator_spec.rb

@@ -1,5 +1,5 @@
-require 'pliny/commands/creator'
-require 'spec_helper'
+require "pliny/commands/creator"
+require "spec_helper"
 
 describe Pliny::Commands::Creator do
   before do
@@ -7,21 +7,23 @@ describe Pliny::Commands::Creator do
   end
 
   describe "#run!" do
-    before do
-      FileUtils.rm_rf("/tmp/plinytest")
-      FileUtils.mkdir_p("/tmp/plinytest")
-      Dir.chdir("/tmp/plinytest")
+    around do |example|
+      Dir.mktmpdir("plinytest-") do |dir|
+        Dir.chdir(dir) do
+          example.run
+        end
+      end
     end
 
     it "copies the template app over" do
       @gen.run!
-      assert File.exists?("./foobar")
-      assert File.exists?("./foobar/Gemfile")
+      assert File.exist?("./foobar")
+      assert File.exist?("./foobar/Gemfile")
     end
 
     it "deletes the .git from it" do
       @gen.run!
-      refute File.exists?("./foobar/.git")
+      refute File.exist?("./foobar/.git")
     end
 
     it "deletes the .erb files from it" do
@@ -31,25 +33,25 @@ describe Pliny::Commands::Creator do
 
     it "changes DATABASE_URL in .env.sample to use the app name" do
       @gen.run!
-      db_url = File.read("./foobar/.env.sample").split("\n").detect do |line|
+      db_url = File.read("./foobar/.env.sample").split("\n").detect { |line|
         line.include?("DATABASE_URL=")
-      end
+      }
       assert_equal "DATABASE_URL=postgres:///foobar-development", db_url
     end
 
     it "changes DATABASE_URL in .env.test to use the app name" do
       @gen.run!
-      db_url = File.read("./foobar/.env.test").split("\n").detect do |line|
+      db_url = File.read("./foobar/.env.test").split("\n").detect { |line|
         line.include?("DATABASE_URL=")
-      end
+      }
       assert_equal "DATABASE_URL=postgres:///foobar-test", db_url
     end
 
     it "changes APP_NAME in app.json to use the app name" do
       @gen.run!
-      db_url = File.read("./foobar/app.json").split("\n").detect do |line|
+      db_url = File.read("./foobar/app.json").split("\n").detect { |line|
         line.include?("\"name\":")
-      end
+      }
       assert_equal "  \"name\": \"foobar\",", db_url
     end
   end

data/spec/commands/updater_spec.rb

@@ -1,5 +1,5 @@
-require 'pliny/commands/updater'
-require 'spec_helper'
+require "pliny/commands/updater"
+require "spec_helper"
 
 describe Pliny::Commands::Updater do
   before do
@@ -10,19 +10,21 @@ describe Pliny::Commands::Updater do
   end
 
   describe "#run!" do
-    before do
-      FileUtils.rm_rf("/tmp/plinytest")
-      FileUtils.mkdir_p("/tmp/plinytest")
-      Dir.chdir("/tmp/plinytest")
-      File.open("/tmp/plinytest/Gemfile.lock", "w") do |f|
-        f.puts "    pliny (0.6.3)"
+    around do |example|
+      Dir.mktmpdir("plinytest-") do |dir|
+        Dir.chdir(dir) do
+          File.open("./Gemfile.lock", "w") do |f|
+            f.puts "    pliny (0.6.3)"
+          end
+          example.run
+        end
       end
     end
 
     it "creates a patch with Pliny diffs between the two versions" do
       @cmd.run!
       patch = File.read(@cmd.patch_file)
-      assert patch.include?('--- a/Gemfile')
+      assert patch.include?("--- a/Gemfile")
       assert patch.include?('-gem "pliny", "~> 0.6"')
     end
   end

data/spec/db_support_spec.rb

@@ -6,15 +6,18 @@ describe Pliny::DbSupport do
   let(:logger) { Logger.new(StringIO.new) }
   let(:support) { Pliny::DbSupport.new(url, logger) }
 
-  before do
-    @path = "/tmp/pliny-test"
+  around do |example|
+    Dir.mktmpdir("plinytest-") do |dir|
+      Dir.chdir(dir) do
+        FileUtils.mkdir_p("./db/migrate")
+
+        example.run
+      end
+    end
   end
 
-  before(:each) do
+  before do
     DB.tables.each { |t| DB.drop_table(t) }
-    FileUtils.rm_rf(@path)
-    FileUtils.mkdir_p("#{@path}/db/migrate")
-    Dir.chdir(@path)
   end
 
   describe ".admin_url" do
@@ -33,7 +36,7 @@ describe Pliny::DbSupport do
 
   describe "#migrate" do
     before do
-      File.open("#{@path}/db/migrate/#{Time.now.to_i}_create_foo.rb", "w") do |f|
+      File.open("./db/migrate/#{Time.now.to_i}_create_foo.rb", "w") do |f|
         f.puts "
           Sequel.migration do
             change do
@@ -57,11 +60,11 @@ describe Pliny::DbSupport do
   describe "#rollback" do
     before do
       @t = Time.now
-      File.open("#{@path}/db/migrate/#{(@t-2).to_i}_first.rb", "w") do |f|
+      File.open("./db/migrate/#{(@t - 2).to_i}_first.rb", "w") do |f|
         f.puts "Sequel.migration { change { create_table(:first) } }"
       end
 
-      File.open("#{@path}/db/migrate/#{(@t-1).to_i}_second.rb", "w") do |f|
+      File.open("./db/migrate/#{(@t - 1).to_i}_second.rb", "w") do |f|
         f.puts "Sequel.migration { change { create_table(:second) } }"
       end
     end
@@ -80,7 +83,7 @@ describe Pliny::DbSupport do
     end
 
     it "handles databases not migrated (schema_migrations is empty)" do
-      support.migrate (@t-2).to_i
+      support.migrate((@t - 2).to_i)
       support.rollback # destroy table, leave schema_migrations
       support.rollback # should noop
     end

data/spec/error_reporters/rollbar_spec.rb

@@ -33,6 +33,24 @@ describe Pliny::ErrorReporters::Rollbar do
       ))
     end
 
+    context "given a rack_env with person data" do
+      let(:rack_env) do
+        { "rollbar.person_data" => {
+          id: SecureRandom.uuid,
+          email: "foo@bar.com",
+          username: "foo"
+        }}
+      end
+
+      it "adds person to the rollbar notification" do
+        scope = nil
+        allow(::Rollbar).to receive(:scoped) { |arg| scope = arg }
+        notify
+        assert_kind_of(Proc, scope[:person])
+        assert_equal(rack_env["rollbar.person_data"], scope[:person].call)
+      end
+    end
+
     it "delegates to #report_exception_to_rollbar" do
       notify
       expect(reporter).to have_received(:report_exception_to_rollbar)

data/spec/helpers/params_spec.rb

@@ -1,7 +1,6 @@
 require "spec_helper"
 
 describe Pliny::Helpers::Params do
-
   def app
     Sinatra.new do
       helpers Pliny::Helpers::Params
@@ -12,17 +11,27 @@ describe Pliny::Helpers::Params do
   end
 
   it "loads json params" do
-    post "/", {hello: "world"}.to_json, {'CONTENT_TYPE' => 'application/json'}
+    post "/", {hello: "world"}.to_json, {"CONTENT_TYPE" => "application/json"}
     assert_equal "{\"hello\":\"world\"}", last_response.body
   end
 
+  it "loads json array of params" do
+    post "/", [{hello: "world"}, {goodbye: "moon"}].to_json, {"CONTENT_TYPE" => "application/json"}
+    assert_equal "[{\"hello\":\"world\"},{\"goodbye\":\"moon\"}]", last_response.body
+  end
+
+  it "loads json array of arrays of params" do
+    post "/", [[{hello: "world"}], [{goodbye: "moon"}]].to_json, {"CONTENT_TYPE" => "application/json"}
+    assert_equal "[[{\"hello\":\"world\"}],[{\"goodbye\":\"moon\"}]]", last_response.body
+  end
+
   it "loads form data params" do
     post "/", {hello: "world"}
     assert_equal "{\"hello\":\"world\"}", last_response.body
   end
 
   it "loads from an unknown content type" do
-    post "/", "<hello>world</hello>", {'CONTENT_TYPE' => 'application/xml'}
+    post "/", "<hello>world</hello>", {"CONTENT_TYPE" => "application/xml"}
     assert_equal "{}", last_response.body
   end
 end

data/spec/helpers/serialize_spec.rb

@@ -4,7 +4,7 @@ describe Pliny::Helpers::Serialize do
   context "without a serializer" do
     def app
       Sinatra.new do
-        helpers Pliny::Helpers::Serialize
+        register Pliny::Helpers::Serialize
 
         get "/" do
           MultiJson.encode(serialize([]))
@@ -29,7 +29,7 @@ describe Pliny::Helpers::Serialize do
 
     def app
       Sinatra.new do
-        helpers Pliny::Helpers::Serialize
+        register Pliny::Helpers::Serialize
 
         serializer Serializer
 

data/spec/helpers/zulu_time_spec.rb

@@ -0,0 +1,24 @@
+require "spec_helper"
+require "active_support/core_ext/numeric/time"
+
+describe Pliny::Helpers::ZuluTime do
+  context "zulu_time" do
+    class ZuluTimeTest
+      extend Pliny::Helpers::ZuluTime
+    end
+
+    it "it formats Time instances" do
+      formatted = ZuluTimeTest.zulu_time(Time.parse("2017-11-28T21:49:52.123+00:00"))
+      assert_equal "2017-11-28T21:49:52Z", formatted
+    end
+
+    it "it formats DateTime instances" do
+      formatted = ZuluTimeTest.zulu_time(DateTime.parse("2017-11-28T21:49:52.123+00:00"))
+      assert_equal "2017-11-28T21:49:52Z", formatted
+    end
+
+    it "when called with nil" do
+      assert_nil ZuluTimeTest.zulu_time(nil)
+    end
+  end
+end

data/spec/integration_spec.rb

@@ -2,19 +2,24 @@ require "spec_helper"
 
 describe "Pliny integration test" do
   before(:all) do
-    FileUtils.rm_rf("/tmp/plinytest")
-    FileUtils.mkdir_p("/tmp/plinytest")
-    Dir.chdir("/tmp/plinytest")
+    @original_dir = Dir.pwd
+
+    test_dir = Dir.mktmpdir("plinytest-")
+    Dir.chdir(test_dir)
 
     bash "pliny-new myapp"
 
-    Dir.chdir("/tmp/plinytest/myapp")
+    Dir.chdir("myapp")
     bash "bin/setup"
   end
 
+  after(:all) do
+    Dir.chdir(@original_dir)
+  end
+
   describe "bin/setup" do
     it "generates .env" do
-      assert File.exists?("./.env")
+      assert File.exist?("./.env")
     end
   end
 
@@ -24,19 +29,19 @@ describe "Pliny integration test" do
     end
 
     it "creates the model file" do
-      assert File.exists?("./lib/models/artist.rb")
+      assert File.exist?("./lib/models/artist.rb")
     end
 
     it "creates the endpoint file" do
-      assert File.exists?("./lib/endpoints/artists.rb")
+      assert File.exist?("./lib/endpoints/artists.rb")
     end
 
     it "creates the serializer file" do
-      assert File.exists?("./lib/serializers/artist.rb")
+      assert File.exist?("./lib/serializers/artist.rb")
     end
 
     it "creates the schema file" do
-      assert File.exists?("./schema/schemata/artist.yaml")
+      assert File.exist?("./schema/schemata/artist.yaml")
     end
   end
 

data/spec/log_spec.rb

@@ -5,7 +5,6 @@ describe Pliny::Log do
     @io = StringIO.new
     Pliny.stdout = @io
     Pliny.stderr = @io
-    allow(@io).to receive(:print)
   end
 
   after do
@@ -130,4 +129,66 @@ describe Pliny::Log do
       end
     end
   end
+
+  describe "unparsing" do
+    it "removes nils from log" do
+      expect(@io).to receive(:print).with("\n")
+
+      Pliny.log(foo: nil)
+    end
+
+    it "leaves bare keys for true values" do
+      expect(@io).to receive(:print).with("foo\n")
+
+      Pliny.log(foo: true)
+    end
+
+    it "truncates floats" do
+      expect(@io).to receive(:print).with("foo=3.142\n")
+
+      Pliny.log(foo: Math::PI)
+    end
+
+    it "outputs times in ISO8601 format" do
+      expect(@io).to receive(:print).with("foo=2020-01-01T00:00:00Z\n")
+
+      Pliny.log(foo: Time.utc(2020))
+    end
+
+    it "quotes strings that contain spaces" do
+      expect(@io).to receive(:print).with("foo=\"string with spaces\"\n")
+
+      Pliny.log(foo: "string with spaces")
+    end
+
+    it "by default interpolates objects into strings" do
+      expect(@io).to receive(:print).with("foo=message\n")
+      expect(@io).to receive(:print).with("foo=42\n")
+      expect(@io).to receive(:print).with("foo=bar\n")
+
+      Pliny.log(foo: StandardError.new("message"))
+      Pliny.log(foo: 42)
+
+      klass = Class.new do
+        def to_s
+          "bar"
+        end
+      end
+      Pliny.log(foo: klass.new)
+    end
+
+    it "quotes strings that are generated from object interpolation" do
+      expect(@io).to receive(:print).with("foo=\"message with space\"\n")
+      expect(@io).to receive(:print).with("foo=\"bar with space\"\n")
+
+      Pliny.log(foo: StandardError.new("message with space"))
+
+      klass = Class.new do
+        def to_s
+          "bar with space"
+        end
+      end
+      Pliny.log(foo: klass.new)
+    end
+  end
 end

metadata

@@ -1,36 +1,36 @@
 --- !ruby/object:Gem::Specification
 name: pliny
 version: !ruby/object:Gem::Version
-  version: 0.26.1
+  version: 0.29.0
 platform: ruby
 authors:
 - Brandur Leach
 - Pedro Belo
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-12-04 00:00:00.000000000 Z
+date: 2020-09-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.0.1
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.0.1
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.0'
 - !ruby/object:Gem::Dependency
   name: multi_json
   requirement: !ruby/object:Gem::Requirement
@@ -157,60 +157,48 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.8.7
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.8.7
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rack-test
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.6.2
+        version: 1.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.6.2
+        version: 1.1.0
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.1'
     - - ">="
       - !ruby/object:Gem::Version
         version: 3.1.0
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.1'
     - - ">="
       - !ruby/object:Gem::Version
         version: 3.1.0
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.1'
 - !ruby/object:Gem::Dependency
   name: sinatra-contrib
   requirement: !ruby/object:Gem::Requirement
@@ -303,62 +291,62 @@ dependencies:
   name: rollbar
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.11'
     - - ">="
       - !ruby/object:Gem::Version
         version: 2.11.0
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.11'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.11'
     - - ">="
       - !ruby/object:Gem::Version
         version: 2.11.0
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.11'
 - !ruby/object:Gem::Dependency
   name: sequel
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '4.9'
     - - ">="
       - !ruby/object:Gem::Version
         version: 4.9.0
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '4.9'
     - - ">="
       - !ruby/object:Gem::Version
         version: 4.9.0
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.9'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.46'
+        version: '0.52'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.46.0
+        version: 0.52.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.46'
+        version: '0.52'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.46.0
+        version: 0.52.1
 description: Pliny is a set of base classes and helpers to help developers write excellent
   APIs in Sinatra
 email:
@@ -395,6 +383,7 @@ files:
 - lib/pliny/helpers/encode.rb
 - lib/pliny/helpers/params.rb
 - lib/pliny/helpers/serialize.rb
+- lib/pliny/helpers/zulu_time.rb
 - lib/pliny/log.rb
 - lib/pliny/metrics.rb
 - lib/pliny/metrics/backends/logger.rb
@@ -490,6 +479,7 @@ files:
 - spec/helpers/encode_spec.rb
 - spec/helpers/params_spec.rb
 - spec/helpers/serialize_spec.rb
+- spec/helpers/zulu_time_spec.rb
 - spec/integration_spec.rb
 - spec/log_spec.rb
 - spec/metrics/backends/logger_spec.rb
@@ -513,7 +503,7 @@ homepage: https://github.com/interagent/pliny
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -528,9 +518,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.13
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: Basic tooling to support API apps in Sinatra
 test_files: []