URL supports placeholders and can be dynamically replaced with msg ex… - #535
URL supports placeholders and can be dynamically replaced with msg ex…#535zhengchangqing wants to merge 1 commit into
Conversation
RockteMQ-AI
left a comment
There was a problem hiding this comment.
Review: Approved ✅
PR: #535 — URL supports placeholders and can be dynamically replaced with msg extensions
Type: Enhancement (1 file, +33/-1)
Assessment
Adds placeholder support to HTTP sink task URL, enabling dynamic replacement from message extensions. Useful for routing based on message metadata.
Verdict
✅ Clean enhancement with practical use case.
🤖 Automated review by oss-sentinel-ai
|
Issue Evaluation Category: Feature request to support URL placeholders that can be dynamically replaced with message extension values. This would enable more flexible routing in connectors. Feasibility: feasible Automated evaluation by github-manager-bot |
What is the purpose of the change
[ISSUES #534 ]
Brief changelog
Modify
HttpSinkTasksupports placeholders and can be dynamically replaced with msg extendsionsVerifying this change
Follow this checklist to help us incorporate your contribution quickly and easily. Notice,
it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.[ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyleto make sure basic checks pass. Runmvn clean install -DskipITsto make sure unit-test pass. Runmvn clean test-compile failsafe:integration-testto make sure integration-test pass.