Skip to content

CAMEL-23864: fix edi-x12-as2 and openapi/server examples after YAML normalization#69

Merged
davsclaus merged 1 commit into
apache:mainfrom
ammachado:CAMEL-23864
Jul 1, 2026
Merged

CAMEL-23864: fix edi-x12-as2 and openapi/server examples after YAML normalization#69
davsclaus merged 1 commit into
apache:mainfrom
ammachado:CAMEL-23864

Conversation

@ammachado

Copy link
Copy Markdown
Contributor

Summary

Fixes two camel-jbang-examples integration tests that fail after the YAML canonical-form normalization in 60a70d8. Verified locally against Camel 4.18.2 / Citrus 4.10.1; all 8 workflow suites now pass.

JIRA: CAMEL-23864 (related to CAMEL-23863)

Changes

edi-x12-as2/camel/edi-x12-as2.camel.yaml

  • Restored onException into the as2Error routeConfiguration (with the AS2 disposition-FAILED script handler). Normalization had moved it into the route steps, causing Route receivePurchaseOrder has no output processors at startup.
  • Fixed ediMessageCharset: charset=US-ASCIIUS-ASCII (invalid charset name, IllegalCharsetNameException). Latent since the example was added; only surfaced once the route started correctly.

openapi/server/petstore.camel.yaml

  • Restored the restConfiguration block with clientRequestValidation: true and apiContextPath: openapi. It had been deleted, so the spec was no longer served at /openapi and the test received HTTP 404.

ftp example (maintenance)

  • Moved runtime dependency declarations from jbang.properties (run.deps) to camel.jbang.dependencies in application.properties (natively supported comma-separated list).

Testing

Ran the full build.yml workflow locally:

ftp/test                                PASS
mqtt/test                               PASS
openapi/server/test                     PASS
openapi/client/test                     PASS
aws/aws-s3-event-based/test             PASS
aws/aws-sqs/test                        PASS
edi-x12-as2/test                        PASS
smart-log-analyzer/first-iteration/test PASS

🤖 Generated with Claude Code

…ormalization

The YAML canonical-form normalization in 60a70d8 introduced two test
regressions, both verified against Camel 4.18.2 / Citrus 4.10.1:

- edi-x12-as2: onException was moved out of the as2Error routeConfiguration
  into the route steps, causing "Route receivePurchaseOrder has no output
  processors" at startup. Restored onException (with the AS2 disposition
  FAILED script handler) into the routeConfiguration. Also fixed an invalid
  ediMessageCharset value (charset=US-ASCII -> US-ASCII) that triggered
  IllegalCharsetNameException once the route started correctly.

- openapi/server: the restConfiguration with apiContextPath: openapi was
  deleted, so the spec was no longer served at /openapi and the test got
  HTTP 404. Restored the restConfiguration block.

Also moves the ftp example runtime dependencies from jbang.properties
(run.deps) to camel.jbang.dependencies in application.properties.

All 8 workflow test suites pass.

Related to CAMEL-23863.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@ammachado
ammachado requested review from christophd and davsclaus July 1, 2026 03:35
@ammachado
ammachado marked this pull request as ready for review July 1, 2026 03:40
@davsclaus
davsclaus merged commit 127cc68 into apache:main Jul 1, 2026
1 check passed
@ammachado
ammachado deleted the CAMEL-23864 branch July 1, 2026 23:28
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