Rate-limited when using SlackAPIFileOperator with channel name

[zachliu]

Apache Airflow version

main (development)

If "Other Airflow 2 version" selected, which one?

2.10.4

What happened?

Reopening #44923

Per Slack API rate limits, a sleep(interval) is needed in

airflow/providers/slack/src/airflow/providers/slack/hooks/slack.py

Line 188 in 1addb55

def send_file_v2(

and interval should be an input argument, otherwise we can never use the friendly channel names and have to use the unreadable channel IDs

What you think should happen instead?

A configurable sleep should be used right after

airflow/providers/slack/src/airflow/providers/slack/hooks/slack.py

Line 204 in 1addb55

retried_channel_id = self.get_channel_id(channel_id[1:])

to get around Slack's rate limits

How to reproduce

Use SlackAPIFileOperator with a channel name:

slack_operator_file_content = SlackAPIFileOperator(
    task_id="slack_file_upload",
    dag=dag,
    slack_conn_id="slack",
    channels="#general",
    initial_comment="Hello World!",
    content="file content in txt",
)

Operating System

ubuntu debian

Versions of Apache Airflow Providers

All versions after 8.9.0+

Deployment

Other Docker-based deployment

Deployment details

No response

Anything else?

No response

Are you willing to submit PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[iamkavinnnn]

Hey there,

Can I work on this issue? I can submit a PR and agree to follow the Code of Conduct. I am also new to contributing to opensource software.

Thanks,
Kavin

[potiuk]

Feel free but check with @zachliu if they are not working on it.

[zachliu]

@iamkavinnnn go ahead. i'm not working on it right now

[AshutoshSrivastavaDev]

@iamkavinnnn @zachliu @potiuk if this is still open I would love to contribute here.