warden_openid_bearer 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 968af30fb6b9b5daf5ec6f2c5a3ad26e776e2749a9f0fd5564458d436b98f244
-  data.tar.gz: b2fb133b65e2bba2c1484c0218de5ae83daf50cee85b02ed3bf4fb24402d1af1
+  metadata.gz: 3185e14bde01f3b44bae689e3942051973e8a38ee25eb382c549a87175a80597
+  data.tar.gz: 00d8d47d1f1656ac9dafd8257b818c80b32ca3df22d63895b50765cabc29d199
 SHA512:
-  metadata.gz: 1f1f9e1be38b4577968d609454c09fd9fff276918f993a02de13bcef716c8f21be3d8a657065b9e8b078c57c4a19d3e03803b4c32505b6942c6a3c77855146d2
-  data.tar.gz: 53febc3f4cfe6d6e2d6928d864c1232cf3571fb72ba4ad8a44b803570cbd46b11666165080a592a573e55ea4d8a5ffd618781254808541d80210009a1c7882ed
+  metadata.gz: 15407ea9f324dca19089df0862d150a303fe695313c3e1aad83dc023c824a6bcdbaa367e182c4f86014d8199d6341e9d62ec75b362e87c2a2e9ab41b742512a0
+  data.tar.gz: 80efe7bdd1fb90077dd74718144b5c014b165b04395ae3d8e4e54894e49436a211c844e74319cf4342b192cc1e6c024835ed4994147f1ef96d9a191389820942

data/CHANGELOG.md

@@ -1,7 +1,9 @@
-## [0.2.0] - 2023-11-02
+## [0.2.1] - 2023-11-02
 - Rewritten to *not* depend on the auth token being JWT (an assumption which only works with Keycloak)
 - Support user-configured (bogus) certificate for development
 
+N.B.: 0.2.0 only differs from 0.2.1 by the fact that `rake standard:fix` ran inbetweeen both.
+
 ## [0.1.4] - 2022-10-11
 - Clean up a stray `puts` left when debugging
 

data/lib/warden_openid_bearer/cache_mixin.rb

@@ -7,21 +7,21 @@ module WardenOpenidBearer
     def cached_by(*keys, &do_it)
       @__cache_mixin__cache ||= {}
 
-      caller_method = caller[0][/`.*'/][1..-2]
+      caller_method = caller(1..1).first[/`.*'/][1..-2]
       keys.unshift(caller_method)
 
       first_keys = keys.slice!(0, keys.length - 1).join("|")
       last_key = keys[0]
 
       last_key_is_value_type = last_key.is_a? String
-      cache = if last_key_is_value_type
-                @__cache_mixin__cache[first_keys] ||= {}
+      cache = @__cache_mixin__cache[first_keys] ||= if last_key_is_value_type
+                {}
               else
                 # Use the ::ObjectSpace::WeakMap private API, because the
                 # endeavor of reinventing weak maps on top of (public)
                 # WeakRef's would be called an inversion of abstraction and
                 # would be considered harmful. Sue me (I have unit tests).
-                @__cache_mixin__cache[first_keys] ||= ::ObjectSpace::WeakMap.new
+                ::ObjectSpace::WeakMap.new
               end
 
       now = Time.now()
@@ -33,7 +33,7 @@ module WardenOpenidBearer
       end
 
       retval = do_it.call
-      cache[last_key] = { payload: retval, fetched_at: now }
+      cache[last_key] = {payload: retval, fetched_at: now}
       retval
     end
   end

data/lib/warden_openid_bearer/discovered_config.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'net/http'
-require 'warden_openid_bearer/net_https'
+require "net/http"
+require "warden_openid_bearer/net_https"
 
 module WardenOpenidBearer
   # Cacheable configuration (periodically re-)fetched starting from
@@ -24,9 +24,7 @@ module WardenOpenidBearer
       metadata[:userinfo_endpoint]
     end
 
-    def peer_cert=(peer_cert)
-      @peer_cert = peer_cert
-    end
+    attr_writer :peer_cert
 
     private
 
@@ -36,10 +34,10 @@ module WardenOpenidBearer
 
     def json(uri)
       cached_by(uri) do
-        if uri.scheme == 'https'
-          response = WardenOpenidBearer::NetHTTPS.get_response(URI(uri), @peer_cert)
+        response = if uri.scheme == "https"
+          WardenOpenidBearer::NetHTTPS.get_response(URI(uri), @peer_cert)
         else
-          response = Net::HTTP.get_response(URI(uri))
+          Net::HTTP.get_response(URI(uri))
         end
         JSON.parse(response.body, symbolize_names: true)
       end

data/lib/warden_openid_bearer/net_https.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'net/http'
+require "net/http"
 
 module WardenOpenidBearer
   # Like Net::HTTP, but with TLS and VERIFY_PEER always on.
@@ -22,7 +22,7 @@ module WardenOpenidBearer
     end
 
     def self.get_response(uri, peer_cert = nil)
-      https = self.new(uri.hostname, uri.port)
+      https = new(uri.hostname, uri.port)
       https.peer_cert = peer_cert if peer_cert
 
       req = Net::HTTP::Get.new(uri)

data/lib/warden_openid_bearer/strategy.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'uri'
-require 'net/http'
-require 'warden_openid_bearer/net_https'
+require "uri"
+require "net/http"
+require "warden_openid_bearer/net_https"
 
 module WardenOpenidBearer
   # Like `WardenOpenidAuth::Strategy` in
@@ -119,9 +119,9 @@ module WardenOpenidBearer
     def _do_oauth2_userinfo
       uri = URI.parse(config.userinfo_endpoint)
       req = Net::HTTP::Get.new(uri)
-      req['Authorization'] = "Bearer #{token}"
+      req["Authorization"] = "Bearer #{token}"
 
-      if uri.scheme == 'https'
+      if uri.scheme == "https"
         http = WardenOpenidBearer::NetHTTPS.new(uri.hostname, uri.port)
         if (peer_cert = WardenOpenidBearer.config.openid_server_certificate)
           http.peer_cert = peer_cert

data/lib/warden_openid_bearer/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module WardenOpenidBearer
-  VERSION = "0.2.0"
+  VERSION = "0.2.1"
 end

data/lib/warden_openid_bearer.rb

@@ -14,6 +14,6 @@ module WardenOpenidBearer
   extend Dry::Configurable
 
   setting :openid_metadata_url, constructor: ->(url) { URI(url) }
-  setting :openid_server_certificate, default: nil, constructor: ->(pem) { if pem; OpenSSL::X509::Certificate.new(pem); else nil; end }
+  setting :openid_server_certificate, default: nil, constructor: ->(pem) { pem ? OpenSSL::X509::Certificate.new(pem) : nil }
   setting :cache_timeout, default: 900
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: warden_openid_bearer
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Dominique Quatravaux