feat: add kid-based signature validation with enhanced error handling

.github/workflows/test.yml

@@ -27,5 +27,27 @@ jobs:
       - uses: actions/checkout@v4
         with:
           submodules: "true"
+
+      - name: Show Docker info
+        run: |
+          docker --version
+          docker compose version
+
       - name: Run Tests
         run: docker compose build --no-cache && docker compose up --exit-code-from tests tests
+
+      - name: Show container status
+        if: failure()
+        run: docker compose ps
+
+      - name: Show Kong logs
+        if: failure()
+        run: docker compose logs kong
+
+      - name: Show Keycloak logs
+        if: failure()
+        run: docker compose logs kc
+
+      - name: Show test logs
+        if: failure()
+        run: docker compose logs tests

docker-compose.yml

@@ -30,6 +30,8 @@ services:
     depends_on:
       - postgres
     entrypoint: [ "/bin/sh", "-c", "until kong migrations bootstrap; do echo waiting for database; sleep 2; done;" ]
+    networks:
+      - kong-net
   kong:
     <<: *kong-image
     depends_on:
@@ -39,6 +41,8 @@ services:
     ports:
       - "8000:8000"
       - "8001:8001"
+    networks:
+      - kong-net
   postgres:
     image: postgres:16
     environment:
@@ -47,6 +51,8 @@ services:
       POSTGRES_PASSWORD: kong
     volumes:
       - pg_data:/var/lib/postgresql/data
+    networks:
+      - kong-net
   kc-pg:
     image: postgres:16
     environment:
@@ -55,6 +61,8 @@ services:
       POSTGRES_PASSWORD: postgres
     volumes:
       - pg_kc_data:/var/lib/postgresql/data
+    networks:
+      - kong-net
   kc:
     image: quay.io/keycloak/keycloak:${KEYCLOAK_VERSION:-26.2}
     depends_on:
@@ -75,6 +83,8 @@ services:
       - KC_HOSTNAME_STRICT=false
       - KC_HTTP_RELATIVE_PATH=/auth
       - KC_HTTP_ENABLED=true
+    networks:
+      - kong-net
 
   tests:
     build:
@@ -91,12 +101,27 @@ services:
     environment:
       - LUA_PATH=/opt/kong-plugin-jwt-keycloak/src/?.lua;/opt/kong-plugin-jwt-keycloak/?.lua;;
       - LUA_CPATH=/usr/local/lib/lua/5.1/?.so;;
+      - HTTP_PROXY=
+      - HTTPS_PROXY=
+      - NO_PROXY="localhost,127.0.0.1"
+      - http_proxy=
+      - https_proxy=
+      - no_proxy="localhost,127.0.0.1"
+    networks:
+      - kong-net
     entrypoint: [ "/bin/sh", "-c", "cd /opt/tests && /bin/sh ./run_tests.sh" ]
 
   httpbin:
     image: mccutchen/go-httpbin:2.18.3
     ports:
       - "8080"
+    networks:
+      - kong-net
+
+networks:
+  kong-net:
+    driver: bridge
+
 volumes:
   pg_data:
   pg_kc_data:

kong-plugin-jwt-keycloak-1.6.0-1.rockspec

@@ -38,8 +38,9 @@ build = {
     ["kong.plugins."..plugin_name..".keycloak_keys"]      = "src/keycloak_keys.lua",
     ["kong.plugins."..plugin_name..".key_conversion"]     = "src/key_conversion.lua",
     ["kong.plugins."..plugin_name..".gateway.securitylog"] = "src/gateway/securitylog.lua",
-    ["kong.plugins."..plugin_name..".validators.issuers"] = "src/validators/issuers.lua",
-    ["kong.plugins."..plugin_name..".validators.roles"]   = "src/validators/roles.lua",
-    ["kong.plugins."..plugin_name..".validators.scope"]   = "src/validators/scope.lua",
+    ["kong.plugins."..plugin_name..".validators.issuers"]   = "src/validators/issuers.lua",
+    ["kong.plugins."..plugin_name..".validators.roles"]     = "src/validators/roles.lua",
+    ["kong.plugins."..plugin_name..".validators.scope"]     = "src/validators/scope.lua",
+    ["kong.plugins."..plugin_name..".validators.signature"] = "src/validators/signature.lua",
   }
 }

spec/01-unit/keycloak_keys_spec.lua

@@ -54,4 +54,24 @@ describe("Plugin: jwt-keycloak (keycloak_keys)", function()
       assert.is_function(keycloak_keys.get_request)
     end)
   end)
+
+  describe("get_issuer_keys", function()
+    it("should return keys and aligned kids from JWKS", function()
+      local well_known_endpoint = "https://keycloak.example.com/auth/realms/test/.well-known/openid-configuration"
+
+      local keys, kids, err, key_metadata = keycloak_keys.get_issuer_keys(well_known_endpoint)
+
+      assert.is_nil(err)
+      assert.is_table(keys)
+      assert.is_table(kids)
+      assert.is_table(key_metadata)
+      assert.equals(2, #keys)
+      assert.equals(2, #kids)
+      assert.equals(2, #key_metadata)
+      assert.same({ "kid1", "kid2" }, kids)
+      -- Verify metadata structure
+      assert.is_table(key_metadata[1])
+      assert.is_table(key_metadata[2])
+    end)
+  end)
 end)