appsignal 1.0.5.beta.2 → 1.0.5.beta.3
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/CHANGELOG.md +1 -0
- data/lib/appsignal/integrations/mongo_ruby_driver.rb +1 -1
- data/lib/appsignal/js_exception_transaction.rb +2 -2
- data/lib/appsignal/transaction.rb +2 -2
- data/lib/appsignal/transmitter.rb +1 -1
- data/lib/appsignal/utils.rb +23 -0
- data/lib/appsignal/version.rb +1 -1
- data/spec/lib/appsignal/utils_spec.rb +28 -0
- metadata +2 -2
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: c11d01582a42d050f75ede7b8b0fbd0736e2c600
|
4
|
+
data.tar.gz: 049e4b5c7728b3dc1efbc3a20527b70d8b9cb5cd
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: d9ffe4b983a86b8b38e9d15969c9887c494311336470648f66837e0a70e4c58ae5fe92a6dd037e0bed8523befab31e0affd439515d1e9f520a29a034953f50a8
|
7
|
+
data.tar.gz: 1a808810e53ba4a781b93fc0f1c44ef4509c927df2b4f661cdff1497d0da8d41e61e1d5f5e4958644c15e42f4bef850b700aa47551c921ae529a699beca32138
|
data/CHANGELOG.md
CHANGED
@@ -28,7 +28,7 @@ module Appsignal
|
|
28
28
|
@transaction_index,
|
29
29
|
@data['name'],
|
30
30
|
@data['message'],
|
31
|
-
|
31
|
+
Appsignal::Utils.json_generate(@data['backtrace'])
|
32
32
|
)
|
33
33
|
end
|
34
34
|
|
@@ -43,7 +43,7 @@ module Appsignal
|
|
43
43
|
Appsignal::Extension.set_transaction_sample_data(
|
44
44
|
@transaction_index,
|
45
45
|
key.to_s,
|
46
|
-
|
46
|
+
Appsignal::Utils.json_generate(data)
|
47
47
|
)
|
48
48
|
rescue JSON::GeneratorError=>e
|
49
49
|
Appsignal.logger.error("JSON generate error (#{e.message}) for '#{data.inspect}'")
|
@@ -113,7 +113,7 @@ module Appsignal
|
|
113
113
|
Appsignal::Extension.set_transaction_sample_data(
|
114
114
|
transaction_index,
|
115
115
|
key.to_s,
|
116
|
-
|
116
|
+
Appsignal::Utils.json_generate(data)
|
117
117
|
)
|
118
118
|
rescue JSON::GeneratorError=>e
|
119
119
|
Appsignal.logger.error("JSON generate error (#{e.message}) for '#{data.inspect}'")
|
@@ -142,7 +142,7 @@ module Appsignal
|
|
142
142
|
transaction_index,
|
143
143
|
error.class.name,
|
144
144
|
error.message,
|
145
|
-
backtrace ?
|
145
|
+
backtrace ? Appsignal::Utils.json_generate(backtrace) : ''
|
146
146
|
)
|
147
147
|
rescue JSON::GeneratorError=>e
|
148
148
|
Appsignal.logger.error("JSON generate error (#{e.message}) for '#{backtrace.inspect}'")
|
@@ -53,7 +53,7 @@ module Appsignal
|
|
53
53
|
request['Content-Type'] = CONTENT_TYPE
|
54
54
|
request['Content-Encoding'] = CONTENT_ENCODING
|
55
55
|
request.body = Zlib::Deflate.deflate(
|
56
|
-
|
56
|
+
Appsignal::Utils.json_generate(payload),
|
57
57
|
Zlib::BEST_SPEED
|
58
58
|
)
|
59
59
|
end
|
data/lib/appsignal/utils.rb
CHANGED
@@ -32,5 +32,28 @@ module Appsignal
|
|
32
32
|
else key
|
33
33
|
end
|
34
34
|
end
|
35
|
+
|
36
|
+
def self.json_generate(body)
|
37
|
+
JSON.generate(jsonify(body))
|
38
|
+
end
|
39
|
+
|
40
|
+
def self.jsonify(value)
|
41
|
+
case value
|
42
|
+
when String
|
43
|
+
value.encode(
|
44
|
+
'utf-8',
|
45
|
+
:invalid => :replace,
|
46
|
+
:undef => :replace
|
47
|
+
)
|
48
|
+
when Numeric, NilClass, TrueClass, FalseClass
|
49
|
+
value
|
50
|
+
when Hash
|
51
|
+
Hash[value.map { |k, v| [jsonify(k), jsonify(v)] }]
|
52
|
+
when Array
|
53
|
+
value.map { |v| jsonify(v) }
|
54
|
+
else
|
55
|
+
jsonify(value.to_s)
|
56
|
+
end
|
57
|
+
end
|
35
58
|
end
|
36
59
|
end
|
data/lib/appsignal/version.rb
CHANGED
@@ -1,3 +1,5 @@
|
|
1
|
+
# encoding: UTF-8
|
2
|
+
|
1
3
|
require 'spec_helper'
|
2
4
|
|
3
5
|
describe Appsignal::Utils do
|
@@ -53,4 +55,30 @@ describe Appsignal::Utils do
|
|
53
55
|
end
|
54
56
|
end
|
55
57
|
end
|
58
|
+
|
59
|
+
describe ".json_generate" do
|
60
|
+
subject { Appsignal::Utils.json_generate(body) }
|
61
|
+
|
62
|
+
context "with a valid body" do
|
63
|
+
let(:body) { {'the' => 'payload'} }
|
64
|
+
|
65
|
+
it { should == "{\"the\":\"payload\"}" }
|
66
|
+
end
|
67
|
+
|
68
|
+
context "with a body that contains strings with invalid utf-8 content" do
|
69
|
+
let(:string_with_invalid_utf8) { [0x61, 0x61, 0x85].pack('c*') }
|
70
|
+
let(:body) { {
|
71
|
+
'field_one' => [0x61, 0x61].pack('c*'),
|
72
|
+
'field_two' => string_with_invalid_utf8,
|
73
|
+
'field_three' => [
|
74
|
+
'one', string_with_invalid_utf8
|
75
|
+
],
|
76
|
+
'field_four' => {
|
77
|
+
'one' => string_with_invalid_utf8
|
78
|
+
}
|
79
|
+
} }
|
80
|
+
|
81
|
+
it { should == "{\"field_one\":\"aa\",\"field_two\":\"aa�\",\"field_three\":[\"one\",\"aa�\"],\"field_four\":{\"one\":\"aa�\"}}" }
|
82
|
+
end
|
83
|
+
end
|
56
84
|
end
|
metadata
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: appsignal
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 1.0.5.beta.
|
4
|
+
version: 1.0.5.beta.3
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Robert Beekman
|
@@ -9,7 +9,7 @@ authors:
|
|
9
9
|
autorequire:
|
10
10
|
bindir: bin
|
11
11
|
cert_chain: []
|
12
|
-
date: 2016-03-
|
12
|
+
date: 2016-03-09 00:00:00.000000000 Z
|
13
13
|
dependencies:
|
14
14
|
- !ruby/object:Gem::Dependency
|
15
15
|
name: rack
|