Add test to reproduce Invalid server response: 19

[christiangoerdes]

Summary by CodeRabbit

Release Notes

• New Features

  • Added node status tracking to monitor per-destination outcomes during load balancing.
  • Introduced TCP health checks as fallback when HTTP health URLs are not configured.
  • Added schema mappings for JSON Schema validation to reference external schema files.
  • Enhanced JSON-to-XML conversion with configurable root, array, and item element names.

• Documentation

  • Added examples and guide for using schema mappings in JSON Schema validation.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR updates the project version from 6.3.11-SNAPSHOT to 6.3.17-SNAPSHOT across all POM files, introduces exchange-level node status tracking for load balancers, refactors JSON-to-XML conversion with a new utility class, adds schema mapping support for JSON Schema validation, improves health monitoring with TCP checks, and includes comprehensive test/example files for schema mapping scenarios.

Changes

Cohort / File(s)	Summary
Version Updates pom.xml, annot/pom.xml, core/pom.xml, test/pom.xml, war/pom.xml, maven-plugin/pom.xml, distribution/pom.xml, distribution/examples/extending-membrane/*/pom.xml, distribution/examples/web-services-soap/*/pom.xml, distribution/examples/xml/*/pom.xml, .factorypath (core, war), membrane.spec	Updated parent/project versions from 6.3.11-SNAPSHOT to 6.3.17-SNAPSHOT across entire build hierarchy; .gitignore adds two distribution config entries
Node Status Tracking Infrastructure core/src/main/java/com/predic8/membrane/core/exchange/AbstractExchange.java, core/src/main/java/com/predic8/membrane/core/exchange/Exchange.java, core/src/main/java/com/predic8/membrane/core/interceptor/balancer/ExchangeNodeStatusTracker.java, core/src/main/java/com/predic8/membrane/core/interceptor/balancer/LoadBalancingInterceptor.java	Introduces ExchangeNodeStatusTracker class for per-destination status tracking; adds getter/setter/init methods to AbstractExchange; removes old per-destination fields from Exchange; adds configurable trackNodeStatus flag to LoadBalancingInterceptor
Node Status Tracker Usage core/src/main/java/com/predic8/membrane/core/transport/http/HttpClient.java, core/src/main/java/com/predic8/membrane/core/transport/http/client/RetryHandler.java	Replaces direct node status methods with null-safe calls to exc.getNodeStatusTracker() for tracking codes and exceptions
Admin UI & REST API Updates core/src/main/java/com/predic8/membrane/core/interceptor/administration/AdminPageBuilder.java, core/src/main/java/com/predic8/membrane/core/interceptor/administration/AdminRESTInterceptor.java, core/src/main/java/com/predic8/membrane/core/interceptor/administration/DynamicAdminPageInterceptor.java, core/src/main/resources/com/predic8/membrane/core/interceptor/administration/docBase/admin/js/membrane.js	Adds createExchangeNodeStatusTable method; introduces detail flag for selective JSON field emission in REST API; adds node status section to dynamic admin UI; updates JavaScript to render node status tables
Load Balancer Health Monitoring core/src/main/java/com/predic8/membrane/core/interceptor/balancer/BalancerHealthMonitor.java, core/src/main/java/com/predic8/membrane/core/interceptor/balancer/Node.java	Adds TCP health check support via new public tcpHealthy method; replaces hardcoded health endpoint resolution with conditional HTTP/TCP routing; updates Javadoc for health check fallback behavior
Balancer Utility Refactoring core/src/main/java/com/predic8/membrane/core/interceptor/balancer/BalancerUtil.java	Refactors balancer collection/lookup logic to use stream-based aggregation; introduces private allFlows helper; replaces nested loops with functional API; updates error messages for not-found lookups
URL Scheme Handling core/src/main/java/com/predic8/membrane/core/interceptor/DispatchingInterceptor.java	Changes URISyntaxException handling to use router's production flag; adjusts target URL resolution for internal scheme to use URI-based base when needed; normalizes slash handling for relative paths
JSON-to-XML Conversion Refactoring core/src/main/java/com/predic8/membrane/core/interceptor/xml/Json2XmlInterceptor.java, core/src/main/java/com/predic8/membrane/core/util/json/JsonToXml.java	Introduces new JsonToXml utility class with fluent configuration (rootName, arrayName, itemName); refactors Json2XmlInterceptor to use external converter; adds public setters/getters for array/item names; replaces inline conversion logic
JSON Schema Validation & Schema Mappings core/src/main/java/com/predic8/membrane/core/interceptor/schemavalidation/ValidatorInterceptor.java, core/src/main/java/com/predic8/membrane/core/interceptor/schemavalidation/json/JSONYAMLSchemaValidator.java, core/src/main/java/com/predic8/membrane/core/interceptor/schemavalidation/json/SchemaMappings.java	Adds SchemaMappings class for structured schema reference mapping; integrates schema mapping support into ValidatorInterceptor and JSONYAMLSchemaValidator; adds public API for setReferenceSchemas/getReferenceSchemas
JWT/JWKS Configuration core/src/main/java/com/predic8/membrane/core/interceptor/jwt/Jwks.java, core/src/main/java/com/predic8/membrane/core/interceptor/jwt/JwtAuthInterceptor.java	Changes Jwks.init signature to accept Router instead of separate resolver/location parameters; adds per-JWK HttpClientConfiguration support; updates Jwk.getJwk to use router-based resolution
Utility & Logging Updates core/src/main/java/com/predic8/membrane/core/interceptor/log/access/AccessLogInterceptorService.java, core/src/main/java/com/predic8/membrane/core/transport/http/ByteStreamLogging.java, core/src/main/java/com/predic8/membrane/core/util/json/JsonUtil.java	Replaces SimpleDateFormat with thread-safe DateTimeFormatter; improves byte stream logging with line separators and guard clauses; relocates JsonUtil from com.predic8.membrane.core.util to com.predic8.membrane.core.util.json package
OpenAPI Validators core/src/main/java/com/predic8/membrane/core/openapi/validators/StringValidator.java, core/src/main/java/com/predic8/membrane/core/openapi/validators/parameters/AbstractArrayParameterParser.java, core/src/main/java/com/predic8/membrane/core/openapi/validators/parameters/ExplodedObjectParameterParser.java, core/src/main/java/com/predic8/membrane/core/openapi/validators/parameters/ObjectParameterParser.java, core/src/main/java/com/predic8/membrane/core/openapi/validators/parameters/ScalarParameterParser.java	Adds context-sensitive coercion for query parameter string validation in StringValidator; updates all parameter parsers to import JsonUtil from relocated package
Test & Example Updates core/src/test/java/com/predic8/membrane/core/interceptor/flow/invocation/internalservice/*Test.java, core/src/test/java/com/predic8/membrane/core/interceptor/xml/Json2XmlInterceptorTest.java, core/src/test/java/com/predic8/membrane/core/transport/http/ConnectionKeepAliveTest.java, core/src/test/java/com/predic8/membrane/core/transport/http/ServiceInvocationTest.java, core/src/test/java/com/predic8/membrane/core/util/JsonUtilTest.java, core/src/test/java/com/predic8/membrane/core/util/json/JsonToXmlTest.java, core/src/test/resources/internal-invocation/proxies.xml	Updates internal service routing URLs from internal: to internal:// format; adds comprehensive JsonToXmlTest suite; enhances Json2XmlInterceptorTest with new helper and expanded coverage; adds new ConnectionKeepAliveTest; updates proxy configurations for internal routing
Schema Mapping Examples distribution/examples/validation/json-schema/schema-mappings/*, distribution/examples/validation/json-schema/schema-mappings/schemas/*.json, distribution/src/test/java/com/predic8/membrane/examples/withoutinternet/validation/JSONSchemaMappingsExampleTest.java	Adds comprehensive example demonstrating schema mappings with proxies configuration, shell/batch scripts, sample JSON payloads, OpenAPI specification, and test suite validating good/bad request scenarios across two API endpoints
Build & Shell Script Updates distribution/scripts/start_router.sh	Restricts JAVA_OPTS normalization to only -Dlog4j.configurationFile key instead of all -D*=* entries

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

Possibly related PRs

• show load balancer node status (and failure reasons) in adminConsole (#2566) #2582 — Directly related: Both PRs modify the same node status tracking infrastructure (AbstractExchange, Exchange, ExchangeNodeStatusTracker, LoadBalancingInterceptor, HttpClient, RetryHandler, admin UI/REST serialization) with identical structural changes
• json2xml: Support for Arrays #2351 — Directly related: Both PRs refactor JSON-to-XML conversion (introducing JsonToXml utility, refactoring Json2XmlInterceptor, and adding comprehensive test coverage)
• Simplify jsonschema example #2504 — Directly related: Both PRs add the same distribution example for schema mappings with identical proxies, test scenarios, and example JSON payloads

Suggested labels

Prio-1

Suggested reviewers

• predic8

Poem

🐰 From versions old to snapshots new,
Node status trackers dream come true!
JSON dances into XML's embrace,
Schema mappings find their perfect place.
Health checks hop—TCP or HTTP—
A gateway's joy in every step! 🎉

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}