🐛 Fix URINorm for Ruby 3+

Author: pboling

.github/workflows/supported.yml

@@ -0,0 +1,57 @@
+name: Supported Ruby Matrix
+
+env:
+  K_SOUP_COV_DO: false
+
+on:
+  push:
+    branches:
+      - 'master'
+    tags:
+      - '!*' # Do not execute on tags
+  pull_request:
+    branches:
+      - '*'
+  # Allow manually triggering the workflow.
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+# Cancels all previous workflow runs for the same branch that have not yet completed.
+concurrency:
+  # The concurrency group contains the workflow name and the branch name.
+  group: "${{ github.workflow }}-${{ github.ref }}"
+  cancel-in-progress: true
+
+jobs:
+  test:
+    name: Specs - Ruby ${{ matrix.ruby }}${{ matrix.name_extra || '' }}
+    if: "!contains(github.event.commits[0].message, '[ci skip]') && !contains(github.event.commits[0].message, '[skip ci]')"
+    env: # $BUNDLE_GEMFILE must be set at the job level, so it is set for all steps
+      BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        include:
+          - ruby: "3.3"
+            rubygems: latest
+            bundler: latest
+            gemfile: vanilla
+          - ruby: "3.2"
+            rubygems: latest
+            bundler: latest
+            gemfile: vanilla
+          #- Ruby 3.1 tests are run by coverage.yml
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Setup Ruby & RubyGems
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: "${{ matrix.ruby }}"
+          rubygems: "${{ matrix.rubygems }}"
+          bundler: "${{ matrix.bundler }}"
+          bundler-cache: true # runs 'bundle install' and caches installed gems automatically
+      - name: Run tests
+        run: bundle exec rake test

lib/openid/urinorm.rb

@@ -3,17 +3,20 @@
 module OpenID
 
   module URINorm
+    VALID_URI_SCHEMES = ['http','https'].freeze
     public
     def URINorm.urinorm(uri)
       uri = URI.parse(uri)
 
       raise URI::InvalidURIError.new('no scheme') unless uri.scheme
+
       uri.scheme = uri.scheme.downcase
-      unless ['http','https'].member?(uri.scheme)
-        raise URI::InvalidURIError.new('Not an HTTP or HTTPS URI')
-      end
+      raise URI::InvalidURIError.new('Not an HTTP or HTTPS URI') unless VALID_URI_SCHEMES.member?(uri.scheme)
+
+      raise URI::InvalidURIError.new('no host') if uri.host.nil? # For Ruby 2.7
+
+      raise URI::InvalidURIError.new('no host') if uri.host.empty? # For Ruby 3+
 
-      raise URI::InvalidURIError.new('no host') unless uri.host
       uri.host = uri.host.downcase
 
       uri.path = remove_dot_segments(uri.path)