Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
728e737dc8 |
@@ -10,7 +10,6 @@ POSTGRES_PASSWORD=change-me-postgres-password
|
|||||||
TOKEN_MEDIATING_CLIENT_SECRET=change-me-token-mediating-client-secret
|
TOKEN_MEDIATING_CLIENT_SECRET=change-me-token-mediating-client-secret
|
||||||
BFF_CLIENT_SECRET=change-me-bff-client-secret
|
BFF_CLIENT_SECRET=change-me-bff-client-secret
|
||||||
EDGE_PROXY_CLIENT_SECRET=change-me-edge-proxy-client-secret
|
EDGE_PROXY_CLIENT_SECRET=change-me-edge-proxy-client-secret
|
||||||
OAUTH2_PROXY_COOKIE_SECRET=generate-a-base64-encoded-32-byte-secret
|
|
||||||
ADMIN_USER_PASSWORD=change-me-admin-user-password
|
ADMIN_USER_PASSWORD=change-me-admin-user-password
|
||||||
REGULAR_USER_PASSWORD=change-me-regular-user-password
|
REGULAR_USER_PASSWORD=change-me-regular-user-password
|
||||||
|
|
||||||
|
|||||||
+3
-2
@@ -4,6 +4,7 @@
|
|||||||
*.iml
|
*.iml
|
||||||
|
|
||||||
backend/target/
|
backend/target/
|
||||||
**/node_modules/
|
|
||||||
frontend/dist/
|
|
||||||
build/
|
build/
|
||||||
|
e2e/node_modules/
|
||||||
|
frontend/node_modules/
|
||||||
|
frontend/dist/
|
||||||
|
|||||||
@@ -1,5 +1,8 @@
|
|||||||
# Keycloak Authentication Patterns
|
# Keycloak Authentication Patterns
|
||||||
|
|
||||||
|
The 39-branch implementation registry is documented in
|
||||||
|
[`docs/keycloak-branch-index.md`](docs/keycloak-branch-index.md).
|
||||||
|
|
||||||
Keycloak을 중심으로 네 가지 브라우저 인증 통합 패턴을 같은 로컬
|
Keycloak을 중심으로 네 가지 브라우저 인증 통합 패턴을 같은 로컬
|
||||||
인프라에서 비교하는 학습 프로젝트입니다.
|
인프라에서 비교하는 학습 프로젝트입니다.
|
||||||
|
|
||||||
@@ -96,19 +99,3 @@ Keycloak을 잠시 중지하고 export한 뒤 자동으로 다시 올립니다.
|
|||||||
|
|
||||||
runtime export에는 실제 client secret과 credential hash가 포함될 수 있어
|
runtime export에는 실제 client secret과 credential hash가 포함될 수 있어
|
||||||
gitignored `build/keycloak-export/`에 권한 `0600`으로만 저장됩니다.
|
gitignored `build/keycloak-export/`에 권한 `0600`으로만 저장됩니다.
|
||||||
|
|
||||||
## AP4: oauth2-proxy Edge Forward Auth
|
|
||||||
|
|
||||||
`develop-keycloak-pattern4`는 oauth2-proxy와 Nginx `auth_request`가
|
|
||||||
인증을 edge에서 강제하는 패턴입니다.
|
|
||||||
|
|
||||||
```bash
|
|
||||||
./scripts/verify-pattern4.sh
|
|
||||||
```
|
|
||||||
|
|
||||||
첫 feature에서는 oauth2-proxy를 `http://localhost:4180`에 직접 노출해
|
|
||||||
OIDC redirect/PKCE/callback과 forwarded-user를 분리 확인합니다. 최종
|
|
||||||
구성은 `http://localhost:8088` Nginx를 단일 진입점으로 사용합니다.
|
|
||||||
자세한 내용은
|
|
||||||
[`docs/ap4-edge-forward-auth.md`](docs/ap4-edge-forward-auth.md)를
|
|
||||||
참고하세요.
|
|
||||||
|
|||||||
@@ -1,47 +0,0 @@
|
|||||||
package com.example.keycloakpattern;
|
|
||||||
|
|
||||||
import java.util.LinkedHashMap;
|
|
||||||
import java.util.Map;
|
|
||||||
|
|
||||||
import jakarta.servlet.http.HttpServletRequest;
|
|
||||||
|
|
||||||
import org.springframework.http.ResponseEntity;
|
|
||||||
import org.springframework.web.bind.annotation.GetMapping;
|
|
||||||
import org.springframework.web.bind.annotation.RestController;
|
|
||||||
|
|
||||||
@RestController
|
|
||||||
public class EdgeIdentityController {
|
|
||||||
|
|
||||||
@GetMapping("/edge/me")
|
|
||||||
ResponseEntity<Map<String, Object>> currentUser(HttpServletRequest request) {
|
|
||||||
String authRequestUser = request.getHeader("X-Auth-Request-User");
|
|
||||||
String forwardedUser = request.getHeader("X-Forwarded-User");
|
|
||||||
String user = hasText(authRequestUser) ? authRequestUser : forwardedUser;
|
|
||||||
if (!hasText(user)) {
|
|
||||||
return ResponseEntity.status(401).body(Map.of(
|
|
||||||
"error",
|
|
||||||
"trusted edge identity header is required"
|
|
||||||
));
|
|
||||||
}
|
|
||||||
|
|
||||||
Map<String, Object> response = new LinkedHashMap<>();
|
|
||||||
response.put("pattern", "AP4-edge-forward-auth");
|
|
||||||
response.put("user", user);
|
|
||||||
response.put("email", firstNonBlank(
|
|
||||||
request.getHeader("X-Auth-Request-Email"),
|
|
||||||
request.getHeader("X-Forwarded-Email")
|
|
||||||
));
|
|
||||||
response.put("identityHeader", hasText(authRequestUser)
|
|
||||||
? "X-Auth-Request-User"
|
|
||||||
: "X-Forwarded-User");
|
|
||||||
return ResponseEntity.ok(response);
|
|
||||||
}
|
|
||||||
|
|
||||||
private static String firstNonBlank(String first, String second) {
|
|
||||||
return hasText(first) ? first : second;
|
|
||||||
}
|
|
||||||
|
|
||||||
private static boolean hasText(String value) {
|
|
||||||
return value != null && !value.isBlank();
|
|
||||||
}
|
|
||||||
}
|
|
||||||
@@ -17,12 +17,7 @@ public class SecurityConfig {
|
|||||||
.sessionManagement(session ->
|
.sessionManagement(session ->
|
||||||
session.sessionCreationPolicy(SessionCreationPolicy.STATELESS))
|
session.sessionCreationPolicy(SessionCreationPolicy.STATELESS))
|
||||||
.authorizeHttpRequests(authorize -> authorize
|
.authorizeHttpRequests(authorize -> authorize
|
||||||
.requestMatchers(
|
.requestMatchers("/actuator/health", "/actuator/health/**", "/api/public")
|
||||||
"/actuator/health",
|
|
||||||
"/actuator/health/**",
|
|
||||||
"/api/public",
|
|
||||||
"/edge/**"
|
|
||||||
)
|
|
||||||
.permitAll()
|
.permitAll()
|
||||||
.anyRequest()
|
.anyRequest()
|
||||||
.authenticated())
|
.authenticated())
|
||||||
|
|||||||
@@ -40,20 +40,4 @@ class ApiSecurityTest {
|
|||||||
.andExpect(jsonPath("$.subject").value("test-subject"))
|
.andExpect(jsonPath("$.subject").value("test-subject"))
|
||||||
.andExpect(jsonPath("$.username").value("regular-user"));
|
.andExpect(jsonPath("$.username").value("regular-user"));
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
|
||||||
void edgeEndpointRejectsMissingIdentityHeader() throws Exception {
|
|
||||||
mockMvc.perform(get("/edge/me"))
|
|
||||||
.andExpect(status().isUnauthorized());
|
|
||||||
}
|
|
||||||
|
|
||||||
@Test
|
|
||||||
void edgeEndpointCurrentlyTrustsForwardedUserHeader() throws Exception {
|
|
||||||
mockMvc.perform(get("/edge/me")
|
|
||||||
.header("X-Forwarded-User", "regular-user")
|
|
||||||
.header("X-Forwarded-Email", "regular-user@example.test"))
|
|
||||||
.andExpect(status().isOk())
|
|
||||||
.andExpect(jsonPath("$.user").value("regular-user"))
|
|
||||||
.andExpect(jsonPath("$.identityHeader").value("X-Forwarded-User"));
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -86,56 +86,6 @@ services:
|
|||||||
- keycloak-net
|
- keycloak-net
|
||||||
restart: unless-stopped
|
restart: unless-stopped
|
||||||
|
|
||||||
oauth2-proxy:
|
|
||||||
image: quay.io/oauth2-proxy/oauth2-proxy:v7.15.2
|
|
||||||
command:
|
|
||||||
- --http-address=0.0.0.0:4180
|
|
||||||
- --provider=keycloak-oidc
|
|
||||||
- --oidc-issuer-url=http://localhost:8080/realms/keycloak-patterns
|
|
||||||
- --skip-oidc-discovery=true
|
|
||||||
- --login-url=http://localhost:8080/realms/keycloak-patterns/protocol/openid-connect/auth
|
|
||||||
- --redeem-url=http://keycloak:8080/realms/keycloak-patterns/protocol/openid-connect/token
|
|
||||||
- --oidc-jwks-url=http://keycloak:8080/realms/keycloak-patterns/protocol/openid-connect/certs
|
|
||||||
- --profile-url=http://keycloak:8080/realms/keycloak-patterns/protocol/openid-connect/userinfo
|
|
||||||
- --validate-url=http://keycloak:8080/realms/keycloak-patterns/protocol/openid-connect/userinfo
|
|
||||||
- --redirect-url=http://localhost:4180/oauth2/callback
|
|
||||||
- --upstream=http://app:8081
|
|
||||||
- --email-domain=*
|
|
||||||
- --scope=openid profile email
|
|
||||||
- --code-challenge-method=S256
|
|
||||||
- --cookie-name=AP4_SESSION
|
|
||||||
- --cookie-secure=false
|
|
||||||
- --cookie-samesite=lax
|
|
||||||
- --cookie-expire=1h
|
|
||||||
- --skip-provider-button=true
|
|
||||||
- --set-xauthrequest=true
|
|
||||||
- --pass-user-headers=true
|
|
||||||
- --whitelist-domain=localhost:4180
|
|
||||||
- --whitelist-domain=localhost:8080
|
|
||||||
environment:
|
|
||||||
OAUTH2_PROXY_CLIENT_ID: edge-proxy
|
|
||||||
OAUTH2_PROXY_CLIENT_SECRET: ${EDGE_PROXY_CLIENT_SECRET:?set EDGE_PROXY_CLIENT_SECRET in .env}
|
|
||||||
OAUTH2_PROXY_COOKIE_SECRET: ${OAUTH2_PROXY_COOKIE_SECRET:?set OAUTH2_PROXY_COOKIE_SECRET in .env}
|
|
||||||
ports:
|
|
||||||
- "127.0.0.1:4180:4180"
|
|
||||||
depends_on:
|
|
||||||
keycloak:
|
|
||||||
condition: service_healthy
|
|
||||||
app:
|
|
||||||
condition: service_healthy
|
|
||||||
healthcheck:
|
|
||||||
test:
|
|
||||||
- CMD
|
|
||||||
- /bin/oauth2-proxy
|
|
||||||
- --version
|
|
||||||
interval: 10s
|
|
||||||
timeout: 5s
|
|
||||||
retries: 3
|
|
||||||
start_period: 5s
|
|
||||||
networks:
|
|
||||||
- keycloak-net
|
|
||||||
restart: unless-stopped
|
|
||||||
|
|
||||||
nginx:
|
nginx:
|
||||||
build:
|
build:
|
||||||
context: ./frontend
|
context: ./frontend
|
||||||
|
|||||||
@@ -1,31 +0,0 @@
|
|||||||
# AP4 · oauth2-proxy Edge Forward Auth
|
|
||||||
|
|
||||||
## 첫 단계: oauth2-proxy 자체 OIDC 흐름
|
|
||||||
|
|
||||||
`feature/keycloak-oauth2-proxy-oidc-flow`에서는 oauth2-proxy를
|
|
||||||
`http://localhost:4180`에 직접 노출해 구성 요소를 분리해서 확인합니다.
|
|
||||||
|
|
||||||
1. `/edge/me` 미인증 요청이 Keycloak로 redirect됩니다.
|
|
||||||
2. oauth2-proxy는 confidential `edge-proxy` client와 PKCE S256을 사용합니다.
|
|
||||||
3. callback에서 code/token 교환과 ID/access token 검증은 서버끼리
|
|
||||||
수행합니다.
|
|
||||||
4. 브라우저에는 HttpOnly `AP4_SESSION` cookie만 남습니다.
|
|
||||||
5. oauth2-proxy가 backend 요청에 `X-Forwarded-User`를 붙여 200을 받습니다.
|
|
||||||
|
|
||||||
Keycloak이 발급하는 issuer는 브라우저 기준
|
|
||||||
`http://localhost:8080/realms/keycloak-patterns`입니다. 컨테이너 내부의
|
|
||||||
`localhost`는 oauth2-proxy 자신이므로 discovery endpoint에 도달할 수
|
|
||||||
없습니다. 그래서 이 로컬 Compose 구성은 issuer 검증값은 외부 URL로
|
|
||||||
유지하되, login URL은 브라우저용 외부 주소, token/JWKS/userinfo는
|
|
||||||
`http://keycloak:8080` 내부 주소로 각각 명시합니다.
|
|
||||||
|
|
||||||
HTTP 로컬 시연이라 `cookie-secure=false`를 사용합니다. 운영 HTTPS에서는
|
|
||||||
반드시 secure cookie로 되돌려야 합니다.
|
|
||||||
|
|
||||||
## 다음 단계의 보안 전제
|
|
||||||
|
|
||||||
이 첫 feature의 backend는 전달된 사용자 헤더를 신뢰하며 8081도
|
|
||||||
loopback에 publish되어 있습니다. 따라서 로컬에서 직접
|
|
||||||
`X-Forwarded-User: spoofed-admin`을 보내면 우회가 재현됩니다. 이후
|
|
||||||
Nginx `auth_request` 통합을 거쳐 최종 feature에서 backend no-publish와
|
|
||||||
내부 shared-secret 검증을 함께 적용합니다.
|
|
||||||
@@ -0,0 +1,29 @@
|
|||||||
|
# Keycloak branch implementation index
|
||||||
|
|
||||||
|
The source inventory contains 39 `feature-keycloak-*.md` branch notes. This
|
||||||
|
repository preserves one local Git feature branch for every note and merges it
|
||||||
|
with `--no-ff` into either the common `develop` baseline or one of the four
|
||||||
|
authentication-pattern branches.
|
||||||
|
|
||||||
|
| Target | Meaning |
|
||||||
|
|---|---|
|
||||||
|
| `common` | Shared realm, federation, deployment, or governance contract. Merge into `develop`, then propagate to AP1–AP4. |
|
||||||
|
| `ap1` | Browser-based OAuth client: vanilla SPA, Authorization Code + PKCE, Resource Server. |
|
||||||
|
| `ap2` | Token-mediating confidential backend: browser receives access token only. |
|
||||||
|
| `ap3` | BFF: backend owns every OAuth token and browser owns only a session cookie. |
|
||||||
|
| `ap4` | Edge forward-auth: oauth2-proxy/Nginx owns login and backend trusts an isolated identity header. |
|
||||||
|
|
||||||
|
The machine-readable registry is
|
||||||
|
[`keycloak-branch-manifest.tsv`](keycloak-branch-manifest.tsv). Run:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
./scripts/audit-keycloak-branches.sh
|
||||||
|
```
|
||||||
|
|
||||||
|
The audit succeeds only when all 39 note names have matching local feature
|
||||||
|
branches and each feature tip is reachable from its declared target branch.
|
||||||
|
|
||||||
|
Google credentials are never committed. The default local acceptance harness
|
||||||
|
uses a second Keycloak realm as a controllable OIDC provider so claim mapping
|
||||||
|
and unsafe-linking failure paths can be reproduced. A real Google login remains
|
||||||
|
an explicit credentialed/public-HTTPS verification profile.
|
||||||
@@ -0,0 +1,40 @@
|
|||||||
|
branch target delivery
|
||||||
|
feature/keycloak-account-linking-spa-ux ap1 documented-and-contract-tested
|
||||||
|
feature/keycloak-account-linking-sub-vs-email common documented-and-contract-tested
|
||||||
|
feature/keycloak-bff-csrf-samesite-defense ap3 locally-verified
|
||||||
|
feature/keycloak-bff-oauth2login-session ap3 locally-verified
|
||||||
|
feature/keycloak-bff-vs-spa-direct ap3 documented
|
||||||
|
feature/keycloak-docker-compose-stack common locally-verified
|
||||||
|
feature/keycloak-edge-forwardauth-google-federation ap4 documented-and-config-tested
|
||||||
|
feature/keycloak-edge-forwardauth-no-google ap4 documented-and-config-tested
|
||||||
|
feature/keycloak-federation-spa-zero-change ap1 contract-tested
|
||||||
|
feature/keycloak-first-broker-login-flow common locally-verified-with-mock-idp
|
||||||
|
feature/keycloak-four-pattern-tradeoff-matrix common documented-and-evidence-linked
|
||||||
|
feature/keycloak-google-claim-attribute-mapping common locally-verified-with-mock-idp
|
||||||
|
feature/keycloak-google-redirect-uri-policy common config-tested
|
||||||
|
feature/keycloak-header-spoofing-defense ap4 locally-verified
|
||||||
|
feature/keycloak-https-termination-caddy-nginx common config-tested
|
||||||
|
feature/keycloak-idp-brokering-google-client common locally-verified-with-mock-idp
|
||||||
|
feature/keycloak-idp-mappers-claim-to-role common locally-verified-with-mock-idp
|
||||||
|
feature/keycloak-internal-spa-direct-google-federation ap1 documented-and-contract-tested
|
||||||
|
feature/keycloak-internal-spa-direct-no-google ap1 documented-and-contract-tested
|
||||||
|
feature/keycloak-iss-claim-hostname-mismatch ap1 locally-verified
|
||||||
|
feature/keycloak-nginx-auth-request-integration ap4 locally-verified
|
||||||
|
feature/keycloak-oauth2-proxy-oidc-flow ap4 locally-verified
|
||||||
|
feature/keycloak-patterns common governance
|
||||||
|
feature/keycloak-pkce-flow-stages ap1 contract-tested
|
||||||
|
feature/keycloak-public-domain-tunneling common config-tested
|
||||||
|
feature/keycloak-realm-client-export common locally-verified
|
||||||
|
feature/keycloak-refresh-rotation-and-logout ap1 locally-verified
|
||||||
|
feature/keycloak-refresh-token-rotation ap1 contract-tested
|
||||||
|
feature/keycloak-reverse-proxy-headers common config-tested
|
||||||
|
feature/keycloak-single-ec2-google-federation ap1 documented-and-config-tested
|
||||||
|
feature/keycloak-single-ec2-no-google ap1 documented-and-contract-tested
|
||||||
|
feature/keycloak-spa-token-storage-tradeoff ap1 locally-verified
|
||||||
|
feature/keycloak-spring-rs-audience-validator ap1 locally-verified
|
||||||
|
feature/keycloak-spring-rs-role-mapping ap1 locally-verified
|
||||||
|
feature/keycloak-three-leg-trust-chain ap1 documented-and-contract-tested
|
||||||
|
feature/keycloak-token-mediating-access-handoff ap2 locally-verified
|
||||||
|
feature/keycloak-token-mediating-confidential-client ap2 locally-verified
|
||||||
|
feature/keycloak-traefik-forwardauth-alternative ap4 config-tested
|
||||||
|
feature/keycloak-vanilla-js-spa-pkce ap1 locally-verified
|
||||||
|
Generated
-28
@@ -1,28 +0,0 @@
|
|||||||
{
|
|
||||||
"name": "keycloak-pattern-e2e",
|
|
||||||
"version": "1.0.0",
|
|
||||||
"lockfileVersion": 3,
|
|
||||||
"requires": true,
|
|
||||||
"packages": {
|
|
||||||
"": {
|
|
||||||
"name": "keycloak-pattern-e2e",
|
|
||||||
"version": "1.0.0",
|
|
||||||
"devDependencies": {
|
|
||||||
"playwright-core": "1.62.0"
|
|
||||||
}
|
|
||||||
},
|
|
||||||
"node_modules/playwright-core": {
|
|
||||||
"version": "1.62.0",
|
|
||||||
"resolved": "https://registry.npmjs.org/playwright-core/-/playwright-core-1.62.0.tgz",
|
|
||||||
"integrity": "sha512-nsNRyq0r2zsG8AcRHWknc9QRA5XCueC7gWMrs+Gx2tlZn9hcl8zudfh00lhJPY1DE7NmZ6bDsT9g2yey8mXljA==",
|
|
||||||
"dev": true,
|
|
||||||
"license": "Apache-2.0",
|
|
||||||
"bin": {
|
|
||||||
"playwright-core": "cli.js"
|
|
||||||
},
|
|
||||||
"engines": {
|
|
||||||
"node": ">=20"
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
@@ -1,12 +0,0 @@
|
|||||||
{
|
|
||||||
"name": "keycloak-pattern-e2e",
|
|
||||||
"private": true,
|
|
||||||
"version": "1.0.0",
|
|
||||||
"type": "module",
|
|
||||||
"scripts": {
|
|
||||||
"test:pattern4": "node pattern4.mjs"
|
|
||||||
},
|
|
||||||
"devDependencies": {
|
|
||||||
"playwright-core": "1.62.0"
|
|
||||||
}
|
|
||||||
}
|
|
||||||
@@ -1,111 +0,0 @@
|
|||||||
import assert from "node:assert/strict";
|
|
||||||
import { chromium } from "playwright-core";
|
|
||||||
|
|
||||||
const password = process.env.E2E_PASSWORD;
|
|
||||||
assert.ok(password, "E2E_PASSWORD must be set");
|
|
||||||
|
|
||||||
async function completeKeycloakLogin(page) {
|
|
||||||
for (let attempt = 1; attempt <= 2; attempt += 1) {
|
|
||||||
await page.locator("#username").fill(
|
|
||||||
process.env.E2E_USERNAME ?? "regular-user",
|
|
||||||
);
|
|
||||||
await page.locator("#password").fill(password);
|
|
||||||
await page.locator("#kc-login").click();
|
|
||||||
await page.waitForLoadState("domcontentloaded");
|
|
||||||
|
|
||||||
if (page.url() === "http://localhost:4180/edge/me") {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
if (attempt === 1) {
|
|
||||||
await page.goto("http://localhost:4180/oauth2/start?rd=%2Fedge%2Fme");
|
|
||||||
await page.waitForURL(/localhost:8080/u);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
throw new Error(`Keycloak login did not return to AP4: ${page.url()}`);
|
|
||||||
}
|
|
||||||
|
|
||||||
const browser = await chromium.launch({
|
|
||||||
executablePath: process.env.CHROME_BIN ?? "/usr/bin/google-chrome",
|
|
||||||
headless: true,
|
|
||||||
args: ["--no-sandbox"],
|
|
||||||
});
|
|
||||||
|
|
||||||
try {
|
|
||||||
const context = await browser.newContext();
|
|
||||||
const page = await context.newPage();
|
|
||||||
const browserRequests = [];
|
|
||||||
page.on("request", (request) =>
|
|
||||||
browserRequests.push({ method: request.method(), url: request.url() }),
|
|
||||||
);
|
|
||||||
|
|
||||||
const edgeResponsePromise = page.waitForResponse(
|
|
||||||
(response) =>
|
|
||||||
response.url() === "http://localhost:4180/edge/me" &&
|
|
||||||
response.status() === 302,
|
|
||||||
);
|
|
||||||
const authorizationRequestPromise = page.waitForRequest((request) =>
|
|
||||||
request.url().includes(
|
|
||||||
"/protocol/openid-connect/auth?approval_prompt=",
|
|
||||||
),
|
|
||||||
);
|
|
||||||
await page.goto("http://localhost:4180/edge/me");
|
|
||||||
const unauthenticatedEdgeResponse = await edgeResponsePromise;
|
|
||||||
assert.equal(unauthenticatedEdgeResponse.status(), 302);
|
|
||||||
|
|
||||||
const authorizationRequest = await authorizationRequestPromise;
|
|
||||||
const authorizationUrl = new URL(authorizationRequest.url());
|
|
||||||
assert.equal(authorizationUrl.searchParams.get("client_id"), "edge-proxy");
|
|
||||||
assert.equal(authorizationUrl.searchParams.get("code_challenge_method"), "S256");
|
|
||||||
assert.ok(authorizationUrl.searchParams.get("code_challenge"));
|
|
||||||
|
|
||||||
await page.waitForURL(/localhost:8080/u);
|
|
||||||
await completeKeycloakLogin(page);
|
|
||||||
const edgeIdentity = JSON.parse(await page.locator("body").innerText());
|
|
||||||
assert.equal(edgeIdentity.pattern, "AP4-edge-forward-auth");
|
|
||||||
assert.ok(edgeIdentity.user);
|
|
||||||
assert.equal(edgeIdentity.identityHeader, "X-Forwarded-User");
|
|
||||||
|
|
||||||
const callbackRequest = browserRequests.find(({ url }) =>
|
|
||||||
url.startsWith("http://localhost:4180/oauth2/callback?"),
|
|
||||||
);
|
|
||||||
assert.ok(callbackRequest);
|
|
||||||
assert.equal(callbackRequest.method, "GET");
|
|
||||||
assert.equal(
|
|
||||||
browserRequests.some(({ url }) =>
|
|
||||||
url.includes("/protocol/openid-connect/token"),
|
|
||||||
),
|
|
||||||
false,
|
|
||||||
"the confidential token exchange must be server-to-server",
|
|
||||||
);
|
|
||||||
|
|
||||||
const cookies = await context.cookies("http://localhost:4180/");
|
|
||||||
const sessionCookie = cookies.find((cookie) => cookie.name === "AP4_SESSION");
|
|
||||||
assert.ok(sessionCookie);
|
|
||||||
assert.equal(sessionCookie.httpOnly, true);
|
|
||||||
assert.equal(sessionCookie.sameSite, "Lax");
|
|
||||||
assert.equal(sessionCookie.secure, false);
|
|
||||||
|
|
||||||
const storage = await page.evaluate(() => ({
|
|
||||||
localStorage: Object.values(localStorage),
|
|
||||||
sessionStorage: Object.values(sessionStorage),
|
|
||||||
readableCookies: document.cookie,
|
|
||||||
}));
|
|
||||||
assert.deepEqual(storage.localStorage, []);
|
|
||||||
assert.deepEqual(storage.sessionStorage, []);
|
|
||||||
assert.equal(storage.readableCookies.includes("AP4_SESSION"), false);
|
|
||||||
|
|
||||||
const missingHeader = await fetch("http://localhost:8081/edge/me");
|
|
||||||
assert.equal(missingHeader.status, 401);
|
|
||||||
const directSpoof = await fetch("http://localhost:8081/edge/me", {
|
|
||||||
headers: { "X-Forwarded-User": "spoofed-admin" },
|
|
||||||
});
|
|
||||||
assert.equal(directSpoof.status, 200);
|
|
||||||
const spoofedIdentity = await directSpoof.json();
|
|
||||||
assert.equal(spoofedIdentity.user, "spoofed-admin");
|
|
||||||
|
|
||||||
console.log(
|
|
||||||
"pattern4 oauth2-proxy verified: redirect, PKCE login, forwarded-user 200, direct spoof precondition",
|
|
||||||
);
|
|
||||||
} finally {
|
|
||||||
await browser.close();
|
|
||||||
}
|
|
||||||
@@ -120,7 +120,6 @@
|
|||||||
],
|
],
|
||||||
"webOrigins": [],
|
"webOrigins": [],
|
||||||
"attributes": {
|
"attributes": {
|
||||||
"pkce.code.challenge.method": "S256",
|
|
||||||
"post.logout.redirect.uris": "http://localhost:8088/*"
|
"post.logout.redirect.uris": "http://localhost:8088/*"
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Executable
+62
@@ -0,0 +1,62 @@
|
|||||||
|
#!/usr/bin/env sh
|
||||||
|
set -eu
|
||||||
|
|
||||||
|
manifest="${1:-docs/keycloak-branch-manifest.tsv}"
|
||||||
|
notes_dir="${KEYCLOAK_BRANCH_NOTES_DIR:-/home/donghyeon/workspace/ai-tools/llm-wiki/raw/branch-notes}"
|
||||||
|
|
||||||
|
expected_count="$(awk 'NR > 1 { count += 1 } END { print count + 0 }' "$manifest")"
|
||||||
|
if [ "$expected_count" -ne 39 ]; then
|
||||||
|
echo "manifest must contain exactly 39 Keycloak branches; found $expected_count" >&2
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
|
||||||
|
note_count="$(find "$notes_dir" -maxdepth 1 -type f -name 'feature-keycloak-*.md' | wc -l)"
|
||||||
|
if [ "$note_count" -ne 39 ]; then
|
||||||
|
echo "branch-note inventory must contain exactly 39 files; found $note_count" >&2
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
|
||||||
|
missing=0
|
||||||
|
unmerged=0
|
||||||
|
tab="$(printf '\t')"
|
||||||
|
|
||||||
|
while IFS="$tab" read -r branch target delivery; do
|
||||||
|
[ "$branch" = "branch" ] && continue
|
||||||
|
|
||||||
|
note_name="$(printf '%s\n' "$branch" |
|
||||||
|
sed 's#^feature/keycloak-#feature-keycloak-#').md"
|
||||||
|
if [ ! -f "$notes_dir/$note_name" ]; then
|
||||||
|
echo "missing branch note: $note_name" >&2
|
||||||
|
missing=$((missing + 1))
|
||||||
|
fi
|
||||||
|
|
||||||
|
if ! git show-ref --verify --quiet "refs/heads/$branch"; then
|
||||||
|
echo "missing local branch: $branch" >&2
|
||||||
|
missing=$((missing + 1))
|
||||||
|
continue
|
||||||
|
fi
|
||||||
|
|
||||||
|
case "$target" in
|
||||||
|
common) target_branch="develop" ;;
|
||||||
|
ap1) target_branch="develop-keycloak-pattern1" ;;
|
||||||
|
ap2) target_branch="develop-keycloak-pattern2" ;;
|
||||||
|
ap3) target_branch="develop-keycloak-pattern3" ;;
|
||||||
|
ap4) target_branch="develop-keycloak-pattern4" ;;
|
||||||
|
*)
|
||||||
|
echo "unknown target '$target' for $branch ($delivery)" >&2
|
||||||
|
exit 1
|
||||||
|
;;
|
||||||
|
esac
|
||||||
|
|
||||||
|
if ! git merge-base --is-ancestor "$branch" "$target_branch"; then
|
||||||
|
echo "feature tip is not merged: $branch -> $target_branch" >&2
|
||||||
|
unmerged=$((unmerged + 1))
|
||||||
|
fi
|
||||||
|
done < "$manifest"
|
||||||
|
|
||||||
|
if [ "$missing" -ne 0 ] || [ "$unmerged" -ne 0 ]; then
|
||||||
|
echo "Keycloak branch audit failed: missing=$missing unmerged=$unmerged" >&2
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
|
||||||
|
echo "Keycloak branch audit passed: 39/39 branches exist and are merged"
|
||||||
@@ -1,21 +0,0 @@
|
|||||||
#!/usr/bin/env sh
|
|
||||||
set -eu
|
|
||||||
|
|
||||||
if [ ! -f .env ]; then
|
|
||||||
echo "missing .env; copy .env.example and set development values" >&2
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
|
|
||||||
set -a
|
|
||||||
. ./.env
|
|
||||||
set +a
|
|
||||||
|
|
||||||
docker compose down --volumes --remove-orphans
|
|
||||||
docker compose up --build -d --wait
|
|
||||||
|
|
||||||
npm --prefix e2e ci
|
|
||||||
E2E_USERNAME=regular-user \
|
|
||||||
E2E_PASSWORD="$REGULAR_USER_PASSWORD" \
|
|
||||||
npm --prefix e2e run test:pattern4
|
|
||||||
|
|
||||||
echo "AP4 oauth2-proxy edge flow verified"
|
|
||||||
Reference in New Issue
Block a user