ferruzzi commented on issue #55237: URL: https://github.com/apache/airflow/issues/55237#issuecomment-3304634547
Thanks for looking into them, that is disappointing. I know we had to go with a 3rd party async library for the smtp one and had hoped there would be options for the others, but I suppose that makes sense since SMTP is a pretty universal standard and not a proprietary protocol. I picked them as high priority because they "felt" like ones that folks would use more often, but I don't have any data to back up that gut feeling. One option that was mentioned elsewhere for another Notifier was making use of a generic HTTP notifier if either of those have an HTTP API that could be used? I think @seanghaeli is planning to work on an HTTP notifier, but maybe if you can get to it first and if it can be used to those two, that could be an option. I'm still pretty new to async stuff in python, which is why I wanted to work on this and learn, so I can't really say offhand if asgiref sync_to_asyn is the right call or not. If you are familiar with it and think it'll work, and we don't have the option of using an API with an async http call then sure, give it a shot. I was going to start on a new AWS SES Notifier (both sync and async versions) in the next couple days by request, so if none of the above sounds good to you, you can claim any of the others that are listed. -- 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]
