Do not log errors by default

Author: lalunamel

README.md

@@ -148,7 +148,7 @@ or when configuring the gem:
 ```ruby
 OpenAI.configure do |config|
     config.access_token = ENV.fetch("OPENAI_ACCESS_TOKEN")
-    config.log_errors = false # Optional
+    config.log_errors = true # Optional
     config.organization_id = ENV.fetch("OPENAI_ORGANIZATION_ID") # Optional
     config.uri_base = "https://oai.hconeai.com/" # Optional
     config.request_timeout = 240 # Optional
@@ -173,12 +173,12 @@ client.add_headers("X-Proxy-TTL" => "43200")
 
 ##### Errors
 
-By default, `ruby-openai` logs any `Faraday::Error`s encountered while executing a network request (e.g. 400s, 500s, SSL errors and more - see [here](https://www.rubydoc.info/github/lostisland/faraday/Faraday/Error) for a complete list of subclasses of `Faraday::Error` and what can cause them).
+By default, `ruby-openai` does not log any `Faraday::Error`s encountered while executing a network request to avoid leaking data (e.g. 400s, 500s, SSL errors and more - see [here](https://www.rubydoc.info/github/lostisland/faraday/Faraday/Error) for a complete list of subclasses of `Faraday::Error` and what can cause them).
 
-If you would like to disable this functionality, you can set `log_errors` to `false` when configuring the client:
+If you would like to enable this functionality, you can set `log_errors` to `true` when configuring the client:
 
 ```ruby
-  client = OpenAI::Client.new(log_errors: false)
+  client = OpenAI::Client.new(log_errors: true)
 ```
 
 ##### Faraday middleware

lib/openai.rb

@@ -49,7 +49,7 @@ class Configuration
     DEFAULT_API_VERSION = "v1".freeze
     DEFAULT_URI_BASE = "https://api.openai.com/".freeze
     DEFAULT_REQUEST_TIMEOUT = 120
-    DEFAULT_LOG_ERRORS = true
+    DEFAULT_LOG_ERRORS = false
 
     def initialize
       @access_token = nil

spec/openai/client/http_spec.rb

@@ -284,14 +284,14 @@
       $stdout = @original_stdout
     end
 
-    it "happens by default" do
+    it "is disabled by default" do
       VCR.use_cassette(cassette, record: :none) do
         expect { OpenAI::Client.new.models.retrieve(id: "text-ada-001") }
           .to raise_error Faraday::Error
 
         $stdout.rewind
         captured_stdout = $stdout.string
-        expect(captured_stdout).to include("OpenAI HTTP Error")
+        expect(captured_stdout).not_to include("OpenAI HTTP Error")
       end
     end