atiaomar1978-hub opened a new pull request, #25474:
URL: https://github.com/apache/camel/pull/25474

   ## Description
   
   Phase 2 of [CAMEL-24373](https://issues.apache.org/jira/browse/CAMEL-24373) 
adds four new Alibaba Cloud Camel components on top of the merged Phase 1 MVP 
(OSS + MNS).
   
   ### New components
   
   | Module | URI scheme | Operations |
   |--------|------------|------------|
   | `camel-alibaba-fc` | `alibaba-fc:` | `invokeFunction`, `getFunction` |
   | `camel-alibaba-sms` | `alibaba-sms:` | `sendSms` |
   | `camel-alibaba-kms` | `alibaba-kms:` | `encrypt`, `decrypt`, 
`generateDataKey` |
   | `camel-alibaba-eventbridge` | `alibaba-eventbridge:` | `putEvents` |
   
   ### Shared infrastructure
   
   - **`OpenApiClientSupport`** in `camel-alibaba-common` — shared Tea OpenAPI 
`Config` builder and credential/override resolution (headers → properties → 
endpoint)
   - **`PrepareCatalogMojo`** — register `camel-alibaba` parent folder (per 
@davsclaus JIRA comment)
   - **`MojoHelper`** — register all Phase 2 component paths
   - Catalog, endpoint DSL, component DSL, nav.adoc, and doc symlinks 
regenerated
   
   ### Bugbot / Grok review fixes
   
   | Severity | Issue | Fix |
   |----------|-------|-----|
   | High | KMS encrypt sent raw plaintext instead of Base64 | 
`encodePlaintextForEncrypt()` before calling KMS API |
   | High | FC invoke converted binary response to UTF-8 String | Return 
`byte[]` body in result map |
   | Medium | KMS `plaintext` / `ciphertextBlob` not marked secret | Added 
`secret = true`, `security = "secret"` |
   
   ### Follow-ups (out of scope)
   
   - **Spring Boot starters** — per JIRA comment from @davsclaus, starters 
should be added via the `camel-spring-boot` repo `starter-create` script 
(separate PR)
   - EventBridge consumer (MNS/RocketMQ target polling)
   - STS/RAM token support in common module
   - Integration tests with testcontainers
   
   ### Tests
   
   ```bash
   ./mvnw -pl components/camel-alibaba -am test 
-Dtest='org.apache.camel.component.alibaba.**' 
-Dsurefire.failIfNoSpecifiedTests=false
   ```
   
   All alibaba module unit tests passing locally.
   
   ## Target
   
   - [x] Targeting `main` (Camel 4.23.0-SNAPSHOT)
   
   ## Tracking
   
   - [x] [CAMEL-24373](https://issues.apache.org/jira/browse/CAMEL-24373)
   
   ## Apache Camel coding standards
   
   - [x] Meaningful commit messages with JIRA keys
   - [x] Generated catalog/DSL/docs committed
   
   ## AI-assisted contributions
   
   _AI-generated PR on behalf of atiaomar1978-hub (Cursor Agent)_
   


-- 
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