-
Notifications
You must be signed in to change notification settings - Fork 635
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature] Add Webhook support for HTTP Source Connector #4869
Labels
Comments
6 tasks
Pil0tXia
changed the title
Use the existing implementation of Webhook as an enhancement to the HTTP Source Connector, needs to be able to support the user's own implementation of a customized Webhook protocol.
[Feature] Add Webhook support for HTTP Source Connector
Apr 30, 2024
@Pil0tXia Please assign it to me |
xwm1992
pushed a commit
that referenced
this issue
Jun 27, 2024
* feat: add webhook support * feat: add CommonProtocol and optimize some logic * doc: update some javaDoc * fix: update something * fix: Optimize queue related operations * fix: Optimize queue related operations * fix: Optimize queue related operations * fix: update something * fix: Remove some meaningless code * fix: update something * perf: Updates to protocol-related code, with a focus on request handling * perf: use CircularFifoQueue * perf: Use OkHttp Client to completely replace Apache Client and resolve code conflicts.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Search before asking
Feature Request
This issue is a subtask of #4825.
Referencing the existing
eventmesh-webhook
module and its documentation, support the following functionalities:org.apache.eventmesh.webhook.receive.protocol.GithubProtocol
. Another third-party platform can be supported by implementing theAnotherProtocol
class)."1+n" protocol support: 1 for a common standard Webhook protocol, and n for special non-standard Webhook protocols. Users need to select a protocol in the
source-config.yml
.Are you willing to submit PR?
Code of Conduct
The text was updated successfully, but these errors were encountered: