Skip to content

ci: use relative links in security-model.adoc#8661

Merged
davsclaus merged 1 commit into
apache:mainfrom
oscerd:ci-issue/cq-security-model-relative-links
May 16, 2026
Merged

ci: use relative links in security-model.adoc#8661
davsclaus merged 1 commit into
apache:mainfrom
oscerd:ci-issue/cq-security-model-relative-links

Conversation

@oscerd

@oscerd oscerd commented May 15, 2026

Copy link
Copy Markdown
Contributor

CI Fix

Failed run (camel-website): https://github.com/apache/camel-website/actions/runs/25926572539

Error

camel-website's check:html step enforces the camel/relative-links rule:
in-site links must be relative, not absolute https://camel.apache.org/....
The Camel Quarkus security model page
(docs/modules/ROOT/pages/user-guide/security-model.adoc) used 2 absolute
in-site URLs, producing 2 errors in
public/camel-quarkus/next/user-guide/security-model.html and failing the
build for every camel-website pull request (unrelated to whichever
camel-website PR triggers the run).

Fix

Convert the 2 in-site links to root-relative link:/security/[...], matching
apache/camel core security-model.adoc (fixed identically in
apache/camel#23224; companion PRs: apache/camel-kamelets#2836,
apache/camel-kafka-connector#1774). The private-security@camel.apache.org
email reference is left unchanged.

Documentation-only, single file, same link target rendered relative. No local
Antora CLI in the environment; the camel-website build is the definitive check.

Deferred Issues

None.


Claude Code on behalf of Andrea Cosentino

The Camel Quarkus security model user-guide page linked the camel.apache.org
security page with absolute https://camel.apache.org/... URLs. camel-website's
check:html step enforces the camel/relative-links rule and fails on in-site
absolute links (2 errors for this page), turning every camel-website pull
request red.

Convert the two in-site links to root-relative link:/security/, matching
apache/camel core security-model.adoc (fixed the same way in apache/camel#23224).
The private-security@camel.apache.org email reference is intentionally left
unchanged. Documentation-only; same link target, rendered relative.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
@davsclaus davsclaus merged commit 24b53fc into apache:main May 16, 2026
1 check passed
@oscerd oscerd requested review from jamesnetherton and ppalaga May 16, 2026 16:38
oscerd added a commit to apache/camel-kamelets that referenced this pull request May 16, 2026
Follow-up to #2836. That PR was squash-merged with only its first commit
(placeholder escaping); the relative-link commit was pushed afterwards and
never merged, so camel-kamelets main still carries 4 absolute
https://camel.apache.org/... links. They are now the sole remaining
camel/relative-links failure in the camel-website check:html step (4 errors
in camel-kamelets/next/security-model.html), keeping every camel-website PR
red.

Convert the 4 in-site links to root-relative link: macros (link:/security/,
link:/manual/security-model.html), matching apache/camel core
security-model.adoc (apache/camel#23224) and the sibling subproject fixes
(apache/camel-kafka-connector#1774, apache/camel-quarkus#8661). The merged
placeholder escaping is left intact; private-security@camel.apache.org email
references are unchanged. Documentation-only; same link targets, rendered
relative.

Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants