Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Mod] Hidden error in slowmode with large ints #4019

Closed
Flame442 opened this issue Jun 26, 2020 · 0 comments · Fixed by #4630
Closed

[Mod] Hidden error in slowmode with large ints #4019

Flame442 opened this issue Jun 26, 2020 · 0 comments · Fixed by #4630
Labels
Category: Core - API - Commands Package This is related to the `redbot.core.commands` package or `redbot.core.checks` module. Status: In Progress There's a PR open to deal with this, or an org member is working on this internally Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing.

Comments

@Flame442
Copy link
Member

Command bugs

Command name

[p]slowmode 10000000000000000000000
Technically this is an issue with commands.TimedeltaConverter's handling of large ints

What cog is this command from?

Mod

What happened?

Nothing in chat, this error in console

[2020-06-26 02:30:54] [ERROR] red: ConversionError
Traceback (most recent call last):
  File "\redbot\core\commands\converter.py", line 287, in convert
    delta = parse_timedelta(
  File "\redbot\core\commands\converter.py", line 115, in parse_timedelta
    delta = timedelta(**params)
OverflowError: Python int too large to convert to C int
@Flame442 Flame442 added the Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing. label Jun 26, 2020
@github-actions github-actions bot added the Status: Needs Triage This has not been labeled or discussed for handling yet. label Jun 26, 2020
@Drapersniper Drapersniper added Category: Bot Core Status: Accepted We want this and removed Status: Needs Triage This has not been labeled or discussed for handling yet. labels Jun 26, 2020
@Jackenmen Jackenmen added Category: Core - API - Commands Package This is related to the `redbot.core.commands` package or `redbot.core.checks` module. and removed Category: Bot Core labels Jun 30, 2020
@Kowlin Kowlin mentioned this issue Nov 20, 2020
Merged
3 tasks
@Jackenmen Jackenmen added Status: In Progress There's a PR open to deal with this, or an org member is working on this internally and removed Status: Accepted We want this labels Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Category: Core - API - Commands Package This is related to the `redbot.core.commands` package or `redbot.core.checks` module. Status: In Progress There's a PR open to deal with this, or an org member is working on this internally Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Catch OverflowErrors in the timedelta converter Kowlin/Red-DiscordBot
3 participants
@Jackenmen @Drapersniper @Flame442