apache / eventmesh

EventMesh is a new generation serverless event middleware for building distributed event-driven applications.
https://eventmesh.apache.org/
Apache License 2.0
1.62k stars 642 forks source link

[Feature] Add Feishu sink connector #4420

Closed qqeasonchen closed 1 year ago

qqeasonchen commented 1 year ago

Search before asking

Feature Request

Add Feishu sink connector

Are you willing to submit PR?

Code of Conduct

SunnyBoy-WYH commented 1 year ago

hi,as we know ,feishu is a chat tool , this issue expected send msg card for notification or feishu table?

xwm1992 commented 1 year ago

hi,as we know ,feishu is a chat tool , this issue expected send msg card for notification or feishu table?

The Feishu sink connector expected send msg to Feishu Bot, which you can get the webhook config of Feishu Bot and using in the sink connector.

SunnyBoy-WYH commented 1 year ago

hi,as we know ,feishu is a chat tool , this issue expected send msg card for notification or feishu table?

The Feishu sink connector expected send msg to Feishu Bot, which you can get the webhook config of Feishu Bot and using in the sink connector.

i see the feishu openapi doc, after send msg to bot,bot will send msg to somebody or group? it need user's id ,where we expected config the user's id(tell the bot we expected send to whom)?

Pil0tXia commented 1 year ago

@SunnyBoy-WYH Yes, the message destination is required if necessary.

SunnyBoy-WYH commented 1 year ago

if no required ddl, can you assign it to me temporarily?

Pil0tXia commented 1 year ago

You may submit a PR.

SunnyBoy-WYH commented 1 year ago

we need request to feishu open api, should we use HTTPClientPool tool in eventmesh-runtime module? thanks for your advice~

or we can use feishu sdk? https://github.com/larksuite/oapi-sdk-java/blob/v2_main/README.zh.md,but seems not support send msg: image

Pil0tXia commented 1 year ago

@SunnyBoy-WYH SDK and OpenAPI are both OK, however it is not recommended to add code to runtime that only a certain connector will use.

SunnyBoy-WYH commented 1 year ago

@SunnyBoy-WYH SDK and OpenAPI are both OK, however it is not recommended to add code to runtime that only a certain connector will use.

ok, maybe we can use okhttp? and how can we connectRecord transfer to feishu msg? just let connectRecord.getData() as feishu message ?

xwm1992 commented 1 year ago

@SunnyBoy-WYH SDK and OpenAPI are both OK, however it is not recommended to add code to runtime that only a certain connector will use.

ok, maybe we can use okhttp? and how can we connectRecord transfer to feishu msg? just let connectRecord.getData() as feishu message ?

yes, you can treat the data as feishu message.