✨ OAuth2::OAUTH_DEBUG

Author: pboling

.rubocop_gradual.lock

@@ -3,9 +3,9 @@
     [66, 5, 20, "ThreadSafety/ClassInstanceVariable: Avoid class instance variables.", 2485198147],
     [78, 5, 74, "Style/InvertibleUnlessCondition: Prefer `if Gem.rubygems_version >= Gem::Version.new(\"2.7.0\")` over `unless Gem.rubygems_version < Gem::Version.new(\"2.7.0\")`.", 2453573257]
   ],
-  "lib/oauth2.rb:3930909031": [
-    [31, 5, 21, "ThreadSafety/ClassAndModuleAttributes: Avoid mutating class and module attributes.", 622027168],
-    [34, 11, 7, "ThreadSafety/ClassInstanceVariable: Avoid class instance variables.", 651502127]
+  "lib/oauth2.rb:649414550": [
+    [34, 5, 21, "ThreadSafety/ClassAndModuleAttributes: Avoid mutating class and module attributes.", 622027168],
+    [37, 11, 7, "ThreadSafety/ClassInstanceVariable: Avoid class instance variables.", 651502127]
   ],
   "lib/oauth2/authenticator.rb:3711266135": [
     [42, 5, 113, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 734523108]
@@ -41,26 +41,26 @@
     [69, 15, 38, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 1480816240],
     [79, 13, 23, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 2314399065]
   ],
-  "spec/oauth2/client_spec.rb:824695973": [
+  "spec/oauth2/client_spec.rb:2242285093": [
     [6, 1, 29, "RSpec/SpecFilePathFormat: Spec path should end with `o_auth2/client*_spec.rb`.", 439549885],
     [174, 7, 492, "RSpec/NoExpectationExample: No expectation found in this example.", 1272021224],
     [193, 7, 592, "RSpec/NoExpectationExample: No expectation found in this example.", 3428877205],
     [206, 15, 20, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 2320605227],
     [221, 15, 20, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 1276531672],
     [236, 15, 43, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 1383956904],
     [251, 15, 43, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 3376202107],
-    [869, 5, 360, "RSpec/NoExpectationExample: No expectation found in this example.", 536201463],
-    [878, 5, 461, "RSpec/NoExpectationExample: No expectation found in this example.", 3392600621],
-    [889, 5, 340, "RSpec/NoExpectationExample: No expectation found in this example.", 244592251],
-    [934, 63, 2, "RSpec/BeEq: Prefer `be` over `eq`.", 5860785],
-    [979, 11, 99, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 3084776886],
-    [983, 11, 82, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 1524553529],
-    [991, 7, 89, "RSpec/NoExpectationExample: No expectation found in this example.", 4609419],
-    [1079, 11, 99, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 3084776886],
-    [1083, 11, 82, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 1524553529],
-    [1163, 17, 12, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 664794325],
-    [1188, 5, 459, "RSpec/NoExpectationExample: No expectation found in this example.", 2216851076],
-    [1198, 7, 450, "RSpec/NoExpectationExample: No expectation found in this example.", 2619808549]
+    [867, 5, 360, "RSpec/NoExpectationExample: No expectation found in this example.", 536201463],
+    [876, 5, 461, "RSpec/NoExpectationExample: No expectation found in this example.", 3392600621],
+    [887, 5, 340, "RSpec/NoExpectationExample: No expectation found in this example.", 244592251],
+    [932, 63, 2, "RSpec/BeEq: Prefer `be` over `eq`.", 5860785],
+    [977, 11, 99, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 3084776886],
+    [981, 11, 82, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 1524553529],
+    [989, 7, 89, "RSpec/NoExpectationExample: No expectation found in this example.", 4609419],
+    [1077, 11, 99, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 3084776886],
+    [1081, 11, 82, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 1524553529],
+    [1161, 17, 12, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 664794325],
+    [1186, 5, 459, "RSpec/NoExpectationExample: No expectation found in this example.", 2216851076],
+    [1196, 7, 450, "RSpec/NoExpectationExample: No expectation found in this example.", 2619808549]
   ],
   "spec/oauth2/error_spec.rb:1209122273": [
     [23, 1, 28, "RSpec/SpecFilePathFormat: Spec path should end with `o_auth2/error*_spec.rb`.", 3385870076],
@@ -97,7 +97,7 @@
   "spec/oauth2/version_spec.rb:1001406821": [
     [3, 1, 30, "RSpec/SpecFilePathFormat: Spec path should end with `o_auth2/version*_spec.rb`.", 1099517182]
   ],
-  "spec/oauth2_spec.rb:1511642301": [
+  "spec/oauth2_spec.rb:438638294": [
     [3, 1, 21, "RSpec/SpecFilePathFormat: Spec path should end with `o_auth2*_spec.rb`.", 3359091140],
     [5, 68, 2, "RSpec/BeEq: Prefer `be` over `eq`.", 5860785]
   ]

lib/oauth2.rb

@@ -25,7 +25,10 @@
 
 # The namespace of this library
 module OAuth2
-  DEFAULT_CONFIG = SnakyHash::SymbolKeyed.new(silence_extra_tokens_warning: false)
+  OAUTH_DEBUG = ENV.fetch("OAUTH_DEBUG", "false").casecmp("true").zero?
+  DEFAULT_CONFIG = SnakyHash::SymbolKeyed.new(
+    silence_extra_tokens_warning: !OAUTH_DEBUG,
+  )
   @config = DEFAULT_CONFIG.dup
   class << self
     attr_accessor :config

lib/oauth2/client.rb

@@ -449,7 +449,7 @@ def build_access_token_legacy(response, access_token_opts, extract_access_token)
     end
 
     def oauth_debug_logging(builder)
-      builder.response(:logger, options[:logger], bodies: true) if ENV["OAUTH_DEBUG"] == "true"
+      builder.response(:logger, options[:logger], bodies: true) if OAUTH_DEBUG
     end
   end
 end

spec/oauth2/client_spec.rb

@@ -267,13 +267,12 @@
 
   describe "#connection" do
     context "when debugging" do
-      include_context "with stubbed env"
       before do
-        stub_env("OAUTH_DEBUG" => debug_value)
+        stub_const("OAuth2::OAUTH_DEBUG", debug_value)
       end
 
       context "when OAUTH_DEBUG=true" do
-        let(:debug_value) { "true" }
+        let(:debug_value) { true }
 
         it "smoothly handles successive requests" do
           silence_all do
@@ -301,7 +300,7 @@
       end
 
       context "when OAUTH_DEBUG=false" do
-        let(:debug_value) { "false" }
+        let(:debug_value) { false }
 
         it "smoothly handles successive requests" do
           silence_all do
@@ -358,10 +357,9 @@
     end
 
     context "with ENV" do
-      include_context "with stubbed env"
       context "when OAUTH_DEBUG=true" do
         before do
-          stub_env("OAUTH_DEBUG" => "true")
+          stub_const("OAuth2::OAUTH_DEBUG", true)
         end
 
         it "outputs to $stdout when OAUTH_DEBUG=true" do

spec/oauth2_spec.rb

@@ -2,7 +2,7 @@
 
 RSpec.describe OAuth2 do
   it "has a default config for silence_extra_tokens_warning" do
-    expect(described_class.config.silence_extra_tokens_warning).to eq(false)
+    expect(described_class.config.silence_extra_tokens_warning).to eq(true)
   end
 
   describe ".configure" do