Skip to content

[Cloudflare One] Update Keycloak SAML docs#32071

Open
kennyj42 wants to merge 1 commit into
cloudflare:productionfrom
kennyj42:kjohnson/auth-8879-keycloak-docs
Open

[Cloudflare One] Update Keycloak SAML docs#32071
kennyj42 wants to merge 1 commit into
cloudflare:productionfrom
kennyj42:kjohnson/auth-8879-keycloak-docs

Conversation

@kennyj42

Copy link
Copy Markdown
Collaborator

Closes https://jira.cfdata.org/browse/AUTH-8879

Summary

  • Updates the Keycloak SAML setup steps for the current Admin Console
  • Changes Keycloak SAML URL examples to the v17+ default path without /auth
  • Adds a v16-and-earlier compatibility note
  • Removes stale Keycloak v16 screenshots and the old lab repo link

Validation

  • pnpm exec prettier --check src/content/docs/cloudflare-one/integrations/identity-providers/keycloak.mdx passed
  • git diff --check passed
  • pnpm run check fails on generated skills/turnstile-spin Worker template types unrelated to this change (ExportedHandler / Fetcher missing)

@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Review

⚠️ 1 warning found in commit fe78cb5.

Fix in your agent
Fix the following review findings in PR #32071 (https://github.com/cloudflare/cloudflare-docs/pull/32071).

Before making changes, review each finding and present a brief summary table:
- For each finding, state whether you agree, disagree, or need clarification
- If you disagree (e.g. the fix requires disproportionate effort for minimal benefit,
  or the finding is factually incorrect), explain why
- If you need clarification before deciding, ask those questions
- Then share your plan for which issues to tackle and in what order

After triaging, fix all legitimate findings. For any you decide to skip,
post a comment on this PR with the finding ID and your reasoning.

---

## Style Guide Review

### Warnings (1)

#### SG-690f10b9299b · Use turn on/turn off for toggles
- **File:** `src/content/docs/cloudflare-one/integrations/identity-providers/keycloak.mdx` line 51
- **Issue:** Added line reads `If you wish to enable client signatures, enable **Client Signature Required** and select **Save**.`
- **Fix:** Replace `enable` with `turn on` for the toggle: `If you want to turn on client signatures, turn on **Client Signature Required** and select **Save**.`

Code Review

This code review is in beta and may not always be helpful — use your judgment.

No code review issues found.

Conventions

No convention issues found.

Style Guide Review

Warnings (1)
File Issue
cloudflare-one/integrations/identity-providers/keycloak.mdx line 51 Use turn on/turn off for toggles — Added line reads If you wish to enable client signatures, enable \*\*Client Signature Required\*\* and select \*\*Save\*\*. Fix: Replace enable with turn on for the toggle: If you want to turn on client signatures, turn on \*\*Client Signature Required\*\* and select \*\*Save\*\*.
Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.
/ignore-review-limit Permanently lifts the 2-review automatic limit for this PR. Future pushes will trigger reviews as normal.
/disable-auto-review Stops automatic reviews from triggering on future pushes to this PR. Codeowners can still run /review or /full-review manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants