rack-exception_notifier 0.3.1 → 0.3.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
checksums.yaml CHANGED
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  SHA1:
3
- metadata.gz: 831f3eb0493ce8f92e0b543bfeed9da8d0b40e8c
4
- data.tar.gz: 68fc3df5db1d31239fe076a1d7377600c92b3032
3
+ metadata.gz: af68cee5a1ae828ca8d935a675800f38e13be954
4
+ data.tar.gz: 099cd9b3e73574cb4c0e0d28498aa7468e3ff14a
5
5
  SHA512:
6
- metadata.gz: bc1729664da7caacec2a61d0c87a85cf4848f882adec34641d25443adb8018b8d0eb66fc736240aa8b82fea237bc2803caf215f669bb0331ed515c0b86aa66c2
7
- data.tar.gz: 3a00d434a5bd64194bfea88f1c8c10b2c4bba5a5df32996e5a476685cd5b3603f99de2cfd875e931af380e4c1c138dcbf62cdaf756f966a293d1e2eacb8007f7
6
+ metadata.gz: 02d009be9296b8a13ee7c40ef78257ac9f52e338a020ac9b4160ef862824560e440bf70a5b3d7f556dbdd6be99385e5b9c52bb114cbbb32fe2c17d015ad38736
7
+ data.tar.gz: 4a848d2870470ae92d7d6ab8c5192e0a1fe6b5158e2bae2459c10acbbacf1854d4b834aa9f969c761ffaf0593b561703fc708820b651818815c23a69710ca1a9
data/README.md CHANGED
@@ -28,7 +28,3 @@ use Rack::ExceptionNotifier,
28
28
  * `:from` - address to originate email exceptions from (default: running username)
29
29
  * `:subject` - subject line for emails, %s will be interpolated with the exception #to_s (default: [ERROR] %s)
30
30
  * `:include_body` - should the body of the request be included in the email (default: false)
31
-
32
- ## License
33
-
34
- Rack-exception_notifier is released under the [MIT license](http://www.opensource.org/licenses/MIT).
@@ -3,6 +3,8 @@ require 'mail'
3
3
 
4
4
  module Rack
5
5
  class ExceptionNotifier
6
+ ExcludeBodyKeys = %w[rack.input rack.request.form_hash rack.request.form_input rack.request.form_vars]
7
+
6
8
  def initialize(app, options)
7
9
  default_options = {
8
10
  :to => nil,
@@ -40,6 +42,14 @@ module Rack
40
42
  env['rack.input'].size > 0
41
43
  end
42
44
 
45
+ def _exclude_env_key?(env, key)
46
+ if _render_body?(env)
47
+ false
48
+ else
49
+ ExcludeBodyKeys.include?(key)
50
+ end
51
+ end
52
+
43
53
  def _render_body?(env)
44
54
  _body_present?(env) && @options[:include_body]
45
55
  end
@@ -69,8 +79,9 @@ module Rack
69
79
  PWD: <%= Dir.getwd %>
70
80
 
71
81
  <%= env.to_a.
72
- sort{|a,b| a.first <=> b.first}.
73
- map{ |k,v| "%-25s%p" % [k+':', v] }.
82
+ reject { |key, value| _exclude_env_key?(env, key) }.
83
+ sort { |a, b| a.first <=> b.first }.
84
+ map{ |key, value| "%-25s%p" % [key + ':', value] }.
74
85
  join("\n ") %>
75
86
 
76
87
  <% if exception.respond_to?(:backtrace) %>
@@ -1,5 +1,5 @@
1
1
  module Rack
2
2
  class ExceptionNotifier
3
- VERSION = '0.3.1'
3
+ VERSION = '0.3.2'
4
4
  end
5
5
  end
@@ -65,6 +65,8 @@ describe Rack::ExceptionNotifier do
65
65
 
66
66
  mail = Mail::TestMailer.deliveries.first
67
67
  mail.body.raw_source.should_not include('Request Body')
68
+ mail.body.raw_source.should_not include('rack.input')
69
+ mail.body.raw_source.should_not include('rack.request.form_')
68
70
  end
69
71
 
70
72
  it 'includes the body if configured' do
metadata CHANGED
@@ -1,14 +1,14 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: rack-exception_notifier
3
3
  version: !ruby/object:Gem::Version
4
- version: 0.3.1
4
+ version: 0.3.2
5
5
  platform: ruby
6
6
  authors:
7
7
  - John Downey
8
8
  autorequire:
9
9
  bindir: bin
10
10
  cert_chain: []
11
- date: 2013-04-12 00:00:00.000000000 Z
11
+ date: 2013-05-09 00:00:00.000000000 Z
12
12
  dependencies:
13
13
  - !ruby/object:Gem::Dependency
14
14
  name: mail