Turn off verbose_logging by default and updated readme

README.md

@@ -161,10 +161,9 @@ You could replace our logger if you want different output:
 Rails.application.config.twirp.logger = Rack::CommonLogger
 ```
 
-Additionally, you can log the Twirp response object to help with debugging: 
+Additionally, you can log the full Twirp response object to help with debugging: 
 
 ```ruby
-# Defaults to true if your log_level is :debug
 Rails.application.config.twirp.verbose_logging = true
 ```
 

lib/twirp/rails/configuration.rb

@@ -12,7 +12,8 @@ class Configuration
       # Logger to use for Twirp requests. Defaults to Rails.logger
       attr_accessor :logger
 
-      # Whether to log Twirp responses. Defaults to true if log level is :debug
+      # Whether to log full Twirp responses. Can be useful for debugging, but can expose sensitive data.
+      # Defauts to false
       # Example:
       #   Twirp Response: <Twirp::Example::Haberdasher::Hat: inches: 24, color: "Tan", name: "Pork Pie">
       attr_accessor :verbose_logging
@@ -34,7 +35,7 @@ def initialize
         @endpoint = "/twirp"
         @load_paths = ["lib"]
         @logger = Logger
-        @verbose_logging = nil
+        @verbose_logging = false
         @middleware = []
         @service_hooks = {}
       end

lib/twirp/rails/engine.rb

@@ -28,8 +28,6 @@ class Engine < ::Rails::Engine
 
         # Set up logging
         app.config.middleware.use app.config.twirp.logger, ::Rails.logger
-        app.config.twirp.verbose_logging = ::Rails.logger.level == ::Logger::DEBUG if app.config.twirp.verbose_logging.nil?
-
         app.config.twirp.middleware.each do |middleware|
           app.config.middleware.use middleware
         end

spec/rails_app/config/application.rb

@@ -8,5 +8,7 @@
 module RailsApp
   class Application < Rails::Application
     config.root = File.expand_path("../../", __FILE__)
+
+    config.twirp.verbose_logging = true
   end
 end