Discord.py: create_custom_emoji hangs forever

[Znunu]

Summary

Calling create_custom_emoji while being emote rate-limited, will cause it to never return. Well.... I can only guess that it maybe keeps retrying, but the emote rate limit seems to be 30min!!!!

Reproduction Steps

Spam a server with emotes

Expected Results

Throw an exception

Actual Results

Nothing, bot is stuck

Checklist

• [ x] I have searched the open issues for duplicates.
• [ x] I have shown the entire traceback, if possible.
• [ x] I have removed my token from display, if visible.

[Zomatree]

The lib sleeps until the rate limit is over rather than erroring, see https://github.com/Rapptz/discord.py/issues/4155#issuecomment-636269552

[Znunu]

Awesome, guess every dev has to go through an obligatory one hour of debugging, because of this.

[Rapptz]

Consider not spamming. Rate limits are dynamic and some tend to be long.

[Znunu]

Who said I was spamming? My users might though, and they'd probably like to know why the bot suddenly stopped working. In their defense, the ratelimit triggers very quickly too. Yes, before you say anything, I did work around the issue. It doesn't matter.

[applebee1558]

Bypassing ratelimits is against discord’s api terms of service, I would advise you not to do so.

[applebee1558]

You can use command cooldowns to block users from spamming.

[muddyfish]

A way to return without creating the emoji if it would be rate limited would be useful

[applebee1558]

You are always welcome to make your own bot without a library btw. Nobody at Discord said you had to use library wrappers.

From: Simon Beal [e-mail]
Sent: Friday, June 5, 2020 4:35 AM
To: Rapptz/discord.py [e-mail]
Cc: Alex Liu [e-mail]; Comment [e-mail]
Subject: Re: [Rapptz/discord.py] create_custom_emoji hangs forever (#4173)

A way to return without creating the emoji if it would be rate limited would be useful

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/Rapptz/discord.py/issues/4173#issuecomment-639427097, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHH3VSSG6GGCTJGN6QQOGX3RVDJ53ANCNFSM4NQ3Y7OQ.

[muddyfish]

The point of feature requests is to extend the library. I already have a solution to this problem myself by patching out the retrying for my own purposes, but that's completely not the point

[Rapptz]

As mentioned in the issue I linked, if you want a timeout then add one yourself using wait_for.