Skip to content

[RHIDP-9783]: Add PingIdentity authentication documentation#2045

Open
themr0c wants to merge 27 commits intoredhat-developer:mainfrom
themr0c:rhidp-9783-pingidentity-auth-docs
Open

[RHIDP-9783]: Add PingIdentity authentication documentation#2045
themr0c wants to merge 27 commits intoredhat-developer:mainfrom
themr0c:rhidp-9783-pingidentity-auth-docs

Conversation

@themr0c
Copy link
Copy Markdown
Member

@themr0c themr0c commented Apr 14, 2026

IMPORTANT: Do Not Merge - To be merged by Docs Team Only

Version(s): 1.6
Issue: https://issues.redhat.com/browse/RHIDP-9783
Preview: N/A

Summary

  • Add procedure for enabling user authentication with PingIdentity (PingFederate or PingOne) via the OIDC provider
  • Add Tech Preview procedure for provisioning users and groups from PingOne to the RHDH software catalog via the PingIdentity catalog provider plugin
  • Add assembly wrapping both procedures, included in the authentication title after GitLab

themr0c and others added 3 commits April 14, 2026 15:45
Add procedure for enabling user authentication with PingIdentity
(PingFederate or PingOne) via the OIDC provider.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add Tech Preview procedure for provisioning users and groups from
PingOne to the RHDH software catalog via the PingIdentity catalog
provider plugin.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…itle

Add assembly wrapping PingIdentity auth and provisioning procedures.
Include assembly in the authentication title after GitLab.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rhdh-bot
Copy link
Copy Markdown
Collaborator

rhdh-bot commented Apr 14, 2026

PR Build Results

Build failed -- 33/34 titles | 174s
Preview: https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-2045/ (stale -- title build failed, showing previous version)

control-access_authentication-in-rhdh

Error: /docs/titles/control-access_authentication-in-rhdh/build/en-US/master.xml fails to validate
Cause: Document structure validation failed
Fix: Run 'ccutil compile' locally and check the full error output

View full logs


Content Quality Assessment

CQA Report

  • CQA-00a: Orphaned modules
  • CQA-00b: Directory structure
  • CQA-01: Vale AsciiDoc DITA compliance
  • CQA-02: Verify assembly structure
  • CQA-03: Verify content type metadata
  • CQA-04: Verify module templates
  • CQA-05: Verify required modular elements
  • CQA-06: Verify assemblies follow official template (one user story)
  • CQA-07: Verify TOC depth (max 3 levels)
  • CQA-08: Verify short description content quality
  • CQA-09: Verify short description format
  • CQA-10: Verify titles are brief, complete, and descriptive
  • CQA-11: Verify procedure prerequisites
  • CQA-12: Verify grammar and style (Vale)
  • CQA-13: Verify content matches declared type
  • CQA-14: Verify no broken links
  • CQA-15: Check redirects
  • CQA-16: Verify official product names
  • CQA-17: Verify legal disclaimers for preview features

Summary

Checks: 19 total, 19 pass, 0 fail

19 checks: 19 pass, 0 fail

Run node build/scripts/cqa/index.js --all --fix locally to review and auto-fix issues.


Updated 2026-04-23 15:57:37 UTC

@themr0c themr0c requested a review from JessicaJHee April 21, 2026 12:58
Replace generic OIDC auth procedure with detailed PingFederate+LDAP
configuration covering LDAP binary attributes, authentication policy
contract with OGNL GUID transformation, and oidcLdapUuidMatchingAnnotation
resolver. Remove PingOne provisioning procedure in favor of existing
LDAP provisioning module.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Member

@pabel-rh pabel-rh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with minor comments.

Comment thread assemblies/shared/assembly-enable-authentication-with-pingidentity.adoc Outdated
Comment thread modules/shared/proc-enable-user-authentication-with-pingidentity.adoc Outdated
Comment thread modules/shared/proc-enable-user-authentication-with-pingidentity.adoc Outdated
Comment thread modules/shared/proc-enable-user-authentication-with-pingidentity.adoc Outdated
Comment thread modules/shared/proc-enable-user-authentication-with-pingidentity.adoc Outdated
Comment thread modules/shared/proc-enable-user-authentication-with-pingidentity.adoc Outdated
Comment thread modules/shared/proc-enable-user-authentication-with-pingidentity.adoc Outdated
Comment thread modules/shared/proc-enable-user-authentication-with-pingidentity.adoc Outdated
Comment thread modules/shared/proc-enable-user-authentication-with-pingidentity.adoc Outdated
Comment thread modules/shared/proc-enable-user-authentication-with-pingidentity.adoc Outdated
…ty.adoc

Co-authored-by: Priyanka Abel <pabel@redhat.com>
Co-authored-by: Priyanka Abel <pabel@redhat.com>
Copy link
Copy Markdown
Member

@JessicaJHee JessicaJHee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one last small comment - otherwise LGTM!

:_mod-docs-content-type: PROCEDURE

[id="enable-user-authentication-with-pingidentity_{context}"]
= Enable user authentication with PingIdentity
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
= Enable user authentication with PingIdentity
= Enable user authentication with PingFederate

Please change the product name to PingFederate, PingIdentity is the name of the company that has a line of IAM products but PingFederate is the authentication provider that we're support

…ttern

PingIdentity is the company name; PingFederate is the specific
authentication provider product. Rename file, ID, title, and all
content references to use PingFederate consistently.

Title now follows the JTBD pattern used by other auth providers:
"Enable authentication with <provider>"

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sonarqubecloud
Copy link
Copy Markdown

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.

5 participants