You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2020/11/05 03:30:44 UTC

[GitHub] [incubator-dolphinscheduler] samz406 opened a new issue #4027: [Feature][SPI] Implement SMS Alert Plugin

samz406 opened a new issue #4027:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/4027


   Implement SMS  Alert Plugin
   
   #3049
   
   I will finish it.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-dolphinscheduler] samz406 commented on issue #4027: [Feature][SPI] Implement SMS Alert Plugin

Posted by GitBox <gi...@apache.org>.
samz406 commented on issue #4027:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/4027#issuecomment-723005080


   Hi.You are right. We are using short message service from two suppliers in our current project. They are called through HTTP request, and the request parameters are different. What I want is to send through HTTP request. Request parameters are not dead written in the configuration. Load the required configuration at startup and try to make it universal. Do you think this is feasible. No, I'll close the issue
   
   
   你说的有道理,我们目前项目中用的有两家供应商的短息服务。都是通过http请求方式调用,请求参数有差别。我想的是通过http请求方式实现发送功能。请求参数在配置中不写死。启动时加载所需配置,尽量做成通用。你们觉得这样是否可行。不行我就关闭issue


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-dolphinscheduler] gaojun2048 commented on issue #4027: [Feature][SPI] Implement SMS Alert Plugin

Posted by GitBox <gi...@apache.org>.
gaojun2048 commented on issue #4027:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/4027#issuecomment-723016607


   If you are use http/https to send message . You need a http/https alert plugin. So you can implement a http/https plugin .


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-dolphinscheduler] samz406 edited a comment on issue #4027: [Feature][SPI] Implement SMS Alert Plugin

Posted by GitBox <gi...@apache.org>.
samz406 edited a comment on issue #4027:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/4027#issuecomment-723005080


   Hi.You are right. We are using short message service from two suppliers in our current project. They are called through HTTP request, and the request parameters are different. What I want is to send through HTTP request. Request parameters are not dead written in the configuration. Load the required configuration at startup and try to make it universal. Do you think this is feasible. No, I'll close the issue
   
   
   你说的有道理,我们目前项目中用的有两家供应商的短息服务。都是通过http请求方式调用,请求参数有差别。我想的是通过http请求方式实现发送功能。请求参数在配置中不写死。启动时加载所需配置,尽量做成通用。你们觉得这样是否可行。不行我就关闭
   @gaojun2048 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-dolphinscheduler] CalvinKirs commented on issue #4027: [Feature][SPI] Implement SMS Alert Plugin

Posted by GitBox <gi...@apache.org>.
CalvinKirs commented on issue #4027:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/4027#issuecomment-723356060


   > Hi.You are right. We are using short message service from two suppliers in our current project. They are called through HTTP request, and the request parameters are different. What I want is to send through HTTP request. Request parameters are not dead written in the configuration. Load the required configuration at startup and try to make it universal. Do you think this is feasible. No, I'll close the issue
   > 
   > 你说的有道理,我们目前项目中用的有两家供应商的短息服务。都是通过http请求方式调用,请求参数有差别。我想的是通过http请求方式实现发送功能。请求参数在配置中不写死。启动时加载所需配置,尽量做成通用。你们觉得这样是否可行。不行我就关闭
   > @gaojun2048
   
   very good!
   
   I checked it again today, and there is no difference in the process. They are all in the form of http. For the time being, there are no additional SMS vendors that need to dynamically obtain the token first. As you said, I think we can solve the difference in parameters.
   
   very good!
   
   我今天核查了一遍,流程上没有什么差异,都是通过http的形式,暂时没有那种额外的需要先动态获取token的短信厂商。正如你所说,参数上的差异我想我们是可以解决的。


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-dolphinscheduler] CalvinKirs commented on issue #4027: [Feature][SPI] Implement SMS Alert Plugin

Posted by GitBox <gi...@apache.org>.
CalvinKirs commented on issue #4027:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/4027#issuecomment-722354096


   Hi, as far as I know, SMS vendors have not formed a set of standards. Can we provide implementation? If not, it can only be implemented by the user.
   
   嗨,据我所知,目前短信厂商并没有形成一套标准,我们是否可以提供实现呢?如果不能的话,只能用户自己实现了。


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-dolphinscheduler] CalvinKirs closed issue #4027: [Feature][SPI] Implement SMS Alert Plugin

Posted by GitBox <gi...@apache.org>.
CalvinKirs closed issue #4027:
URL: https://github.com/apache/incubator-dolphinscheduler/issues/4027


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org