Hi there ! I'd like to use `prefect.utilities.notifications.slack_notifier` behind a proxy. A search...

Vincent Chéry

04/23/2021, 1:02 PM

Hi there ! I'd like to use

prefect.utilities.notifications.slack_notifier

behind a proxy. A search in the docs and in the slack community give nothing so before submitting a new issue I wanted to check if I was not missing something ! 🙂 Thanks

Kostas Chalikias

04/23/2021, 1:51 PM

It shouldn't really matter right? It's an outgoing connection from your scheduler to Slack's servers

Vincent Chéry

04/23/2021, 2:05 PM

My scheduler is also behind the proxy so I guess I do need to pass proxy information to the post request ?

Vincent Chéry

04/23/2021, 2:06 PM

I have prefect server running on the same machine as my flows

Kostas Chalikias

04/23/2021, 2:07 PM

oh so you need a proxy for outbound connections?

Kostas Chalikias

04/23/2021, 2:09 PM

it looks to me like the notification handler makes a call using requests.post and there is no proxy hook, unless you can integrate this in the webhook url somehow

Kostas Chalikias

04/23/2021, 2:10 PM

I am guessing it wont be hard to plugin your own to be honest

Kevin Kho

04/23/2021, 2:26 PM

Hi @Vincent Chéry! Thanks for checking in here before opening an issue. Thanks for chiming in @Kostas Chalikias. I understand the need here. You can open an issue, but I honestly can’t guarantee that team will choose to work on it.

Vincent Chéry

04/23/2021, 2:30 PM

Thanks both. I will open the issue then 😉

<http://requests.post|requests.post>

accepts a

proxy

argument, so it looks pretty simple to add a keyword argument to

prefect.utilities.notifications.slack_notifier

and pass it to

<http://requests.post|requests.post>

. I might spend some time to draft a PR for it.

🙌 1

🧑‍💻 1

6 Views

Open in Slack

Previous Next

Prefect Community

Bring your towel and join one of the fastest growing data communities. Welcome to our second-generation open source orchestration platform, a completely rethought approach to dataflow automation.