Skip to content

RHDHBUGS-2260: trust corporate-issued certificate#2211

Closed
deerskindoll wants to merge 19 commits into
redhat-developer:mainfrom
deerskindoll:RHDHBUGS-2260
Closed

RHDHBUGS-2260: trust corporate-issued certificate#2211
deerskindoll wants to merge 19 commits into
redhat-developer:mainfrom
deerskindoll:RHDHBUGS-2260

Conversation

@deerskindoll

Copy link
Copy Markdown
Member

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

Version(s):

main, release-1.10
Issue:

https://redhat.atlassian.net/browse/RHDHBUGS-2260
Preview:

@rhdh-bot

rhdh-bot commented May 18, 2026

Copy link
Copy Markdown
Collaborator

PR Build Results

Build failed -- 32/34 titles | 89s
Preview: https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-2211/ (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

get-started_setting-up-and-configuring-your-first-red-hat-developer-hub-instance

Error: /docs/titles/get-started_setting-up-and-configuring-your-first-red-hat-developer-hub-instance/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-06-07 22:48:28 UTC

@deerskindoll deerskindoll changed the title [DRAFT] RHDHBUGS-2260: trust CA-issued certificate [DRAFT] RHDHBUGS-2260: trust corporate-issued certificate May 18, 2026
Comment thread modules/RHDHBUGS-2260.adoc Outdated
Comment thread modules/RHDHBUGS-2260.adoc Outdated
Comment thread modules/RHDHBUGS-2260.adoc Outdated
@deerskindoll deerskindoll changed the title [DRAFT] RHDHBUGS-2260: trust corporate-issued certificate RHDHBUGS-2260: trust corporate-issued certificate May 20, 2026
@deerskindoll deerskindoll added Peer review needed 📖 Technical review done ⛅ Any procedure has been succesfully tested and removed Technical review needed 🔩 Test all the procedures labels Jun 3, 2026

@rh-tokeefe rh-tokeefe left a comment

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.

LGTM

…rporate-certificate-authority-with-node-extra-ca-certs.adoc

Co-authored-by: Tim O'Keefe <tokeefe@redhat.com>
…dhat-developer#2302)

* Fix book-link xref format and add filecheck procedure

Remove leading '#' from book-link attribute xrefs across 104 files
to match the correct cross-reference format for external book links.

Add Scorecard file-level checks procedure to the Scorecards title.

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

* Revert "Fix book-link xref format and add filecheck procedure"

This reverts commit ad9bae2.

* RHDHBUGS-3298: Remove SNIPPET attribute from snippets to fix list continuation

The :_mod-docs-content-type: SNIPPET attribute at the start of snippet
files breaks AsciiDoc list continuation when the snippet is included
inside a procedure step with +. The attribute acts as a block boundary,
causing the snippet content to escape the step and float after the
procedure list, breaking DITA <steps> mapping.

CQA identifies snippets by their snip- filename prefix, not by this
attribute, so removing it is safe.

Also add missing + before the OCI tag snippet include in the Tekton
procedure, and re-enable the AsciiDocDITA.TaskInclude Vale rule.

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

* RHDHBUGS-3298: Wrap step 1 in open block for GitHub aux auth procedure

The guest user step snippet has no blank line after its attribute, so
the attribute is not the problem here. The issue is that step 1's
complex continuation content (definition list + WARNING) prevents
step 2 from continuing the numbered list.

Wrap step 1 content in an open block (--) so the included step 2
continues the list correctly. Restore the SNIPPET attribute in the
guest user snippet since it was not causing list breaks.

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

* RHDHBUGS-3298: Rewrite OCI tag snippet to use where: definition list convention

Replace the paragraph + bulleted list format with the project's
standard where: + definition list pattern for placeholder explanations.
This also helps list continuation since definition lists attach
naturally as step content.

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

* RHDHBUGS-3298: Restore SNIPPET attribute without blank line after it

The attribute + blank line combination broke list continuation. The
attribute alone (without a trailing blank line) is consumed silently
by Asciidoctor and does not break the + continuation.

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

* RHDHBUGS-3298: Use 'Enter' for placeholder description per convention

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

* RHDHBUGS-3298: Escape double underscores in OCI tag snippet with pass:c,a,q

Double underscores in backtick-quoted inline code were interpreted as
AsciiDoc emphasis markers, producing invalid DocBook XML that broke
ccutil builds for extend_configuring-dynamic-plugins and
extend_dynamic-plugins-reference titles.

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

* RHDHBUGS-3298: Inline sha256 tip into OCI snippet and delete orphan

The included snip-tag-for-sha256-tip.adoc SNIPPET attribute was
leaking as visible text when included after a + list continuation.
Inlining the TIP block follows the project where: + definition list
convention and removes the orphaned snippet file.

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

* RHDHBUGS-3298: Fix OCI snippet rendering and GitHub auth step numbering

Add blank line between where: and definition list so the TIP attaches
to the definition list item via +. Remove blank line before guest-user
snippet include so step 2 continues the numbered list.

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

* Update artifacts/snip-tag-for-OCI-package-paths.adoc

Co-authored-by: Priyanka Abel <pabel@redhat.com>

* Apply suggestion from @themr0c

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Priyanka Abel <pabel@redhat.com>
@sonarqubecloud

sonarqubecloud Bot commented Jun 7, 2026

Copy link
Copy Markdown

@deerskindoll

Copy link
Copy Markdown
Member Author

replaced by #2379

@deerskindoll

Copy link
Copy Markdown
Member Author

created PR against release-1.10 branch #2383

@deerskindoll deerskindoll deleted the RHDHBUGS-2260 branch June 16, 2026 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Peer review done 📘 Technical review done ⛅ Any procedure has been succesfully tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants