omniauth-cas 3.0.0 → 3.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fd3e863e924ef181ec4a6bfdb1c64187258b78ffc8bd8857df7017607dc60a47
-  data.tar.gz: 2dda8861195390b89ebaf60f74bd9e1985a44410abd91e7cc1a4608cd8cbaed9
+  metadata.gz: c081ab74fc1dca73df7c4a403c77714e378467d304f0bc461d63455944eee813
+  data.tar.gz: 13d48e3204d30d85b0d77a06cb0c9a4ae5e6a101075beef8f8cf9c781098305d
 SHA512:
-  metadata.gz: 181b83703819f127f099eb14455a4160f71810f04fc1ce3886a87968efe801cb951c49613726b50f0408f7f4ce69a42d56f7d3be0a42f6b61cbdfaefcf76a015
-  data.tar.gz: 45f2a3b3b3a3827c3fb57c2e537a3b21adf6c04f8e3d5b33cf981400602407c9d6b1c33f89145210c1b700f300d6cb0a0dc54fb8f06bcc3ac8fa614d8be3af1f
+  metadata.gz: ca58926494b20100c4fa7400fd2bf173a0bb6e932875d80d7b9e5c660fc6a7848cf965294a6e3e7eabec1504e667070400089b70157d5ad6a43818de7dd91cda
+  data.tar.gz: 4658cf8858bbc9571cf2497c93c413f6c86512ad1c2149d9bdd02c38bf384a39198274d408065f8d9b9cbbf67da74f1608638908c2514b7607e61d0bb239a594

data/CHANGELOG.md

@@ -5,6 +5,15 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/) and this
 project adheres to [Semantic Versioning](https://semver.org/)
 
+## 3.0.2 - 2025-07-26
+
+* Minor refactor to service ticket validator (#96).
+
+## 3.0.1 - 2025-02-28
+
+* Prefer `require_relative` over `require`
+* Remove redundant require of `net/https`
+
 ## 3.0.0 - 2024-02-24
 
 ### Changed

data/README.md

@@ -98,7 +98,7 @@ provider :cas,
          service_validate_url: '/cas/serviceValidate'
 ```
 
-If you encounter problems wih SSL certificates you may want to set the `ca_path` parameter or activate `disable_ssl_verification` (not recommended).
+If you encounter problems with SSL certificates you may want to set the `ca_path` parameter or activate `disable_ssl_verification` (not recommended).
 
 ## Contributing
 

data/lib/omniauth/cas/version.rb

@@ -2,6 +2,6 @@
 
 module OmniAuth
   module Cas
-    VERSION = '3.0.0'
+    VERSION = '3.0.2'
   end
 end

data/lib/omniauth/cas.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 
-require 'omniauth/cas/version'
-require 'omniauth/strategies/cas'
+require_relative 'cas/version'
+require_relative 'strategies/cas'

data/lib/omniauth/strategies/cas/service_ticket_validator.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require 'net/http'
-require 'net/https'
 require 'nokogiri'
 
 module OmniAuth
@@ -59,23 +58,22 @@ module OmniAuth
         def parse_user_info(node)
           return nil if node.nil?
 
-          {}.tap do |hash|
-            node.children.each do |e|
-              node_name = e.name.sub(/^cas:/, '')
-              unless e.is_a?(Nokogiri::XML::Text) || node_name == 'proxies'
-                # There are no child elements
-                if e.element_children.count == 0
-                  hash[node_name] = attribute_value(hash, node_name, e.content)
-                elsif e.element_children.count
-                  # JASIG style extra attributes
-                  if node_name == 'attributes'
-                    hash.merge!(parse_user_info(e))
-                  else
-                    hash[node_name] = [] if hash[node_name].nil?
-                    hash[node_name].push(parse_user_info(e))
-                  end
+          node.children.each_with_object({}) do |e, hash|
+            node_name = e.name.sub(/^cas:/, '')
+            unless e.is_a?(Nokogiri::XML::Text) || node_name == 'proxies'
+              # rubocop:disable Style/IfInsideElse -- preserve `else` branch for clarity
+              if e.element_children.empty?
+                hash[node_name] = attribute_value(hash, node_name, e.content)
+              else
+                # JASIG style extra attributes
+                if node_name == 'attributes'
+                  hash.merge!(parse_user_info(e))
+                else
+                  hash[node_name] = [] if hash[node_name].nil?
+                  hash[node_name].push(parse_user_info(e))
                 end
               end
+              # rubocop:enable Style/IfInsideElse
             end
           end
         end

data/lib/omniauth/strategies/cas.rb

@@ -152,7 +152,7 @@ module OmniAuth
       # +service+ and add it and the +ticket+ as paraemters to the
       # CAS serviceValidate URL.
       #
-      # @param [String] service the service (a.k.a. return-to) URL
+      # @param [String] service_url the service (a.k.a. return-to) URL
       # @param [String] ticket the ticket to validate
       #
       # @return [String] a URL like `http://cas.mycompany.com/serviceValidate?service=...&ticket=...`

data/lib/omniauth-cas.rb

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-require 'omniauth/cas'
+require_relative 'omniauth/cas'

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-cas
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.0.2
 platform: ruby
 authors:
 - Derek Lindahl
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-24 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -78,7 +77,6 @@ metadata:
   rubygems_mfa_required: 'true'
   source_code_uri: https://github.com/dlindahl/omniauth-cas
   wiki_uri: https://github.com/dlindahl/omniauth-cas/wiki
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -93,8 +91,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.5
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: CAS Strategy for OmniAuth
 test_files: []