ramu11 commented on PR #27686: URL: https://github.com/apache/camel/pull/27686#issuecomment-6129127110
Thanks for the detailed review. I've addressed the outstanding code and documentation findings in commit `2424f9310144`. - Re-wrap URI-sensitive option values in `RAW(...)` to prevent query corruption, with regression coverage for special characters and existing RAW/property-placeholder behavior. - Replace the producer-only `source.block` documentation example with `source.bridgeErrorHandler`. - Preserve original option names, including camelCase keys such as `pageSize`. - Exclude nested `kamelet:` destinations from advanced-option injection and fail clearly when no eligible non-Kamelet destination exists. - Remove the redundant exception handler and keep the changes scoped to the feature. - Synchronize the source and generated catalog documentation. Validation completed: - `KameletAdvancedOptionsTest`: 27 tests, 0 failures, 0 errors. - `camel-kamelet` module: 121 tests, 0 failures, 0 errors, 2 skipped. - `git diff --check` passed. The changes are committed. Please let me know if anything remains unresolved. -- 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]
