omniauth-idcat_mobil 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c8300ff0763d3f698d9554aaea9af39888e587e27ebe5a0932833dccf7540da0
-  data.tar.gz: 17ec09c05d66d34b82ba663b1947d0db218b91bacc47894278eef36bf9b8cf8c
+  metadata.gz: 37d52ae58802955d43e15d2492906f84dba0b4a9f6a042436fad68ebc1093f33
+  data.tar.gz: 131dc83ac5dc43b4ee47d66ff3213341394ae01469273ef6adb8580e72be6129
 SHA512:
-  metadata.gz: a96b0fa966251a2ae1e8247e49e811584f6010dce67f7f9931b35d08e9beb31c08ac47c27ee80975e12acc931d6e4e6d3d526fff946c772c81517637c2592743
-  data.tar.gz: a3c684b87be83c364752e7c528e848259003f5a9cd1ab9edaf7e5c93d805e6ef15f78c5a366a8d60b5687eb28c8d2bea16fc65aa8ed7012a1afa7e6d259e8a08
+  metadata.gz: 129387454f49df2062c14dce03ba6a5720b1be7fa11697f28026096d38da62ebcbbc62ff4953c5833cc7269a398dcb9bf8ef0a99a152bb4d61c4b0a7030d5707
+  data.tar.gz: b6dea50779fdd2b3835e6944b0dee4b58bccb9907d036fa6e3d1d9189c77f402525057399acb6085988abff36145dccf971adca0144be0801bb8729095e51268

data/.ruby-version

@@ -1 +1 @@
-2.7.3
+2.7.5

data/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## next version:
 
+## Version 0.2.2 (PATCH)
+- Fix internal `log` method is wrongly invoked from `omniauth`.
+- Bump Ruby version to 2.7.5
+
 ## Version 0.2.1 (PATCH)
 - Apply security upgrades
 - Add a .ruby-version file

data/lib/omniauth/idcat_mobil/version.rb

@@ -1,5 +1,5 @@
 module Omniauth
   module IdCatMobil
-    VERSION = "0.2.1"
+    VERSION = "0.2.2"
   end
 end

data/lib/omniauth/strategies/idcat_mobil.rb

@@ -77,8 +77,8 @@ module OmniAuth
       # 
       # We're overriding solely to log.
       def request_phase
-        log("In `request_phase`, with params: redirect_uri=>#{callback_url}, options=>#{options.authorize_params}")
-        log("`request_phase`, redirecting the user to AOC...")
+        idcat_log("In `request_phase`, with params: redirect_uri=>#{callback_url}, options=>#{options.authorize_params}")
+        idcat_log("`request_phase`, redirecting the user to AOC...")
         super
       end
 
@@ -88,18 +88,18 @@ module OmniAuth
       # In case of success we still have to ask the authentication provider for the access_token.
       # That's what we do in this callback.
       def callback_phase
-        log("In `callback_phase` with request params: #{request.params}")
-        log("Both should be equal otherwise a 'CSRF detected' error is raised: params state[#{request.params["state"]}] =? [#{session.delete("omniauth.state")}] session state.")
+        idcat_log("In `callback_phase` with request params: #{request.params}")
+        idcat_log("Both should be equal otherwise a 'CSRF detected' error is raised: params state[#{request.params["state"]}] =? [#{session.delete("omniauth.state")}] session state.")
         super
       end
 
       def raw_info
-        log("Access token response was: #{access_token.response}")
-        log("Performing getUserInfo...")
+        idcat_log("Access token response was: #{access_token.response}")
+        idcat_log("Performing getUserInfo...")
         unless @raw_info
           response= access_token.get(options.user_info_path)
           result= %i(status headers body).collect  {|m| response.send(m)}
-          log("getUserInfo response status/headers/body: #{result}")
+          idcat_log("getUserInfo response status/headers/body: #{result}")
           @raw_info= response.parsed
           # Logout to avoid problems with IdCat mòbil's cookie session when trying to login again.
           logout_url= URI.join(options.site, "/o/oauth2/logout?token=#{access_token.token}").to_s
@@ -114,12 +114,16 @@ module OmniAuth
         full_host + script_name + callback_path
       end
 
-      def log(msg)
-        logger.debug(msg)
+      # --------------------------------------------------
+      private
+      # --------------------------------------------------
+
+      def idcat_log(msg)
+        idcat_logger.debug(msg)
       end
 
-      def logger
-        @logger||= defined?(Rails.logger) ? Rails.logger : Logger.new(STDOUT, progname: 'idcat_mobil')
+      def idcat_logger
+        @idcat_logger||= defined?(Rails.logger) ? Rails.logger : Logger.new(STDOUT, progname: 'idcat_mobil')
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-idcat_mobil
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Oliver Valls
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-06-21 00:00:00.000000000 Z
+date: 2022-01-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth