oscerd opened a new pull request, #23470:
URL: https://github.com/apache/camel/pull/23470

   ## Summary
   
   Renames the two Exchange header string values in `Jt400Constants` that were
   **not** in the `Camel` namespace (and therefore not filtered by the default
   `HeaderFilterStrategy`) to `CamelJt400<Name>`, following the convention used
   across the rest of the Camel component catalog and matching the pattern
   established in CAMEL-23526 (`camel-cxf`), CAMEL-23522 (`camel-mail`),
   CAMEL-23461 (`camel-aws-bedrock`), CAMEL-23532 (`camel-vertx-websocket` /
   `camel-atmosphere-websocket` / `camel-iggy`), and CAMEL-23576 (`camel-jira`).
   
   | Constant | Previous value | New value |
   |----------|----------------|-----------|
   | `Jt400Constants.KEY` | `KEY` | `CamelJt400Key` |
   | `Jt400Constants.SENDER_INFORMATION` | `SENDER_INFORMATION` | 
`CamelJt400SenderInformation` |
   
   `Jt400Constants.KEY` is the data-queue key used for keyed-data-queue read and
   write operations (the header called out in the CAMEL-23577 sweep);
   `SENDER_INFORMATION` is included because it was the only other unprefixed
   constant in the class.
   
   The Java field names are unchanged so routes referencing the constants
   symbolically continue to work; routes using the literal string values must be
   updated (documented in the 4.21 upgrade guide).
   
   ## Already-compliant constants left unchanged
   
   `MESSAGE`, `MESSAGE_ID`, `MESSAGE_FILE`, `MESSAGE_TYPE`, `MESSAGE_SEVERITY`,
   `MESSAGE_DFT_RPY`, and `MESSAGE_REPLYTO_KEY` were already `Camel`-prefixed.
   
   ## Generated artifacts
   
   - `components/camel-jt400/.../jt400.json` + catalog mirror
   - `dsl/camel-endpointdsl/.../Jt400EndpointBuilderFactory.java` — DSL header
     accessors renamed (`kEY()` → `jt400Key()`,
     `senderInformation()` → `jt400SenderInformation()`)
   
   ## Backports
   
   `camel-jt400` exists on `camel-4.18.x` and `camel-4.14.x` with the same 
legacy
   values — backports apply and will be filed as follow-up PRs.
   
   ## Test plan
   
   - [x] `mvn test` in `components/camel-jt400` — 34 tests pass (2 skipped)
   - [x] Module + catalog + endpointdsl built cleanly; only `camel-jt400` regen
         artifacts included
   - [x] Tests use symbolic constant references, so no test changes needed
   - [x] Upgrade guide entry added under `=== camel-jt400`
   
   Tracker: CAMEL-23577
   
   _Reported by Claude Code on behalf of Andrea Cosentino_


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to