Skip to content

CAMEL-24973: docs - Add the 4.22.2 and 4.18.5 upgrade guide notes for URI masking - #26963

Merged
davsclaus merged 1 commit into
apache:mainfrom
oscerd:docs/CAMEL-24973-backport-guides
Sep 28, 2026
Merged

davsclaus merged 1 commit into
apache:mainfrom
oscerd:docs/CAMEL-24973-backport-guides

Conversation

@oscerd

@oscerd oscerd commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Description

CAMEL-24973 was fixed on main in #26803 and is backported to camel-4.22.x (#26895) and camel-4.18.x (#26962). The upgrade guides for all release lines live on main, so this adds the matching notes:

  • camel-4x-upgrade-guide-4_22.adoc, 4.22.2 section: same note as in the 4.23 guide.
  • camel-4x-upgrade-guide-4_18.adoc, 4.18.5 section: same note without the SensitiveUtils.maskUserInfoCredentials() paragraph, since that method is not on camel-4.18.x.

This should be merged once the two backports are merged.

Claude Code on behalf of oscerd

🤖 Generated with Claude Code

@oscerd
oscerd requested review from davsclaus and gnodet September 28, 2026 07:06
@github-actions

Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@github-actions github-actions Bot added the docs label Sep 28, 2026
@github-actions

github-actions Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • docs

🔬 Scalpel shadow comparison — Scalpel: 0 of 695 tested, 0 compile-only — current: 0 all tested

Maveniverse Scalpel detected 0 affected modules (current approach: 0).

Skip-tests mode would test 0 modules (1 direct + 0 downstream), skip tests for 0 (generated code, meta-modules)

Modules Scalpel would test (0)

ℹ️ Shadow mode — Scalpel observes but does not affect test execution. Learn more


⚙️ View full build and test results

@apupier

apupier commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

conflicts to resolve

@apupier

apupier commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

there was a force-push but it is still or again marked with conflicts to resolve.

I will use same advice than for the latest version but with a subtility. try to insert in the correct chapter level 2 and insert alphabetically instead of at the end of it to limit risk of collision

… URI masking

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
@oscerd
oscerd force-pushed the docs/CAMEL-24973-backport-guides branch from 6550b5f to 9c034ac Compare September 28, 2026 16:17
@oscerd

oscerd commented Sep 28, 2026

Copy link
Copy Markdown
Contributor Author

@apupier thanks, done in 9c034ac. The branch is now rebuilt on current main. Each note sits inside its own level-2 chapter (Upgrading from 4.18.4 to 4.18.5, Upgrading from 4.22.1 to 4.22.2), in alphabetical order as the first camel-core entry, just before camel-core - Rest DSL response Content-Type, instead of at the end where the camel-saga and camel-coap notes kept landing. The text of both notes is unchanged.

Claude Code on behalf of @oscerd

@davsclaus davsclaus added this to the 4.23.0 milestone Sep 28, 2026
@davsclaus
davsclaus merged commit 6b0d41a into apache:main Sep 28, 2026
7 checks passed
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.

3 participants