rhcp 0.2.22 → 0.2.23

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.
Files changed (3) hide show
  1. data/lib/rhcp.rb +1 -1
  2. data/lib/rhcp/logging_broker.rb +5 -3
  3. metadata +23 -23
@@ -23,7 +23,7 @@ module RHCP #:nodoc:
23
23
 
24
24
  MAJOR = 0
25
25
  MINOR = 2
26
- TINY = 22
26
+ TINY = 23
27
27
 
28
28
  def Version.to_s
29
29
  [ MAJOR, MINOR, TINY ].join(".")
@@ -8,7 +8,7 @@ module RHCP
8
8
  super()
9
9
  @wrapped_broker = wrapped_broker
10
10
  @logger = RHCP::ModuleHelper::instance.logger
11
- @blacklisted_commands = [ "log_to_jabber", "log_to_jabber_detail", "on_host", "hostname", "ssh_options_for_machine" ]
11
+ @blacklisted_commands = [ ]
12
12
  end
13
13
 
14
14
  def get_command_list(context = RHCP::Context.new())
@@ -24,7 +24,7 @@ module RHCP
24
24
  end
25
25
 
26
26
  def blacklist_defaults
27
- [ "log_to_jabber", "log_to_jabber_detail", "on_host", "hostname", "ssh_options_for_machine" ]
27
+ [ ]
28
28
  end
29
29
 
30
30
  def get_blacklisted_commands
@@ -52,7 +52,9 @@ module RHCP
52
52
  command = get_command(request.command.name, request.context)
53
53
  mode = command.is_read_only ? 'r/o' : 'r/w'
54
54
 
55
- should_be_logged = (not blacklist.include?(command.name))
55
+ should_be_logged = (! blacklist.include?(command.name) && ! command.is_read_only)
56
+
57
+ #puts "#{command.name} [#{mode}] #{should_be_logged ? 'pass' : 'skip'}"
56
58
 
57
59
  if should_be_logged
58
60
  request_id = Thread.current[var_name("request_id")]
metadata CHANGED
@@ -1,7 +1,7 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: rhcp
3
3
  version: !ruby/object:Gem::Version
4
- version: 0.2.22
4
+ version: 0.2.23
5
5
  prerelease:
6
6
  platform: ruby
7
7
  authors:
@@ -9,7 +9,7 @@ authors:
9
9
  autorequire:
10
10
  bindir: bin
11
11
  cert_chain: []
12
- date: 2013-10-13 00:00:00.000000000 Z
12
+ date: 2013-11-09 00:00:00.000000000 Z
13
13
  dependencies:
14
14
  - !ruby/object:Gem::Dependency
15
15
  name: json
@@ -35,39 +35,39 @@ extensions: []
35
35
  extra_rdoc_files: []
36
36
  files:
37
37
  - bin/rhcp_test_server
38
+ - lib/rhcp/logging_broker.rb
39
+ - lib/rhcp/command.rb
40
+ - lib/rhcp/response.rb
38
41
  - lib/rhcp/dispatching_broker.rb
42
+ - lib/rhcp/rhcp_exception.rb
43
+ - lib/rhcp/command_param.rb
39
44
  - lib/rhcp/context.rb
40
- - lib/rhcp/encoding_helper.rb
41
- - lib/rhcp/client/context_aware_broker.rb
42
45
  - lib/rhcp/client/command_stub.rb
43
- - lib/rhcp/client/command_param_stub.rb
46
+ - lib/rhcp/client/context_aware_broker.rb
44
47
  - lib/rhcp/client/http_broker.rb
45
- - lib/rhcp/response.rb
46
- - lib/rhcp/rhcp_exception.rb
47
- - lib/rhcp/command.rb
48
- - lib/rhcp/request.rb
49
- - lib/rhcp/command_param.rb
50
- - lib/rhcp/broker.rb
51
- - lib/rhcp/logging_broker.rb
48
+ - lib/rhcp/client/command_param_stub.rb
49
+ - lib/rhcp/encoding_helper.rb
52
50
  - lib/rhcp/http_exporter.rb
51
+ - lib/rhcp/broker.rb
52
+ - lib/rhcp/request.rb
53
53
  - lib/rhcp.rb
54
- - test/rhcp/tests_for_writable_brokers.rb
55
- - test/rhcp/logging_broker_test.rb
56
- - test/rhcp/http_test_server.rb
57
- - test/rhcp/context_test.rb
54
+ - test/rhcp/http_exporter_test.rb
55
+ - test/rhcp/dispatching_broker_test.rb
56
+ - test/rhcp/response_test.rb
58
57
  - test/rhcp/request_test.rb
59
- - test/rhcp/broker_test.rb
60
- - test/rhcp/client/command_param_stub_test.rb
61
- - test/rhcp/client/command_stub_test.rb
62
58
  - test/rhcp/test_base.rb
59
+ - test/rhcp/http_test_server.rb
60
+ - test/rhcp/logging_broker_test.rb
63
61
  - test/rhcp/tests_for_brokers.rb
64
- - test/rhcp/dispatching_broker_test.rb
62
+ - test/rhcp/broker_test.rb
63
+ - test/rhcp/context_aware_broker_test.rb
65
64
  - test/rhcp/command_test.rb
65
+ - test/rhcp/client/command_param_stub_test.rb
66
+ - test/rhcp/client/command_stub_test.rb
67
+ - test/rhcp/context_test.rb
68
+ - test/rhcp/tests_for_writable_brokers.rb
66
69
  - test/rhcp/command_param_test.rb
67
- - test/rhcp/response_test.rb
68
- - test/rhcp/http_exporter_test.rb
69
70
  - test/rhcp/http_broker_test.rb
70
- - test/rhcp/context_aware_broker_test.rb
71
71
  homepage: http://rubyforge.org/projects/rhcp
72
72
  licenses: []
73
73
  post_install_message: